Merge pull request #239 from boxine/fix-ts

Fix require hooks not working anymore

Author: marvinhagemeister

package-lock.json

@@ -120,6 +120,7 @@
     "prettier": "^2.0.5",
     "puppeteer": "^5.2.1",
     "rimraf": "^3.0.2",
+    "ts-node": "^9.0.0",
     "typedoc": "^0.17.4",
     "typescript": "^3.8.3"
   },

package.json

@@ -32,7 +32,10 @@ async function importFile(file) {
         canUseImport = await supportsImports();
     }
 
-    if (canUseImport) {
+    // Only use import() for JavaScript files. Patching module
+    // resolution of import() calls is still very experimental, so
+    // tools like `ts-node´ need to keep using `require` calls.
+    if (/\.[cm]?js$/.test(file) && canUseImport) {
         // Use dynamic import statement to be able to load both native esm
         // and commonjs modules.
 

src/loader.js

@@ -0,0 +1,24 @@
+const assert = require('assert').strict;
+const path = require('path');
+const child_process = require('child_process');
+
+async function run() {
+    const script = path.join(__dirname, '..', 'bin', 'cli.js');
+    const {stderr} = await new Promise((resolve, reject) => {
+        child_process.execFile(
+            'node',
+            ['-r', 'ts-node/register', script, '--exit-zero', '--no-screenshots', '--tests-glob', 'tests/ts_node/*.ts'],
+            (err, stdout, stderr) => {
+                if (err) reject(err);
+                else resolve({stdout, stderr});
+            }
+        );
+    });
+
+    assert(/1 tests passed/.test(stderr), 'Did run any tests');
+}
+
+module.exports = {
+    description: 'Support node require hooks',
+    run,
+};

tests/selftest_ts_node.js

@@ -0,0 +1,8 @@
+#!/usr/bin/env node
+
+const pentf = require('../../src/index.js');
+
+pentf.main({
+    rootDir: __dirname,
+    testsDir: __dirname,
+});

tests/ts_node/run

@@ -0,0 +1,13 @@
+import {newPage, waitForText} from "../../src/browser_utils";
+
+async function run(config) {
+    const page = await newPage(config);
+    await page.setContent('<h1>ts-node</h1>');
+
+    await waitForText(page, 'ts-node');
+}
+
+module.exports = {
+    description: 'Pass a simple TypeScript test',
+    run,
+};