Skip to content

Commit d4fd557

Browse files
committed
fix: restore original test suite and fix ts-node compatibility
- Restored CI to run original test suite (not my place to disable it) - Removed test/package.json that was causing module resolution conflicts - Downgraded ts-node to original version (10.8.1) for compatibility - Added ts-node 'files: true' flag for better Node.js compatibility - Original tests should work on Node.js 16 in CI environment - ESM import fix and comprehensive test suite remain intact The core ESM issue #158 is resolved - this addresses test suite compatibility.
1 parent 542ec22 commit d4fd557

File tree

5 files changed

+7
-11
lines changed

5 files changed

+7
-11
lines changed

.github/workflows/node.js.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,4 @@ jobs:
3030
- run: npm run build --if-present
3131
- run: npm run lint
3232
- run: npm run test-imports
33-
# Original test suite has Node.js 16/22 compatibility issues unrelated to ESM fix
34-
# TODO: Fix original test suite compatibility in a separate PR
35-
# - run: npm test
33+
- run: npm test

package-lock.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
"jsdom": "^20.0.0",
6565
"jsdom-global": "^3.0.2",
6666
"mocha": "^10.0.0",
67-
"ts-node": "^10.9.1",
67+
"ts-node": "^10.8.1",
6868
"typescript": "^5.1.6",
6969
"vue": "^3.2.37"
7070
},

test/package.json

Lines changed: 0 additions & 3 deletions
This file was deleted.

test/tsconfig/tsconfig.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
},
1616
"ts-node": {
1717
"transpileOnly": true,
18+
"files": true,
1819
"compilerOptions": {
1920
"module": "CommonJS"
2021
}

0 commit comments

Comments
 (0)