feat: ability to read tests in ".jsx" and ".tsx" files

README.md

@@ -1300,7 +1300,7 @@ The maximum number of tests to be run in one worker before it will be restarted.
 By default, `hermione` will run all browsers simultaneously. Sometimes (i.e. when using cloud services, such as SauceLabs) you have to limit the amount of browsers that can be run at the same time. This option effectively limits how many browsers `hermione` will try to run in parallel. Default value is `Infinity`.
 
 #### fileExtensions
-Ability to set file extensions, which hermione will search on the file system. Default value is `[.js]`.
+Ability to set file extensions, which hermione will search on the file system. Default value is `[".js", ".mjs", ".ts", ".mts", ".jsx", ".tsx"]`.
 
 ### plugins
 `Hermione` plugins are commonly used to extend built-in functionality. For example, [html-reporter](https://github.com/gemini-testing/html-reporter) and [hermione-safari-commands](https://github.com/gemini-testing/hermione-safari-commands).
@@ -1621,8 +1621,6 @@ Using `-r` or `--require` option you can load external modules, which exists in
 - compilers such as TypeScript via [ts-node](https://www.npmjs.com/package/ts-node) (using `--require ts-node/register`) or Babel via [@babel/register](https://www.npmjs.com/package/@babel/register) (using `--require @babel/register`);
 - loaders such as ECMAScript modules via [esm](https://www.npmjs.com/package/esm).
 
-Be sure to update [fileExtensions](#fileExtensions) apropriately, if you are planning to import anything other than `.js`.
-
 ### Overriding settings
 
 All options can also be overridden via command-line flags or environment variables. Priorities are the following:

package.json

@@ -44,6 +44,10 @@
   },
   "license": "MIT",
   "dependencies": {
+    "@babel/core": "7.24.0",
+    "@babel/plugin-transform-modules-commonjs": "7.23.3",
+    "@babel/plugin-transform-react-jsx": "7.23.4",
+    "@babel/preset-react": "7.23.3",
     "@gemini-testing/commander": "2.15.3",
     "@types/mocha": "10.0.1",
     "@wdio/globals": "8.21.0",
@@ -66,6 +70,7 @@
     "looks-same": "9.0.0",
     "micromatch": "4.0.5",
     "mocha": "10.2.0",
+    "pirates": "4.0.6",
     "plugins-loader": "1.2.0",
     "png-validator": "1.1.0",
     "sharp": "0.30.7",
@@ -84,6 +89,7 @@
     "@commitlint/config-conventional": "17.1.0",
     "@sinonjs/fake-timers": "10.3.0",
     "@swc/core": "1.3.40",
+    "@types/babel__core": "7.20.5",
     "@types/bluebird": "3.5.38",
     "@types/chai": "4.3.4",
     "@types/chai-as-promised": "7.1.5",

src/config/defaults.js

@@ -81,7 +81,7 @@ module.exports = {
     resetCursor: true,
     strictTestsOrder: false,
     saveHistoryMode: SAVE_HISTORY_MODE.ALL,
-    fileExtensions: [".js", ".mjs", ".ts", ".mts"],
+    fileExtensions: [".js", ".mjs", ".ts", ".mts", ".jsx", ".tsx"],
     outputDir: null,
     agent: null,
     headers: null,

src/test-reader/test-parser.js

@@ -8,6 +8,7 @@ const { TreeBuilder } = require("./tree-builder");
 const { readFiles } = require("./mocha-reader");
 const { TestReaderEvents } = require("../events");
 const { TestParserAPI } = require("./test-parser-api");
+const { setupTransformHook } = require("./test-transformer");
 const { MasterEvents } = require("../events");
 const _ = require("lodash");
 const clearRequire = require("clear-require");
@@ -47,9 +48,13 @@ class TestParser extends EventEmitter {
 
         this.#clearRequireCache(files);
 
+        const revertTransformHook = setupTransformHook();
+
         const rand = Math.random();
         const esmDecorator = f => f + `?rand=${rand}`;
         await readFiles(files, { esmDecorator, config: mochaOpts, eventBus });
+
+        revertTransformHook();
     }
 
     #applyInstructionsEvents(eventBus) {

src/test-reader/test-transformer.ts

@@ -0,0 +1,47 @@
+import * as babel from "@babel/core";
+import { addHook } from "pirates";
+
+import type { NodePath, PluginObj, TransformOptions } from "@babel/core";
+import type { ImportDeclaration } from "@babel/types";
+
+const TRANSFORM_EXTENSIONS = [".jsx", ".tsx"];
+
+export const setupTransformHook = (): VoidFunction => {
+    const transformOptions: TransformOptions = {
+        browserslistConfigFile: false,
+        babelrc: false,
+        configFile: false,
+        compact: false,
+        plugins: [
+            [
+                require("@babel/plugin-transform-react-jsx"),
+                {
+                    throwIfNamespace: false,
+                    runtime: "automatic",
+                },
+            ],
+            require("@babel/plugin-transform-modules-commonjs"),
+            [
+                (): PluginObj => ({
+                    name: "remove-css-imports",
+                    visitor: {
+                        ImportDeclaration(path: NodePath<ImportDeclaration>): void {
+                            if (path.node.source.value.match(/\.(css|less|scss)$/)) {
+                                path.remove();
+                            }
+                        },
+                    },
+                }),
+            ],
+        ],
+    };
+
+    const revertTransformHook = addHook(
+        (originalCode, filename) => {
+            return babel.transform(originalCode, { filename, ...transformOptions })!.code as string;
+        },
+        { exts: TRANSFORM_EXTENSIONS },
+    );
+
+    return revertTransformHook;
+};

test/src/test-reader/test-parser.js

@@ -22,18 +22,21 @@ describe("test-reader/test-parser", () => {
     let TestParser;
     let clearRequire;
     let readFiles;
+    let setupTransformHook;
     let browserVersionController = {
         mkProvider: sinon.stub().returns(() => {}),
     };
 
     beforeEach(() => {
         clearRequire = sandbox.stub().named("clear-require");
         readFiles = sandbox.stub().named("readFiles").resolves();
+        setupTransformHook = sandbox.stub().named("setupTransformHook").returns(sinon.stub());
 
         TestParser = proxyquire("src/test-reader/test-parser", {
             "clear-require": clearRequire,
             "./mocha-reader": { readFiles },
             "./controllers/browser-version-controller": browserVersionController,
+            "./test-transformer": { setupTransformHook },
         }).TestParser;
 
         sandbox.stub(InstructionsList.prototype, "push").returnsThis();
@@ -369,6 +372,23 @@ describe("test-reader/test-parser", () => {
                     assert.notEqual(firstCallModuleName, lastCallModuleName);
                 });
             });
+
+            describe("transform hook", () => {
+                it("should setup hook before read files", async () => {
+                    await loadFiles_({ files: ["foo/bar", "baz/qux"] });
+
+                    assert.callOrder(setupTransformHook, readFiles);
+                });
+
+                it("should call revert transformation after read files", async () => {
+                    const revertFn = sinon.stub();
+                    setupTransformHook.returns(revertFn);
+
+                    await loadFiles_({ files: ["foo/bar", "baz/qux"] });
+
+                    assert.callOrder(readFiles, revertFn);
+                });
+            });
         });
 
         describe("root suite decorators", () => {

test/src/test-reader/test-transformer.ts

@@ -0,0 +1,58 @@
+import * as pirates from "pirates";
+import proxyquire from "proxyquire";
+import sinon, { SinonStub } from "sinon";
+
+type SetupTransformHook = typeof import("../../../src/test-reader/test-transformer").setupTransformHook;
+
+describe("test-transformer", () => {
+    const sandbox = sinon.createSandbox();
+    let babelTransform: SinonStub;
+    let setupTransformHook: SetupTransformHook;
+
+    beforeEach(() => {
+        babelTransform = sandbox.stub().returns({ code: "default-code" });
+
+        sandbox.stub(pirates, "addHook").returns(sandbox.stub());
+
+        ({ setupTransformHook } = proxyquire("src/test-reader/test-transformer", {
+            "@babel/core": { transform: babelTransform },
+        })).default;
+    });
+
+    afterEach(() => sandbox.restore());
+
+    describe("setupTransformHook", () => {
+        it("should transform '.jsx' and '.tsx' files", () => {
+            setupTransformHook();
+
+            assert.calledOnceWith(pirates.addHook, sinon.match.any, { exts: [".jsx", ".tsx"] });
+        });
+
+        it("should return function to revert transformation", () => {
+            const revertFn = sandbox.stub();
+            (pirates.addHook as SinonStub).returns(revertFn);
+
+            assert.equal(setupTransformHook(), revertFn);
+        });
+
+        it("should transform code using babel", () => {
+            let transformedCode;
+            babelTransform.returns({ code: "transformed-code" });
+            (pirates.addHook as SinonStub).callsFake(cb => {
+                transformedCode = cb("original-code", "/path/test.jsx");
+            });
+
+            setupTransformHook();
+
+            assert.calledOnceWith(babelTransform, "original-code", {
+                browserslistConfigFile: false,
+                babelrc: false,
+                configFile: false,
+                compact: false,
+                plugins: sinon.match.array,
+                filename: "/path/test.jsx",
+            });
+            assert.equal(transformedCode, "transformed-code");
+        });
+    });
+});