feat: support esm tests in electron main thread (#921)

Swaps `electron-mocha` for `electron-mocha-main` that lets us run ESM tests in the electron main thread.  Running them in the renderer thread is still not possible.

Fixes #918

Author: achingbrain

package.json

@@ -82,7 +82,7 @@
     "dependency-check": "^5.0.0-2",
     "detective-cjs": "^3.1.1",
     "detective-es6": "^2.2.0",
-    "electron-mocha": "^10.0.0",
+    "electron-mocha-main": "^11.0.3",
     "env-paths": "^2.2.1",
     "esbuild": "^0.12.15",
     "esbuild-register": "^2.3.0",

src/test/browser.js

@@ -28,8 +28,8 @@ module.exports = async (argv, execaOptions) => {
   const files = argv.files.length > 0
     ? argv.files
     : [
-        '**/*.spec.{js,ts}',
-        'test/browser.{js,ts}'
+        '**/*.spec.{js,ts,cjs,mjs}',
+        'test/browser.{js,ts,cjs,mjs}'
       ]
 
   // before hook

src/test/electron.js

@@ -17,7 +17,7 @@ const merge = require('merge-options')
 module.exports = async (argv, execaOptions) => {
   const forwardOptions = argv['--'] ? argv['--'] : []
   const watch = argv.watch ? ['--watch'] : []
-  const files = argv.files.length > 0 ? [...argv.files] : ['test/**/*.spec.{js,ts}']
+  const files = argv.files.length > 0 ? [...argv.files] : ['test/**/*.spec.{js,ts,mjs,cjs}']
   const grep = argv.grep ? ['--grep', argv.grep] : []
   const progress = argv.progress ? ['--reporter=progress'] : []
   const bail = argv.bail ? ['--bail'] : []

src/test/node.js

@@ -31,8 +31,8 @@ async function testNode (argv, execaOptions) {
   const files = argv.files.length > 0
     ? argv.files
     : [
-        'test/node.{js,ts}',
-        'test/**/*.spec.{js,ts}'
+        'test/node.{js,ts,cjs,mjs}',
+        'test/**/*.spec.{js,ts,cjs,mjs}'
       ]
 
   const args = [

src/test/react-native.js

@@ -27,8 +27,8 @@ module.exports = async (argv, execaOptions) => {
   const files = argv.files.length > 0
     ? argv.files
     : [
-        '**/*.spec.{js,ts}',
-        'test/browser.{js,ts}'
+        '**/*.spec.{js,ts,cjs,mjs}',
+        'test/browser.{js,ts,cjs,mjs}'
       ]
 
   // before hook

test/fixtures/esm/an-esm-project/test/esm-test.spec.js

@@ -0,0 +1,9 @@
+/* eslint-env mocha */
+import { useHerp, useDerp } from '../src/index.js'
+
+describe('esm test', () => {
+  it('runs an esm test', () => {
+    useHerp()
+    useDerp()
+  })
+})

test/node.js

@@ -8,3 +8,4 @@ require('./dependency-check')
 require('./utils/echo-server')
 require('./utils/get-port')
 require('./config/user')
+require('./test')

test/test.js

@@ -0,0 +1,28 @@
+/* eslint-env mocha */
+'use strict'
+
+const execa = require('execa')
+const { copy } = require('fs-extra')
+const { join } = require('path')
+const bin = require.resolve('../')
+const tempy = require('tempy')
+
+describe('test', () => {
+  describe('esm', function () {
+    let projectDir = ''
+
+    before(async () => {
+      projectDir = tempy.directory()
+
+      await copy(join(__dirname, 'fixtures', 'esm', 'an-esm-project'), projectDir)
+    })
+
+    it('should test an esm project', async function () {
+      this.timeout(20 * 1000) // slow ci is slow
+
+      await execa(bin, ['test'], {
+        cwd: projectDir
+      })
+    })
+  })
+})