yujunjun/playwright
1
0
Fork 0
mirror of https://github.com/microsoft/playwright.git synced 2025-06-26 21:40:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(ct): baseURL playwright config (#27689)

Browse Source 
closes: https://github.com/microsoft/playwright/issues/27627
This commit is contained in:
Sander 2023-10-18 22:53:58 +02:00 committed by GitHub
parent fd82b2b3fa
commit 562938b271
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
packages/playwright-ct-core/src/vitePlugin.ts
View File

@ -69,8 +69,7 @@ export function createPlugin(
// - frameworks overrides (frameworkOverrides);
const use = config.projects[0].use as CtConfig;
const port = use.ctPort || 3100;
const host = use.baseURL || 'localhost';
const baseURL = new URL(use.baseURL || 'http://localhost');
const relativeTemplateDir = use.ctTemplateDir || 'playwright';
// FIXME: use build plugin to determine html location to resolve this.
@ -79,7 +78,7 @@ export function createPlugin(
const templateDir = path.join(configDir, relativeTemplateDir);
// Compose base config from the playwright config only.
const baseConfig = {
const baseConfig: InlineConfig = {
root: configDir,
configFile: false,
define: {
@ -92,8 +91,9 @@ export function createPlugin(
outDir: use.ctCacheDir ? path.resolve(configDir, use.ctCacheDir) : path.resolve(templateDir, '.cache')
},
preview: {
port,
host,
https: baseURL.protocol.startsWith('https:'),
host: baseURL.hostname,
port: use.ctPort || Number(baseURL.port) || 3100
},
// Vite preview server will otherwise always return the index.html with 200.
appType: 'custom',

8
tests/playwright-test/playwright.ct-react.spec.ts
View File

@ -377,7 +377,7 @@ test('should handle the baseUrl config', async ({ runInlineTest }) => {
const result = await runInlineTest({
'playwright.config.ts': `
import { defineConfig } from '@playwright/experimental-ct-react';
export default defineConfig({ use: { baseURL: '127.0.0.1' } });
export default defineConfig({ use: { baseURL: 'http://127.0.0.1:8080' } });
`,
'playwright/index.html': `<script type="module" src="./index.js"></script>`,
'playwright/index.js': ``,
@ -392,7 +392,7 @@ test('should handle the baseUrl config', async ({ runInlineTest }) => {
test('pass component', async ({ page, mount }) => {
const component = await mount(<Component />);
await expect(page).toHaveURL('http://127.0.0.1:3100/');
await expect(page).toHaveURL('http://127.0.0.1:8080/');
});
`,
}, { workers: 1 });
@ -435,7 +435,7 @@ test('should prioritize the vite host config over the baseUrl config', async ({
import { defineConfig } from '@playwright/experimental-ct-react';
export default defineConfig({
use: {
baseURL: 'localhost',
baseURL: 'http://localhost:8080',
ctViteConfig: { preview: { host: '127.0.0.1' } }
},
});
@ -453,7 +453,7 @@ test('should prioritize the vite host config over the baseUrl config', async ({
test('pass component', async ({ page, mount }) => {
const component = await mount(<Component />);
await expect(page).toHaveURL('http://127.0.0.1:3100/');
await expect(page).toHaveURL('http://127.0.0.1:8080/');
});
`,
}, { workers: 1 });