Removed explicit use of ts-node (#1125)

* Remove explicit use of ts-node, and dependency. Speed tests. * Fix typo
tj · Dec 20, 2019 · 808d4bc · 808d4bc
1 parent 9de0968
commit 808d4bc
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 44 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -32,7 +32,6 @@
     "eslint-plugin-jest": "^22.21.0",
     "jest": "^24.8.0",
     "standard": "^14.3.1",
-    "ts-node": "^8.5.2",
     "typescript": "^3.7.2"
   },
   "typings": "typings/index.d.ts",

diff --git a/tests/command.executableSubcommand.lookup.test.js b/tests/command.executableSubcommand.lookup.test.js
@@ -89,9 +89,11 @@ testOrSkipOnWindows('when subcommand file is double symlink then lookup succeeds
 });
 
 test('when subcommand suffix is .ts then lookup succeeds', (done) => {
-  jest.setTimeout(20000); // Extend timeout for GitHub Actions
+  // We support looking for ts files for ts-node in particular, but don't need to test ts-node itself.
+  // The program and the subcommand `pm-install.ts` are both plain JavaScript code.
   const binLinkTs = path.join(__dirname, 'fixtures-ts', 'pm.ts');
-  childProcess.execFile('node', ['-r', 'ts-node/register', binLinkTs, 'install'], function(_error, stdout, stderr) {
+  // childProcess.execFile('node', ['-r', 'ts-node/register', binLinkTs, 'install'], function(_error, stdout, stderr) {
+  childProcess.execFile('node', [binLinkTs, 'install'], function(_error, stdout, stderr) {
     expect(stdout).toBe('install\n');
     done();
   });