@playwright/test 1.32.2 → 1.32.3

package/lib/matchers/matchers.js

@@ -28,7 +28,6 @@ exports.toHaveValues = toHaveValues;
 exports.toPass = toPass;
 var _utilsBundle = require("playwright-core/lib/utilsBundle");
 var _util = require("../util");
-var _globals = require("../common/globals");
 var _toBeTruthy = require("./toBeTruthy");
 var _toEqual = require("./toEqual");
 var _toMatchText = require("./toMatchText");
@@ -323,22 +322,10 @@ async function toBeOK(response) {
   };
 }
 async function toPass(callback, options = {}) {
-  const testInfo = (0, _globals.currentTestInfo)();
   const timeout = options.timeout !== undefined ? options.timeout : 0;
-
-  // Soft expects might mark test as failing.
-  // We want to revert this later if the matcher is actually passing.
-  // See https://github.com/microsoft/playwright/issues/20437
-  let testStateBeforeToPassMatcher;
   const result = await (0, _utils.pollAgainstTimeout)(async () => {
     try {
-      if (testStateBeforeToPassMatcher && testInfo) testInfo._restoreErrorState(testStateBeforeToPassMatcher);
-      testStateBeforeToPassMatcher = testInfo === null || testInfo === void 0 ? void 0 : testInfo._saveErrorState();
       await callback();
-      if (testInfo && testStateBeforeToPassMatcher && testInfo.errors.length > testStateBeforeToPassMatcher.errors.length) return {
-        continuePolling: !this.isNot,
-        result: testInfo.errors[testInfo.errors.length - 1]
-      };
       return {
         continuePolling: this.isNot,
         result: undefined
@@ -358,7 +345,6 @@ async function toPass(callback, options = {}) {
       pass: this.isNot
     };
   }
-  if (testStateBeforeToPassMatcher && testInfo) testInfo._restoreErrorState(testStateBeforeToPassMatcher);
   return {
     pass: !this.isNot,
     message: () => ''

package/lib/worker/testInfo.js

@@ -217,18 +217,6 @@ class TestInfoImpl {
     if (this.status === 'passed' || this.status === 'skipped') this.status = 'failed';
     this.errors.push(error);
   }
-  _saveErrorState() {
-    return {
-      hasHardError: this._hasHardError,
-      status: this.status,
-      errors: this.errors.slice()
-    };
-  }
-  _restoreErrorState(state) {
-    this.status = state.status;
-    this.errors = state.errors.slice();
-    this._hasHardError = state.hasHardError;
-  }
   async _runAsStep(cb, stepInfo) {
     const step = this._addStep({
       ...stepInfo,

package/package.json

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

package/types/test.d.ts

@@ -5887,7 +5887,8 @@ interface TestConfigWebServer {
 
   /**
    * The url on your http server that is expected to return a 2xx, 3xx, 400, 401, 402, or 403 status code when the
-   * server is ready to accept connections. Exactly one of `port` or `url` is required.
+   * server is ready to accept connections. Redirects (3xx status codes) are being followed and the new location is
+   * checked. Exactly one of `port` or `url` is required.
    */
   url?: string;