@playwright/test 1.36.0-alpha-jun-8-2023 → 1.36.0-alpha-jun-10-2023

package/lib/isomorphic/teleReceiver.js

@@ -23,17 +23,19 @@ exports.serializeRegexPatterns = serializeRegexPatterns;
  */
 
 class TeleReporterReceiver {
-  constructor(pathSeparator, reporter, reportConfig) {
+  constructor(pathSeparator, reporter, reuseTestCases, reportConfig) {
     this._rootSuite = void 0;
     this._pathSeparator = void 0;
     this._reporter = void 0;
     this._tests = new Map();
     this._rootDir = void 0;
     this._clearPreviousResultsWhenTestBegins = false;
+    this._reuseTestCases = void 0;
     this._reportConfig = void 0;
     this._rootSuite = new TeleSuite('', 'root');
     this._pathSeparator = pathSeparator;
     this._reporter = reporter;
+    this._reuseTestCases = reuseTestCases;
     this._reportConfig = reportConfig;
   }
   dispatch(message) {
@@ -130,7 +132,7 @@ class TeleReporterReceiver {
     result.status = payload.status;
     result.statusEx = payload.status;
     result.errors = payload.errors;
-    result.attachments = payload.attachments;
+    result.attachments = this._parseAttachments(payload.attachments);
     (_this$_reporter$onTes2 = (_this$_reporter3 = this._reporter).onTestEnd) === null || _this$_reporter$onTes2 === void 0 ? void 0 : _this$_reporter$onTes2.call(_this$_reporter3, test, result);
   }
   _onStepBegin(testId, resultId, payload) {
@@ -145,7 +147,7 @@ class TeleReporterReceiver {
       location: this._absoluteLocation(payload.location),
       parent: parentStep,
       startTime: new Date(payload.startTime),
-      duration: 0,
+      duration: -1,
       steps: []
     };
     if (parentStep) parentStep.steps.push(step);else result.steps.push(step);
@@ -166,11 +168,18 @@ class TeleReporterReceiver {
     (_this$_reporter$onErr = (_this$_reporter6 = this._reporter).onError) === null || _this$_reporter$onErr === void 0 ? void 0 : _this$_reporter$onErr.call(_this$_reporter6, error);
   }
   _onStdIO(type, testId, resultId, data, isBase64) {
-    var _this$_reporter$onStd, _this$_reporter7, _this$_reporter$onStd2, _this$_reporter8;
     const chunk = isBase64 ? globalThis.Buffer ? Buffer.from(data, 'base64') : atob(data) : data;
     const test = testId ? this._tests.get(testId) : undefined;
     const result = test && resultId ? test.resultsMap.get(resultId) : undefined;
-    if (type === 'stdout') (_this$_reporter$onStd = (_this$_reporter7 = this._reporter).onStdOut) === null || _this$_reporter$onStd === void 0 ? void 0 : _this$_reporter$onStd.call(_this$_reporter7, chunk, test, result);else (_this$_reporter$onStd2 = (_this$_reporter8 = this._reporter).onStdErr) === null || _this$_reporter$onStd2 === void 0 ? void 0 : _this$_reporter$onStd2.call(_this$_reporter8, chunk, test, result);
+    if (type === 'stdout') {
+      var _this$_reporter$onStd, _this$_reporter7;
+      result === null || result === void 0 ? void 0 : result.stdout.push(chunk);
+      (_this$_reporter$onStd = (_this$_reporter7 = this._reporter).onStdOut) === null || _this$_reporter$onStd === void 0 ? void 0 : _this$_reporter$onStd.call(_this$_reporter7, chunk, test, result);
+    } else {
+      var _this$_reporter$onStd2, _this$_reporter8;
+      result === null || result === void 0 ? void 0 : result.stderr.push(chunk);
+      (_this$_reporter$onStd2 = (_this$_reporter8 = this._reporter).onStdErr) === null || _this$_reporter$onStd2 === void 0 ? void 0 : _this$_reporter$onStd2.call(_this$_reporter8, chunk, test, result);
+    }
   }
   _onEnd(result) {
     var _this$_reporter$onEnd, _this$_reporter9;
@@ -213,6 +222,14 @@ class TeleReporterReceiver {
       use: {}
     };
   }
+  _parseAttachments(attachments) {
+    return attachments.map(a => {
+      return {
+        ...a,
+        body: a.base64 && globalThis.Buffer ? Buffer.from(a.base64, 'base64') : undefined
+      };
+    });
+  }
   _mergeSuitesInto(jsonSuites, parent) {
     for (const jsonSuite of jsonSuites) {
       let targetSuite = parent.suites.find(s => s.title === jsonSuite.title);
@@ -230,7 +247,7 @@ class TeleReporterReceiver {
   }
   _mergeTestsInto(jsonTests, parent) {
     for (const jsonTest of jsonTests) {
-      let targetTest = parent.tests.find(s => s.title === jsonTest.title);
+      let targetTest = this._reuseTestCases ? parent.tests.find(s => s.title === jsonTest.title) : undefined;
       if (!targetTest) {
         targetTest = new TeleTestCase(jsonTest.testId, jsonTest.title, this._absoluteLocation(jsonTest.location));
         targetTest.parent = parent;

package/lib/reporters/blob.js

@@ -29,7 +29,7 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 
 class BlobReporter extends _teleEmitter.TeleReporterEmitter {
   constructor(options) {
-    super(message => this._messages.push(message));
+    super(message => this._messages.push(message), false);
     this._messages = [];
     this._options = void 0;
     this._salt = void 0;
@@ -65,7 +65,7 @@ class BlobReporter extends _teleEmitter.TeleReporterEmitter {
     _fs.default.promises.writeFile(this._reportFile, content).catch(e => console.error(`Failed to write report ${this._reportFile}: ${e}`))]);
   }
   _serializeAttachments(attachments) {
-    return attachments.map(attachment => {
+    return super._serializeAttachments(attachments).map(attachment => {
       if (!attachment.path || !_fs.default.statSync(attachment.path).isFile()) return attachment;
       // Add run guid to avoid clashes between shards.
       const sha1 = (0, _utils.calculateSha1)(attachment.path + this._salt);

package/lib/reporters/merge.js

@@ -32,7 +32,7 @@ async function createMergedReport(config, dir, reporterDescriptions, resolvePath
   const events = await mergeEvents(dir, shardFiles);
   if (resolvePaths) patchAttachmentPaths(events, dir);
   const reporters = await (0, _reporters.createReporters)(config, 'merge', reporterDescriptions);
-  const receiver = new _teleReceiver.TeleReporterReceiver(_path.default.sep, new _multiplexer.Multiplexer(reporters), config.config);
+  const receiver = new _teleReceiver.TeleReporterReceiver(_path.default.sep, new _multiplexer.Multiplexer(reporters), false, config.config);
   for (const event of events) await receiver.dispatch(event);
 }
 function patchAttachmentPaths(events, resourceDir) {

package/lib/reporters/teleEmitter.js

@@ -26,10 +26,12 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
  */
 
 class TeleReporterEmitter {
-  constructor(messageSink) {
+  constructor(messageSink, skipBuffers) {
     this._messageSink = void 0;
     this._rootDir = void 0;
+    this._skipBuffers = void 0;
     this._messageSink = messageSink;
+    this._skipBuffers = skipBuffers;
   }
   onBegin(config, suite) {
     this._rootDir = config.rootDir;
@@ -97,7 +99,7 @@ class TeleReporterEmitter {
     });
   }
   onStdOut(chunk, test, result) {
-    this._onStdIO('stdio', chunk, test, result);
+    this._onStdIO('stdout', chunk, test, result);
   }
   onStdErr(chunk, test, result) {
     this._onStdIO('stderr', chunk, test, result);
@@ -201,7 +203,13 @@ class TeleReporterEmitter {
     };
   }
   _serializeAttachments(attachments) {
-    return attachments;
+    return attachments.map(a => {
+      return {
+        ...a,
+        // There is no Buffer in the browser, so there is no point in sending the data there.
+        base64: a.body && !this._skipBuffers ? a.body.toString('base64') : undefined
+      };
+    });
   }
   _serializeStepStart(step) {
     var _step$parent;

package/lib/runner/reporters.js

@@ -70,7 +70,7 @@ async function createReporters(config, mode, descriptions) {
     const prints = r.printsToStdio ? r.printsToStdio() : true;
     return prints;
   });
-  if (reporters.length && !someReporterPrintsToStdio && mode !== 'merge') {
+  if (reporters.length && !someReporterPrintsToStdio) {
     // Add a line/dot/list-mode reporter for convenience.
     // Important to put it first, jsut in case some other reporter stalls onEnd.
     if (mode === 'list') reporters.unshift(new ListModeReporter());else reporters.unshift(!process.env.CI ? new _line.default({

package/lib/runner/uiMode.js

@@ -155,7 +155,7 @@ class UIMode {
     (_this$_transport$send = (_this$_transport = this._transport).sendEvent) === null || _this$_transport$send === void 0 ? void 0 : _this$_transport$send.call(_this$_transport, method, params);
   }
   async _listTests() {
-    const listReporter = new _teleEmitter.TeleReporterEmitter(e => this._dispatchEvent(e.method, e.params));
+    const listReporter = new _teleEmitter.TeleReporterEmitter(e => this._dispatchEvent(e.method, e.params), true);
     const reporter = new _internalReporter.InternalReporter([listReporter]);
     this._config.cliListOnly = true;
     this._config.testIdMatcher = undefined;
@@ -179,7 +179,7 @@ class UIMode {
     this._config.cliListOnly = false;
     this._config.testIdMatcher = id => !testIdSet || testIdSet.has(id);
     const reporters = await (0, _reporters.createReporters)(this._config, 'ui');
-    reporters.push(new _teleEmitter.TeleReporterEmitter(e => this._dispatchEvent(e.method, e.params)));
+    reporters.push(new _teleEmitter.TeleReporterEmitter(e => this._dispatchEvent(e.method, e.params), true));
     const reporter = new _internalReporter.InternalReporter(reporters);
     const taskRunner = (0, _tasks.createTaskRunnerForWatch)(this._config, reporter);
     const testRun = new _tasks.TestRun(this._config, reporter);

package/lib/worker/fixtureRunner.js

@@ -71,7 +71,7 @@ class Fixture {
       }
     }
 
-    // Break the regustration function into before/after steps. Create these before/after stacks
+    // Break the registration function into before/after steps. Create these before/after stacks
     // w/o scopes, and create single mutable step that will be converted into the after step.
     const shouldGenerateStep = !this.registration.hideStep && !this.registration.name.startsWith('_') && !this.registration.option;
     const isInternalFixture = this.registration.location && (0, _util.filterStackFile)(this.registration.location.file);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@playwright/test",
-  "version": "1.36.0-alpha-jun-8-2023",
+  "version": "1.36.0-alpha-jun-10-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.36.0-alpha-jun-8-2023"
+    "playwright-core": "1.36.0-alpha-jun-10-2023"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"