@applitools/eyes-playwright 1.37.3 → 1.38.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.38.0](https://github.com/Applitools-Dev/sdk/compare/js/eyes-playwright@1.37.3...js/eyes-playwright@1.38.0) (2025-07-09)
+
+
+### Features
+
+* canvas webgl without preserveDrawingBuffer=1 | FLD-3144 ([#3040](https://github.com/Applitools-Dev/sdk/issues/3040)) ([f82d8f1](https://github.com/Applitools-Dev/sdk/commit/f82d8f148f913098752ec7bef8635a46b453d6fa))
+* support sharding for the eyes-playwright fixture | FLD-3026 ([#3056](https://github.com/Applitools-Dev/sdk/issues/3056)) ([96191bc](https://github.com/Applitools-Dev/sdk/commit/96191bcf8f13b5360ae5e50e13f20a6b198ff78e))
+
+
+### Bug Fixes
+
+* export type Eyes from the PW fixture ([fd3a9f0](https://github.com/Applitools-Dev/sdk/commit/fd3a9f025da6023f6d500cebd2bf65d26acb5f3e))
+* playwright worker configuration | FLD-2837 ([#3029](https://github.com/Applitools-Dev/sdk/issues/3029)) ([07014ec](https://github.com/Applitools-Dev/sdk/commit/07014ec5f3a16ed620c6ac243523cee05f02a7fd))
+
 ## [1.37.3](https://github.com/Applitools-Dev/sdk/compare/js/eyes-playwright@1.37.2...js/eyes-playwright@1.37.3) (2025-06-15)
 
 

package/dist/fixture/index.js

@@ -22,6 +22,9 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.test = exports.expect = void 0;
 /* eslint-disable no-console */
@@ -33,6 +36,7 @@ Object.defineProperty(exports, "expect", { enumerable: true, get: function () {
 const utils = __importStar(require("@applitools/utils"));
 const reporter_1 = require("./reporter");
 const url_1 = require("url");
+const path_1 = __importDefault(require("path"));
 exports.test = test_1.test.extend({
     eyesConfig: [{}, { option: true }],
     eyesRunner: [
@@ -85,10 +89,20 @@ exports.test = test_1.test.extend({
     ],
     closeEyesIfNeeded: [
         async ({ page, eyesConfig, eyesRunner: runner }, use, testInfo) => {
-            const isUsingEyesReporter = testInfo.config.reporter.some(r => r[0].includes('eyes-playwright'));
+            var _a, _b, _c, _d, _e, _f;
+            const isUsingEyesReporter = testInfo.config.reporter.some(
+            // typeof testInfo.config.reporter is [string, ReporterOptions][], that's why [0] is the reporter name
+            r => r[0].includes('eyes-playwright') || r[0].includes('blob'));
+            const blobSettings = (_b = (_a = testInfo.config.reporter.find(r => r[0].includes('blob'))) === null || _a === void 0 ? void 0 : _a[1]) !== null && _b !== void 0 ? _b : {};
+            const blobFolder = (_d = (_c = blobSettings.outputDir) !== null && _c !== void 0 ? _c : process.env.PLAYWRIGHT_BLOB_OUTPUT_DIR) !== null && _d !== void 0 ? _d : (((_e = process.env.PLAYWRIGHT_BLOB_OUTPUT_FILE) !== null && _e !== void 0 ? _e : blobSettings.outputFile)
+                ? path_1.default.basename((_f = process.env.PLAYWRIGHT_BLOB_OUTPUT_FILE) !== null && _f !== void 0 ? _f : blobSettings.outputFile)
+                : '');
             if (isUsingEyesReporter) {
-                const internalId = utils.general.guid();
-                await testInfo.attach('internalId', { body: internalId });
+                const applitoolsIdentifier = utils.general.guid();
+                // Create an identifier that will be collected by the html or reporter
+                await testInfo.attach('applitoolsIdentifier', {
+                    body: `${applitoolsIdentifier}|${blobFolder}`,
+                });
             }
             await use();
             const eyes = page.__eyes;
@@ -132,14 +146,16 @@ function writeResultsWhenReady({ eyes, runner, testInfo }) {
     }));
 }
 async function getEyesConfigForWorker(workerInfo) {
-    var _a, _b, _c;
+    var _a, _b, _c, _d;
     const { configFile } = workerInfo.config;
-    const playwrightConfig = (_b = (_a = ((configFile ? await import((0, url_1.pathToFileURL)(configFile).href) : {}))) === null || _a === void 0 ? void 0 : _a.use) === null || _b === void 0 ? void 0 : _b.eyesConfig;
+    const playwrightConfigModule = configFile ? await import((0, url_1.pathToFileURL)(configFile).href) : {};
+    const playwrightConfig = ((_a = playwrightConfigModule === null || playwrightConfigModule === void 0 ? void 0 : playwrightConfigModule.use) === null || _a === void 0 ? void 0 : _a.eyesConfig) ||
+        ((_c = (_b = playwrightConfigModule === null || playwrightConfigModule === void 0 ? void 0 : playwrightConfigModule.default) === null || _b === void 0 ? void 0 : _b.use) === null || _c === void 0 ? void 0 : _c.eyesConfig);
     const projectConfig = workerInfo.project.use.eyesConfig;
     const eyesConfig = {
         ...playwrightConfig,
         ...projectConfig,
     };
-    (_c = eyesConfig.failTestsOnDiff) !== null && _c !== void 0 ? _c : (eyesConfig.failTestsOnDiff = process.env.CI ? 'afterAll' : 'afterEach');
+    (_d = eyesConfig.failTestsOnDiff) !== null && _d !== void 0 ? _d : (eyesConfig.failTestsOnDiff = process.env.CI ? 'afterAll' : 'afterEach');
     return eyesConfig;
 }

package/dist/fixture/reporter.js

@@ -24,7 +24,6 @@ var __importStar = (this && this.__importStar) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.InternalData = void 0;
-const os = __importStar(require("os"));
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const playwrightPath = require.resolve('playwright');
@@ -46,30 +45,36 @@ const createInjectedScript = (testResultsMap) => `
 class EyesReporter extends HtmlReporter {
     constructor(options) {
         super(options);
-        this.internalIds = [];
+        this.applitoolsIdentifiersAndFolders = [];
     }
     onTestEnd(test, result) {
         var _a;
         (_a = super.onTestEnd) === null || _a === void 0 ? void 0 : _a.call(this, test, result);
-        const index = result.attachments.findIndex(a => a.name === 'internalId');
+        const index = result.attachments.findIndex(a => a.name === 'applitoolsIdentifier');
         if (index > -1) {
-            const internalId = result.attachments[index].body.toString();
-            this.internalIds.push(internalId);
+            const applitoolsIdentifierAndFolder = result.attachments[index].body.toString();
+            this.applitoolsIdentifiersAndFolders.push(applitoolsIdentifierAndFolder);
             this.removeInternalIdAttachment(result);
         }
     }
+    // TODO: let's remove this method! Removing the internal id from the test result is wrong,
+    // it should be hidden in the HTML report and the test result should be read-only.
     removeInternalIdAttachment(result) {
-        var _a;
-        const index = result.attachments ? result.attachments.findIndex(a => a.name === 'internalId') : -1;
+        var _a, _b, _c;
+        const index = result.attachments ? result.attachments.findIndex(a => (a === null || a === void 0 ? void 0 : a.name) === 'applitoolsIdentifier') : -1;
         if (index > -1)
             (_a = result.attachments) === null || _a === void 0 ? void 0 : _a.splice(index, 1);
-        result.steps.forEach(step => this.removeInternalIdAttachment(step));
+        const index2 = result.attachments ? result.attachments.findIndex(a => !a) : -1;
+        if (index2 > -1)
+            (_c = (_b = result._endPayload) === null || _b === void 0 ? void 0 : _b.attachments) === null || _c === void 0 ? void 0 : _c.splice(index2, 1);
+        if (result.steps)
+            result.steps.forEach(step => this.removeInternalIdAttachment(step));
     }
     async onEnd(result) {
         await super.onEnd(result);
         const testResultsMap = {};
-        for (const internalId of this.internalIds) {
-            const content = await exports.InternalData.consume(internalId);
+        for (const applitoolsIdentifierAndFolder of this.applitoolsIdentifiersAndFolders) {
+            const content = await exports.InternalData.consume(applitoolsIdentifierAndFolder);
             if (content) {
                 testResultsMap[content.key] = content.data;
             }
@@ -87,29 +92,36 @@ class EyesReporter extends HtmlReporter {
 exports.default = EyesReporter;
 exports.InternalData = {
     async write({ testInfo, data }) {
-        const internalId = testInfo.attachments.find(a => a.name === 'internalId').body.toString();
+        const [applitoolsIdentifier, folder] = testInfo.attachments
+            .find(a => a.name === 'applitoolsIdentifier')
+            .body.toString()
+            .split('|')
+            .map(s => s.trim())
+            .filter(Boolean); // if the directory is not specified, it will be empty
         const key = `${testInfo.testId}--${testInfo.retry}`;
-        this.ensureFolder();
-        await fs.promises.writeFile(this.getPathForInternalId(internalId), JSON.stringify({ key, data }));
+        this.ensureFolder(folder);
+        await fs.promises.writeFile(this.getPathForInternalId(applitoolsIdentifier, folder), JSON.stringify({ key, data }));
     },
-    async consume(internalId) {
-        const filepath = this.getPathForInternalId(internalId);
+    async consume(applitoolsIdentifierAndFolder) {
+        const [name, folder] = applitoolsIdentifierAndFolder
+            .split('|')
+            .map(s => s.trim())
+            .filter(Boolean); // if the directory is not specified, it will be empty
+        const filepath = this.getPathForInternalId(name, folder);
         try {
             const content = await fs.promises.readFile(filepath, 'utf-8').then(JSON.parse);
-            await fs.promises.unlink(filepath);
             return content;
         }
         catch (err) {
             // TODO no eyes results for test - this is ok, could be that no visual tests occurred for this playwright test
         }
     },
-    ensureFolder() {
-        const folderPath = path.join(os.tmpdir(), 'eyes-report-files');
+    ensureFolder(folderPath = path.join('blob-report', 'eyes-report-files')) {
         if (!fs.existsSync(folderPath)) {
-            fs.mkdirSync(folderPath);
+            fs.mkdirSync(folderPath, { recursive: true });
         }
     },
-    getPathForInternalId(internalId) {
-        return path.join(os.tmpdir(), 'eyes-report-files', internalId);
+    getPathForInternalId(applitoolsIdentifier, folderPath = path.join('blob-report', 'eyes-report-files')) {
+        return path.join(folderPath, applitoolsIdentifier);
     },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/eyes-playwright",
-  "version": "1.37.3",
+  "version": "1.38.0",
   "description": "Applitools Eyes SDK for Playwright",
   "keywords": [
     "eyes-playwright",
@@ -54,14 +54,14 @@
     "test": "APPLITOOLS_SHOW_LOGS=true APPLITOOLS_LOG_FILE=\"./logs/$(uuidgen).log\" APPLITOOLS_BATCH_NAME='JS Tests: eyes-playwright' APPLITOOLS_BATCH_ID=$(uuidgen) run --top-level mocha './test/*.spec.{js,ts}' --exit",
     "test:coverage": "APPLITOOLS_SHOW_LOGS=true APPLITOOLS_LOG_FILE=\"./logs/$(uuidgen).log\" APPLITOOLS_BATCH_NAME='JS Coverage Tests: eyes-playwright' APPLITOOLS_BATCH_ID=$(uuidgen) MOCHA_GROUP=coverage run --top-level mocha './test/generated-coverage/*.spec.js' --parallel --jobs ${MOCHA_JOBS:-8} --exit",
     "test:fixture": "APPLITOOLS_SHOW_LOGS=true APPLITOOLS_LOG_FILE=\"./logs/$(npx uuid).log\" APPLITOOLS_BATCH_NAME='JS Coverage Tests: eyes-playwright-fixture' APPLITOOLS_BATCH_ID=$(npx uuid) run playwright test",
-    "setup": "run playwright:setup",
+    "setup": "run playwright:setup && run --top-level xvfb:setup",
     "playwright:setup": "yarn playwright install --with-deps chromium firefox webkit",
     "up:framework": "echo \"$(jq '.devDependencies.playwright = $ENV.APPLITOOLS_FRAMEWORK_VERSION' ./package.json)\" > ./package.json"
   },
   "dependencies": {
     "@applitools/eyes": "1.35.1",
-    "@applitools/spec-driver-playwright": "1.5.13",
-    "@applitools/utils": "1.8.2",
+    "@applitools/spec-driver-playwright": "1.6.0",
+    "@applitools/utils": "1.9.0",
     "@inquirer/prompts": "7.0.1",
     "chalk": "4.1.2",
     "yargs": "17.7.2"