chore: make @playwright/test depend on playwright (#26946)

.github/workflows/pr_check_client_side_changes.yml

@@ -5,7 +5,7 @@ on:
       - main
     paths:
       - 'packages/playwright-core/src/client/**/*'
-      - 'packages/playwright-test/src/matchers/matchers.ts'
+      - 'packages/playwright/src/matchers/matchers.ts'
 jobs:
   check:
     name: Check

.gitignore

@@ -25,6 +25,7 @@ test-results
 /packages/*/LICENSE
 /packages/*/NOTICE
 /packages/playwright/README.md
+/packages/playwright-test/README.md
 /packages/playwright-core/api.json
 .env
 /tests/installation/output/

docs/src/release-notes-js.md

@@ -15,7 +15,7 @@ import LiteYouTube from '@site/src/components/LiteYouTube';
 
 ### New `npx playwright merge-reports` tool
 
-If you run tests on multiple shards, you can now merge all reports in a single HTML report (or any other report) 
+If you run tests on multiple shards, you can now merge all reports in a single HTML report (or any other report)
 using the new `merge-reports` CLI tool.
 
 Using `merge-reports` tool requires the following steps:
@@ -48,7 +48,7 @@ Let us know if you encounter any issues!
 Linux support looks like this:
 
 |          | Ubuntu 20.04 | Ubuntu 22.04 | Debian 11 | Debian 12 |
-| :--- | :---: | :---: | :---: | :---: | 
+| :--- | :---: | :---: | :---: | :---: |
 | Chromium | ✅ | ✅ | ✅ | ✅ |
 | WebKit | ✅ | ✅ | ✅ | ✅ |
 | Firefox | ✅ | ✅ | ✅ | ✅ |

packages/html-reporter/tsconfig.json

@@ -19,9 +19,9 @@
     "paths": {
       "@protocol/*": ["../protocol/src/*"],
       "@web/*": ["../web/src/*"],
-      "@playwright-test/*": ["../playwright-test/src/*"],
+      "@playwright/*": ["../playwright/src/*"],
       "playwright-core/lib/*": ["../playwright-core/src/*"],
-      "playwright-test/lib/*": ["../playwright-test/src/*"],
+      "playwright/lib/*": ["../playwright/src/*"],
     }
   },
   "include": ["src"],

packages/playwright-ct-core/cli.js

@@ -14,4 +14,4 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-module.exports = require('@playwright/test/lib/cli');
+module.exports = require('playwright/lib/cli');

packages/playwright-ct-core/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from './types/component';
 import type { InlineConfig } from 'vite';
 
