@playwright/test 1.35.0-alpha-may-30-2023 → 1.35.0-alpha-jun-1-2023

package/lib/runner/runner.js

@@ -44,7 +44,6 @@ class Runner {
       report.projects.push({
         name: project.project.name,
         testDir: project.project.testDir,
-        outputDir: project.project.outputDir,
         use: {
           testIdAttribute: project.project.use.testIdAttribute
         },

package/lib/runner/tasks.js

@@ -154,6 +154,7 @@ function createRemoveOutputDirsTask() {
   return async ({
     config
   }) => {
+    if (process.env.PW_TEST_NO_REMOVE_OUTPUT_DIRS) return;
     const outputDirs = new Set();
     for (const p of config.projects) {
       if (!config.cliProjectFilter || config.cliProjectFilter.includes(p.project.name)) outputDirs.add(p.project.outputDir);

package/lib/runner/uiMode.js

@@ -159,7 +159,9 @@ class UIMode {
   }
   _dispatchEvent(message) {
     // eslint-disable-next-line no-console
-    this._page.mainFrame().evaluateExpression(dispatchFuncSource, true, message).catch(e => this._originalStderrWrite.call(process.stderr, String(e)));
+    this._page.mainFrame().evaluateExpression(dispatchFuncSource, {
+      isFunction: true
+    }, message).catch(e => this._originalStderrWrite.call(process.stderr, String(e)));
   }
   async _listTests() {
     const listReporter = new _teleEmitter.TeleReporterEmitter(e => this._dispatchEvent(e));

package/lib/worker/workerMain.js

@@ -284,6 +284,7 @@ class WorkerMain extends _process.ProcessRunner {
         didFailBeforeAllForSuite = undefined;
         return;
       }
+      await removeFolderAsync(testInfo.outputDir).catch(() => {});
       let testFunctionParams = null;
       await testInfo._runAsStep({
         category: 'hook',
@@ -455,7 +456,7 @@ class WorkerMain extends _process.ProcessRunner {
     (0, _globals.setCurrentTestInfo)(null);
     this.dispatchEvent('testEnd', buildTestEndPayload(testInfo));
     const preserveOutput = this._config.config.preserveOutput === 'always' || this._config.config.preserveOutput === 'failures-only' && testInfo._isFailure();
-    if (!preserveOutput) await removeFolderAsync(testInfo.outputDir).catch(e => {});
+    if (!preserveOutput) await removeFolderAsync(testInfo.outputDir).catch(() => {});
   }
   async _runModifiersForSuite(suite, testInfo, scope, timeSlot, extraAnnotations) {
     for (const modifier of suite._modifiers) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@playwright/test",
-  "version": "1.35.0-alpha-may-30-2023",
+  "version": "1.35.0-alpha-jun-1-2023",
   "description": "A high-level API to automate web browsers",
   "repository": "github:Microsoft/playwright",
   "homepage": "https://playwright.dev",
@@ -41,7 +41,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@types/node": "*",
-    "playwright-core": "1.35.0-alpha-may-30-2023"
+    "playwright-core": "1.35.0-alpha-jun-1-2023"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"

package/types/test.d.ts

@@ -3675,7 +3675,7 @@ export interface PlaywrightTestOptions {
    * });
    * ```
    *
-   * Toggles bypassing page's Content-Security-Policy.
+   * Toggles bypassing page's Content-Security-Policy. Defaults to `false`.
    */
   bypassCSP: boolean;
   /**
@@ -3732,7 +3732,7 @@ export interface PlaywrightTestOptions {
    * });
    * ```
    *
-   * An object containing additional HTTP headers to be sent with every request.
+   * An object containing additional HTTP headers to be sent with every request. Defaults to none.
    */
   extraHTTPHeaders: ExtraHTTPHeaders | undefined;
   /**
@@ -3860,8 +3860,8 @@ export interface PlaywrightTestOptions {
    * ```
    *
    * Specify user locale, for example `en-GB`, `de-DE`, etc. Locale will affect `navigator.language` value,
-   * `Accept-Language` request header value as well as number and date formatting rules. Learn more about emulation in
-   * our [emulation guide](https://playwright.dev/docs/emulation#locale--timezone).
+   * `Accept-Language` request header value as well as number and date formatting rules. Defaults to the system default
+   * locale. Learn more about emulation in our [emulation guide](https://playwright.dev/docs/emulation#locale--timezone).
    */
   locale: string | undefined;
   /**
@@ -3898,7 +3898,7 @@ export interface PlaywrightTestOptions {
    *
    * A list of permissions to grant to all pages in this context. See
    * [browserContext.grantPermissions(permissions[, options])](https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions)
-   * for more details.
+   * for more details. Defaults to none.
    */
   permissions: string[] | undefined;
   /**
@@ -3958,7 +3958,7 @@ export interface PlaywrightTestOptions {
    *
    * Changes the timezone of the context. See
    * [ICU's metaZones.txt](https://cs.chromium.org/chromium/src/third_party/icu/source/data/misc/metaZones.txt?rcl=faee8bc70570192d82d2978a71e2a615788597d1)
-   * for a list of supported timezone IDs.
+   * for a list of supported timezone IDs. Defaults to the system timezone.
    */
   timezoneId: string | undefined;
   /**
@@ -4021,7 +4021,7 @@ export interface PlaywrightTestOptions {
    * [page.waitForResponse(urlOrPredicate[, options])](https://playwright.dev/docs/api/class-page#page-wait-for-response)
    * it takes the base URL in consideration by using the
    * [`URL()`](https://developer.mozilla.org/en-US/docs/Web/API/URL/URL) constructor for building the corresponding URL.
-   * Examples:
+   * Unset by default. Examples:
    * - baseURL: `http://localhost:3000` and navigating to `/bar.html` results in `http://localhost:3000/bar.html`
    * - baseURL: `http://localhost:3000/foo/` and navigating to `./bar.html` results in
    *   `http://localhost:3000/foo/bar.html`
@@ -4869,7 +4869,7 @@ interface LocatorAssertions {
     attached?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4890,7 +4890,7 @@ interface LocatorAssertions {
     checked?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4913,7 +4913,7 @@ interface LocatorAssertions {
    */
   toBeDisabled(options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4934,7 +4934,7 @@ interface LocatorAssertions {
     editable?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4953,7 +4953,7 @@ interface LocatorAssertions {
    */
   toBeEmpty(options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4974,7 +4974,7 @@ interface LocatorAssertions {
     enabled?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -4993,7 +4993,7 @@ interface LocatorAssertions {
    */
   toBeFocused(options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5013,7 +5013,7 @@ interface LocatorAssertions {
    */
   toBeHidden(options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5044,7 +5044,7 @@ interface LocatorAssertions {
     ratio?: number;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5063,7 +5063,7 @@ interface LocatorAssertions {
    */
   toBeVisible(options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
 
@@ -5125,7 +5125,7 @@ interface LocatorAssertions {
     ignoreCase?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
 
@@ -5151,7 +5151,7 @@ interface LocatorAssertions {
    */
   toHaveAttribute(name: string, value: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5184,7 +5184,7 @@ interface LocatorAssertions {
    */
   toHaveClass(expected: string|RegExp|Array<string|RegExp>, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5204,7 +5204,7 @@ interface LocatorAssertions {
    */
   toHaveCount(count: number, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5225,7 +5225,7 @@ interface LocatorAssertions {
    */
   toHaveCSS(name: string, value: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5245,7 +5245,7 @@ interface LocatorAssertions {
    */
   toHaveId(id: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5267,7 +5267,7 @@ interface LocatorAssertions {
    */
   toHaveJSProperty(name: string, value: any, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5345,7 +5345,7 @@ interface LocatorAssertions {
     threshold?: number;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5422,7 +5422,7 @@ interface LocatorAssertions {
     threshold?: number;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5481,7 +5481,7 @@ interface LocatorAssertions {
     ignoreCase?: boolean;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
 
@@ -5507,7 +5507,7 @@ interface LocatorAssertions {
    */
   toHaveValue(value: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5539,7 +5539,7 @@ interface LocatorAssertions {
    */
   toHaveValues(values: Array<string|RegExp>, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5675,7 +5675,7 @@ interface PageAssertions {
     threshold?: number;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5782,7 +5782,7 @@ interface PageAssertions {
     threshold?: number;
 
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5801,7 +5801,7 @@ interface PageAssertions {
    */
   toHaveTitle(titleOrRegExp: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;
@@ -5820,7 +5820,7 @@ interface PageAssertions {
    */
   toHaveURL(urlOrRegExp: string|RegExp, options?: {
     /**
-     * Time to retry the assertion for. Defaults to `timeout` in `TestConfig.expect`.
+     * Time to retry the assertion for in milliseconds. Defaults to `timeout` in `TestConfig.expect`.
      */
     timeout?: number;
   }): Promise<void>;