`invalid moduleId` error when using tsx about esmock

[iambumblehead]

At the PR, I removed the invalid moduleId test, because the moduleId was actually invalid (the file path being imported did not exist) and esmock is expected to throw an error in that situation. That is separate from the unknown-file-extension-error which seems to be a real issue.

[tommy-mitchell]

Nice find.

[iambumblehead]

@tonisives respectfully, the example you gave probably does not use esmock correctly. Call esmock to mock values that are imported by the target module. See this diff, for example, which results in a passing test using the two files attached further below.

diff --git a/src/simpletest.test.ts b/src/simpletest.test.ts
index a1633c6..13a3370 100644
--- a/src/simpletest.test.ts
+++ b/src/simpletest.test.ts
@@ -1,13 +1,14 @@
 import { test } from "uvu"
 import esmock from "esmock"
-import { myFun } from "./simpletest.js"
+import simpletest from "./simpletest.js"
 
 test("test not real", async () => {
   let mock = await esmock("./simpletest.js", {
-    myFun: () => console.log("mocked")
+    "./simplefunctions.js": { myFun: () => console.log("mocked") }
   })
-  
-  myFun()
+
+  mock()
+  simpletest()
 })
 
 test.run()

src/simplefunctions.ts

export const myFun = () => {
  console.log("still real");
}

src/simpletest.ts

import { myFun } from './simplefunctions.js'

export default myFun

[iambumblehead]

There's not an esmock tsx test and it would be great to have one. Feel free to submit a PR, otherwise I'll respond in a week or so with a solution.

related,

• current tests folders https://github.com/iambumblehead/esmock/tree/master/tests
• https://github.com/iambumblehead/esmock/blob/master/CONTRIBUTING.md

[iambumblehead]

it would be awesome if you would submit a PR with a failing test

[tommy-mitchell]

Added failing tests at #210.

[iambumblehead]

@tommy-mitchell thank you for the test cases. Possibly related, ava is not fully compatible with esm --loader

• avajs/ava#3195
• avajs/ava#3211

Will take a look later this evening

[iambumblehead]

this seems to come from a configuration issue, such as the configurations discussed here https://stackoverflow.com/questions/62096269/cant-run-my-node-js-typescript-project-typeerror-err-unknown-file-extension

the error occurs at this line, where importing the ts file causes the error

const importedModule = await import(fileURLKey)
//   TypeError {
//     code: 'ERR_UNKNOWN_FILE_EXTENSION',
//     message: 'Unknown file extension ".ts" for /home/bumble/software/esmock/tests/local/main.ts',

I'm not great at setting up configurations for things like this but will keep trying a little bit

[iambumblehead]

PR is here #211

[iambumblehead]

this issue seems related avajs/ava#2593 (comment)

[tommy-mitchell]

Potentially related too: avajs/ava#3213 (reply in thread)

[iambumblehead]

these results indicate the problem may come from tsx,

• if tsx is used, the tests fail if either node:test or ava are used as the test runner
• if ts-node/esm is used, the tests pass if either node:test or ava are used as the test runner

I'll push up some changes to the PR that make it a little easier to switch between the two transpilers and test-runners

[iambumblehead]

@tommy-mitchell I think I've narrowed down the issue to something that could be used to submit an issue at the tsx project,

test('tsx should not break when import paths include query params', async () => {
  assert.ok(await import('../local/main.ts?n=4'))
})

when query params are removed, the above test passes

[iambumblehead]

related issue is opened here esbuild-kit/tsx#264

[iambumblehead]

Closing with this #211

A tsx note is embedded to the README scripts example. Feel free to reopen for any reason.

[tonisives]

I get the same invalid moduleId error for node --loader=ts-node/esm src/simpletest.test.ts

import { test } from "uvu"
import esmock from "esmock"
import { myFun } from "./simpletest.js"


test("test not real", async () => {
  let mock = await esmock("./simpletest.js", {
    myFun: () => console.log("mocked")
  })
  
  myFun()
})


test.run()

export const myFun = () => {
  console.log("still real");
}

➜ node --loader=ts-node/esm src/simpletest.test.ts
(node:5062) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
✘   (0 / 1)

   FAIL  "test not real"
    invalid moduleId: "myFun" (used by file:///Users/tonis/workspace/js/audit-hero/parsers/packages/findings/src/simpletest.test.ts)

uvu runner
"ts-node": "^10.9.1",
node 20.6.1
"uvu": "^0.5.6"