@@ -63,4 +63,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-core/index.js

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-const { test: baseTest, expect, devices, defineConfig: originalDefineConfig } = require('@playwright/test');
+const { test: baseTest, expect, devices, defineConfig: originalDefineConfig } = require('playwright/test');
 const { fixtures } = require('./lib/mount');
 
 const defineConfig = config => originalDefineConfig({

packages/playwright-ct-core/package.json

@@ -26,7 +26,7 @@
   "dependencies": {
     "playwright-core": "1.38.0-next",
     "vite": "^4.3.9",
-    "@playwright/test": "1.38.0-next"
+    "playwright": "1.38.0-next"
   },
   "bin": {
     "playwright": "./cli.js"

packages/playwright-ct-core/src/mount.ts

@@ -14,9 +14,9 @@
  * limitations under the License.
  */
 
-import type { Fixtures, Locator, Page, BrowserContextOptions, PlaywrightTestArgs, PlaywrightTestOptions, PlaywrightWorkerArgs, PlaywrightWorkerOptions, BrowserContext } from '@playwright/test';
+import type { Fixtures, Locator, Page, BrowserContextOptions, PlaywrightTestArgs, PlaywrightTestOptions, PlaywrightWorkerArgs, PlaywrightWorkerOptions, BrowserContext } from 'playwright/test';
 import type { Component, JsxComponent, MountOptions } from '../types/component';
-import type { ContextReuseMode, FullConfigInternal } from '../../playwright-test/src/common/config';
+import type { ContextReuseMode, FullConfigInternal } from '../../playwright/src/common/config';
 
 let boundCallbacksForMount: Function[] = [];
 

packages/playwright-ct-core/src/tsxTransform.ts

@@ -15,9 +15,9 @@
  */
 
 import path from 'path';
-import type { T, BabelAPI } from '@playwright/test/src/transform/babelBundle';
-import { types, declare, traverse } from '@playwright/test/lib/transform/babelBundle';
-import { resolveImportSpecifierExtension } from '@playwright/test/lib/util';
+import type { T, BabelAPI } from 'playwright/src/transform/babelBundle';
+import { types, declare, traverse } from 'playwright/lib/transform/babelBundle';
+import { resolveImportSpecifierExtension } from 'playwright/lib/util';
 const t: typeof T = types;
 
 const fullNames = new Map<string, string | undefined>();

packages/playwright-ct-core/src/vitePlugin.ts

@@ -14,23 +14,23 @@
  * limitations under the License.
  */
 
-import type { Suite } from '@playwright/test/reporter';
-import type { PlaywrightTestConfig as BasePlaywrightTestConfig, FullConfig } from '@playwright/test';
+import type { Suite } from 'playwright/types/testReporter';
+import type { PlaywrightTestConfig as BasePlaywrightTestConfig, FullConfig } from 'playwright/test';
 
 import type { InlineConfig, Plugin, ResolveFn, ResolvedConfig, UserConfig } from 'vite';
-import type { TestRunnerPlugin } from '../../playwright-test/src/plugins';
+import type { TestRunnerPlugin } from '../../playwright/src/plugins';
 import type { ComponentInfo } from './tsxTransform';
 import type { AddressInfo } from 'net';
 import type { PluginContext } from 'rollup';
 import { debug } from 'playwright-core/lib/utilsBundle';
 
 import fs from 'fs';
 import path from 'path';
-import { parse, traverse, types as t } from '@playwright/test/lib/transform/babelBundle';
-import { stoppable } from '@playwright/test/lib/utilsBundle';
+import { parse, traverse, types as t } from 'playwright/lib/transform/babelBundle';
+import { stoppable } from 'playwright/lib/utilsBundle';
 import { assert, calculateSha1 } from 'playwright-core/lib/utils';
 import { getPlaywrightVersion } from 'playwright-core/lib/utils';
-import { setExternalDependencies } from '@playwright/test/lib/transform/compilationCache';
+import { setExternalDependencies } from 'playwright/lib/transform/compilationCache';
 import { collectComponentUsages, componentInfo } from './tsxTransform';
 import { version as viteVersion, build, preview, mergeConfig } from 'vite';
 

packages/playwright-ct-react/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 
@@ -63,4 +63,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-react17/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 
@@ -63,4 +63,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-solid/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 
@@ -63,4 +63,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-svelte/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 import type { SvelteComponent, ComponentProps } from 'svelte/types/runtime';
@@ -71,4 +71,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-vue/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 
@@ -91,4 +91,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-ct-vue2/index.d.ts

@@ -22,7 +22,7 @@ import type {
   PlaywrightWorkerArgs,
   PlaywrightWorkerOptions,
   Locator,
-} from '@playwright/test';
+} from 'playwright/test';
 import type { JsonObject } from '@playwright/experimental-ct-core/types/component';
 import type { InlineConfig } from 'vite';
 
@@ -91,4 +91,4 @@ export function defineConfig(config: PlaywrightTestConfig): PlaywrightTestConfig
 export function defineConfig<T>(config: PlaywrightTestConfig<T>): PlaywrightTestConfig<T>;
 export function defineConfig<T, W>(config: PlaywrightTestConfig<T, W>): PlaywrightTestConfig<T, W>;
 
-export { expect, devices } from '@playwright/test';
+export { expect, devices } from 'playwright/test';

packages/playwright-test/cli.js

@@ -14,4 +14,4 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-module.exports = require('@playwright/test/lib/cli');
+module.exports = require('playwright/cli');

packages/playwright-test/index.d.ts

@@ -14,5 +14,5 @@
  * limitations under the License.
  */
 
-export * from './types/test';
-export { default } from './types/test';
+export * from 'playwright/test';
+export { default } from 'playwright/test';

packages/playwright-test/index.js

@@ -14,15 +14,4 @@
  * limitations under the License.
  */
 
-const pwt = require('./lib/index');
-const { defineConfig } = require('./lib/common/configLoader');
-const playwright = require('playwright-core');
-const combinedExports = {
-  ...playwright,
-  ...pwt,
-  defineConfig,
-};
-
-Object.defineProperty(combinedExports, '__esModule', { value: true });
-
-module.exports = combinedExports;
+module.exports = require('playwright/test');

packages/playwright-test/index.mjs

@@ -13,18 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import playwright from './index.js';
 
-export const chromium = playwright.chromium;
-export const firefox = playwright.firefox;
-export const webkit = playwright.webkit;
-export const selectors = playwright.selectors;
-export const devices = playwright.devices;
-export const errors = playwright.errors;
-export const request = playwright.request;
-export const _electron = playwright._electron;
-export const _android = playwright._android;
-export const test = playwright.test;
-export const expect = playwright.expect;
-export const defineConfig = playwright.defineConfig;
-export default playwright.test;
+export * from 'playwright/test';
+export { default } from 'playwright/test';

packages/playwright-test/package.json

@@ -7,7 +7,10 @@
   "engines": {
     "node": ">=16"
   },
-  "main": "index.js",
+  "author": {
+    "name": "Microsoft Corporation"
+  },
+  "license": "Apache-2.0",
   "exports": {
     ".": {
       "types": "./index.d.ts",
@@ -17,32 +20,14 @@
     },
     "./cli": "./cli.js",
     "./package.json": "./package.json",
-    "./lib/cli": "./lib/cli.js",
-    "./lib/transform/babelBundle": "./lib/transform/babelBundle.js",
-    "./lib/transform/compilationCache": "./lib/transform/compilationCache.js",
-    "./lib/transform/esmLoader": "./lib/transform/esmLoader.js",
-    "./lib/internalsForTest": "./lib/internalsForTest.js",
-    "./lib/plugins": "./lib/plugins/index.js",
-    "./jsx-runtime": {
-      "import": "./jsx-runtime.mjs",
-      "require": "./jsx-runtime.js",
-      "default": "./jsx-runtime.js"
-    },
-    "./lib/util": "./lib/util.js",
-    "./lib/utilsBundle": "./lib/utilsBundle.js",
     "./reporter": "./reporter.js"
   },
   "bin": {
     "playwright": "./cli.js"
   },
-  "author": {
-    "name": "Microsoft Corporation"
+  "scripts": {
   },
-  "license": "Apache-2.0",
   "dependencies": {
-    "playwright-core": "1.38.0-next"
-  },
-  "optionalDependencies": {
-    "fsevents": "2.3.2"
+    "playwright": "1.38.0-next"
   }
 }