skip NodeNext tests on incompatible node versions

Author: cspotcode

src/test/helpers.ts

@@ -68,6 +68,14 @@ export const nodeSupportsImportAssertions = semver.gte(
   process.version,
   '17.1.0'
 );
+// Node 14.13.0 has a bug where it tries to lex CJS files to discover named exports *before*
+// we transform the code.
+// In other words, it tries to parse raw TS as CJS and balks at `export const foo =`, expecting to see `exports.foo =`
+// This lexing only happens when CJS TS is imported from the ESM loader.
+export const nodeSupportsImportingTransformedCjsFromEsm = semver.gte(
+  process.version,
+  '14.13.1'
+);
 /** Supports tsconfig "extends" >= v3.2.0 */
 export const tsSupportsTsconfigInheritanceViaNodePackages = semver.gte(
   ts.version,

src/test/module-node.spec.ts

@@ -2,6 +2,7 @@ import { expect, context } from './testlib';
 import {
   CMD_TS_NODE_WITHOUT_PROJECT_FLAG,
   isOneOf,
+  nodeSupportsImportingTransformedCjsFromEsm,
   resetNodeEnvironment,
   tsSupportsStableNodeNextNode16,
 } from './helpers';
@@ -18,7 +19,9 @@ type Test = typeof test;
 
 // Declare one test case for each permutations of project configuration
 test.suite('TypeScript module=NodeNext and Node16', (test) => {
-  test.runIf(tsSupportsStableNodeNextNode16);
+  test.runIf(
+    tsSupportsStableNodeNextNode16 && nodeSupportsImportingTransformedCjsFromEsm
+  );
 
   for (const allowJs of [true, false]) {
     for (const typecheckMode of [
@@ -63,6 +66,7 @@ function declareTest(test: Test, testParams: TestParams) {
     t.log(stdout);
     t.log(stderr);
     expect(err).toBe(null);
+    expect(stdout).toMatch(/done\n$/);
   });
 }
 
@@ -216,6 +220,8 @@ function writeFixturesToFilesystem(name: string, testParams: TestParams) {
     }
   }
 
+  indexFile.content += `console.log('done');\n`;
+
   proj.rm();
   proj.write();
   return proj;