Merge pull request #770 from wheresrhys/rhys/toolchain-import-tests

Rhys/toolchain import tests

.circleci/config.yml

@@ -91,6 +91,14 @@ jobs:
       - *workspace
       - run: npm run build
       - run: npm run test:jest
+  import-compat:
+    <<: *nodelts
+    steps:
+      - *workspace
+      - run: npm run compat:ts:cjs -w import-compat
+      - run: npm run compat:js:cjs -w import-compat
+      - run: npm run compat:ts:esm -w import-compat
+      - run: npm run compat:js:esm -w import-compat
   # chrome:
   #   <<: *browsers
   #   steps:
@@ -133,12 +141,10 @@ workflows:
           <<: *triggerable-by-tag
           requires:
             - checkout_code
-      # could be parallel with build, lint, and unit but it's a slow job
-      # And circlecifree tier only has 3 concurrent jobs, so overall faster
-      # to defer
       - typelint:
           <<: *triggerable-by-tag
-          <<: *run-after-first-jobs
+          requires:
+            - checkout_code
       - nodefetch3:
           <<: *triggerable-by-tag
           <<: *run-after-first-jobs
@@ -148,6 +154,10 @@ workflows:
       - jest:
           <<: *triggerable-by-tag
           <<: *run-after-first-jobs
+      - import-compat:
+          <<: *triggerable-by-tag
+          requires:
+            - build
       # - chrome:
       #     <<: *triggerable-by-tag
       #     <<: *run-after-first-jobs
@@ -160,6 +170,7 @@ workflows:
             # - chrome
             # - firefox
             - build
+            - import-compat
             - typelint
             - commonjs
             - jest

.gitignore

@@ -13,3 +13,5 @@ coverage/
 # built files
 /docs/fetch-mock/dist/
 /packages/**/dist
+
+import-compat/ts-*.js

import-compat/js-cjs.js

@@ -0,0 +1,5 @@
+const fetchMockCore = require('@fetch-mock/core').default;
+fetchMockCore.route('http://example.com', 200);
+
+const fetchMock = require('fetch-mock').default;
+fetchMock.mock('http://example.com', 200);

import-compat/js-esm.mjs

@@ -0,0 +1,5 @@
+import fetchMockCore from "@fetch-mock/core";
+fetchMockCore.route("http://example.com", 200);
+
+import fetchMock from "fetch-mock";
+fetchMock.mock("http://example.com", 200);

import-compat/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "fetch-mock-compat-tests",
+  "version": "1.0.0",
+  "scripts": {
+    "compat:ts:esm": "tsc ts-esm.ts --target esnext --moduleResolution nodenext --module nodenext && node ts-esm.js",
+    "compat:ts:cjs": "tsc ts-cjs.ts  --target esnext --moduleResolution nodenext --module nodenext && node ts-cjs.js",
+    "compat:js:esm": "node js-esm.mjs",
+    "compat:js:cjs": "node js-cjs.js"
+  }
+}

import-compat/ts-cjs.ts

@@ -0,0 +1,5 @@
+const fetchMockCore = require("@fetch-mock/core").default;
+fetchMockCore.route("http://example.com", 200);
+
+const fetchMock = require("fetch-mock").default;
+fetchMock.mock("http://example.com", 200);

import-compat/ts-esm.ts

@@ -0,0 +1,5 @@
+import fetchMockCore from "@fetch-mock/core";
+fetchMockCore.route("http://example.com", 200);
+
+import fetchMock from "fetch-mock";
+fetchMock.mock("http://example.com", 200);

package.json

@@ -18,7 +18,8 @@
   },
   "workspaces": [
     "packages/*",
-    "docs"
+    "docs",
+    "import-compat"
   ],
   "scripts": {
     "lint:staged": "eslint --cache --fix --ext .js,.cjs",
@@ -43,6 +44,8 @@
     "@commitlint/config-conventional": "^19.2.2",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-node-resolve": "^15.2.3",
+    "@types/events": "^3.0.3",
+    "@types/globrex": "^0.1.4",
     "@types/node": "^20.14.10",
     "@vitest/browser": "^1.1.0",
     "@vitest/coverage-istanbul": "^1.1.0",
@@ -65,8 +68,6 @@
     "typescript": "^5.5.3",
     "v8": "^0.1.0",
     "vitest": "^1.1.0",
-    "@types/events": "^3.0.3",
-    "@types/globrex": "^0.1.4",
     "webdriverio": "^8.27.0"
   },
   "volta": {
@@ -79,7 +80,7 @@
     "packages/**/*.js": [
       "npm run types:check"
     ],
-    "**/*.md":[
+    "**/*.md": [
       "npm run prettier"
     ]
   }

shared-rollup.config.js

@@ -6,6 +6,7 @@ export default {
 		dir: './dist',
 		entryFileNames: 'commonjs.js',
 		format: 'commonjs',
+		// exports: 'named',
 	},
 	plugins: [
 		nodeResolve({ preferBuiltins: false }),