Skip to content

Commit 7ef4673

Browse files
committed
fix: import from dist/ instead of src/ in tests with CJS deps
Tests for packages whose source uses require.resolve/__dirname must import from compiled dist/ (CJS) instead of src/ (TS treated as ESM by Node 24). This avoids "require is not defined" errors. Assisted-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 66be3c0 commit 7ef4673

12 files changed

Lines changed: 20 additions & 20 deletions

File tree

packages/dev-server-hmr/test/HmrPlugin.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import { stubMethod, restore as restoreStubs } from 'hanbi';
44
import { createTestServer, fetchText, expectIncludes } from '@web/dev-server-core/test-helpers';
55
import { posix as pathUtil } from 'path';
66

7-
import { hmrPlugin } from '../src/index.ts';
8-
import { NAME_HMR_CLIENT_IMPORT } from '../src/HmrPlugin.ts';
7+
import { hmrPlugin } from '../dist/index.js';
8+
import { NAME_HMR_CLIENT_IMPORT } from '../dist/HmrPlugin.js';
99
import { mockFile, mockFiles } from './utils.ts';
1010

1111
const __dirname = import.meta.dirname;

packages/dev-server-hmr/test/browser.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import type { Browser, HTTPResponse, Page } from 'puppeteer';
66
import { launch as launchPuppeteer } from 'puppeteer';
77
import { posix as pathUtil } from 'path';
88

9-
import { hmrPlugin } from '../src/index.ts';
9+
import { hmrPlugin } from '../dist/index.js';
1010
import { mockFiles } from './utils.ts';
1111

1212
const __dirname = import.meta.dirname;

packages/dev-server-legacy/test/transform-html.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import assert from 'node:assert/strict';
33
import { createTestServer } from '@web/dev-server-core/test-helpers';
44
import { fetchText, expectIncludes } from '@web/dev-server-core/test-helpers';
55

6-
import { legacyPlugin } from '../src/legacyPlugin.ts';
6+
import { legacyPlugin } from '../dist/legacyPlugin.js';
77
import { modernUserAgents, legacyUserAgents } from './userAgents.ts';
88

99
const __dirname = import.meta.dirname;

packages/dev-server-legacy/test/transform-js.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import assert from 'node:assert/strict';
33
import { createTestServer } from '@web/dev-server-core/test-helpers';
44
import { fetchText, expectIncludes, expectNotIncludes } from '@web/dev-server-core/test-helpers';
55

6-
import { legacyPlugin } from '../src/legacyPlugin.ts';
6+
import { legacyPlugin } from '../dist/legacyPlugin.js';
77
import { modernUserAgents, legacyUserAgents } from './userAgents.ts';
88

99
const __dirname = import.meta.dirname;

packages/polyfills-loader/test/createPolyfillsData.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ import assert from 'node:assert/strict';
22
import { describe, it } from 'node:test';
33
import path from 'path';
44

5-
import type { PolyfillsLoaderConfig, PolyfillFile } from '../src/types.ts';
6-
import { createPolyfillsData } from '../src/createPolyfillsData.ts';
7-
import { noModuleSupportTest, fileTypes } from '../src/utils.ts';
5+
import type { PolyfillsLoaderConfig, PolyfillFile } from '../dist/types.js';
6+
import { createPolyfillsData } from '../dist/createPolyfillsData.js';
7+
import { noModuleSupportTest, fileTypes } from '../dist/utils.js';
88

99
const __dirname = import.meta.dirname;
1010

packages/polyfills-loader/test/createPolyfillsLoader.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ import assert from 'node:assert/strict';
22
import { describe, it } from 'node:test';
33
import fs from 'fs';
44
import path from 'path';
5-
import type { PolyfillsLoaderConfig } from '../src/types.ts';
6-
import { createPolyfillsLoader } from '../src/createPolyfillsLoader.ts';
7-
import { noModuleSupportTest, fileTypes } from '../src/utils.ts';
5+
import type { PolyfillsLoaderConfig } from '../dist/types.js';
6+
import { createPolyfillsLoader } from '../dist/createPolyfillsLoader.js';
7+
import { noModuleSupportTest, fileTypes } from '../dist/utils.js';
88

99
const __dirname = import.meta.dirname;
1010
const updateSnapshots = process.argv.includes('--update-snapshots');

packages/polyfills-loader/test/injectPolyfillsLoader.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ import assert from 'node:assert/strict';
22
import { describe, it } from 'node:test';
33
import path from 'path';
44
import fs from 'fs';
5-
import { injectPolyfillsLoader } from '../src/injectPolyfillsLoader.ts';
6-
import { noModuleSupportTest, fileTypes } from '../src/utils.ts';
7-
import type { PolyfillsLoaderConfig } from '../src/types.ts';
5+
import { injectPolyfillsLoader } from '../dist/injectPolyfillsLoader.js';
6+
import { noModuleSupportTest, fileTypes } from '../dist/utils.js';
7+
import type { PolyfillsLoaderConfig } from '../dist/types.js';
88

99
const __dirname = import.meta.dirname;
1010
const updateSnapshots = process.argv.includes('--update-snapshots');

packages/rollup-plugin-html/test/rollup-plugin-html.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {
88
import assert from 'node:assert/strict';
99
import { describe, it } from 'node:test';
1010
import path from 'path';
11-
import { rollupPluginHTML } from '../src/index.ts';
11+
import { rollupPluginHTML } from '../dist/index.js';
1212

1313
const __dirname = import.meta.dirname;
1414

packages/test-runner-chrome/test/chromeLauncher.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { runIntegrationTests } from '../../../integration/test-runner/index.ts';
2-
import { chromeLauncher } from '../src/index.ts';
2+
import { chromeLauncher } from '../dist/index.js';
33

44
describe('test-runner-chrome', function testRunnerChrome() {
55
this.timeout(20000);

packages/test-runner-playwright/test/playwrightLauncher.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import os from 'os';
22
import { runIntegrationTests } from '../../../integration/test-runner/index.ts';
3-
import { playwrightLauncher } from '../src/index.ts';
3+
import { playwrightLauncher } from '../dist/index.js';
44

55
describe('test-runner-playwright chromium', function testRunnerPlaywright() {
66
this.timeout(100000);

0 commit comments

Comments
 (0)