@zest-pw/test 1.0.10 → 1.0.12

package/README.md

@@ -262,6 +262,35 @@ test('TC-001: Test description', async ({ page }) => {
 });
 ```
 
+### Custom Fixtures (Screenshots Still Work)
+
+If you need custom fixtures, extend the exported `test` from `@zest-pw/test`.
+Automatic screenshots after `test.step()` will continue to work.
+
+```typescript
+import { test as base, expect } from '@zest-pw/test';
+
+type MyFixtures = {
+  userRole: string;
+};
+
+export const test = base.extend<MyFixtures>({
+  userRole: async ({}, use) => {
+    await use('admin');
+  },
+});
+
+test('TC-001: Uses custom fixture', async ({ page, userRole }) => {
+  await test.step('Open page', async () => {
+    await page.goto('https://playwright.dev/');
+  });
+
+  await test.step('Assert role', async () => {
+    await expect(userRole).toBe('admin');
+  });
+});
+```
+
 ### Test Naming Convention
 
 Name your test files with test case keys for Zephyr integration:

package/dist/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../config.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,UAAU;IACzB;;OAEG;IACH,QAAQ,CAAC,EAAE;QACT;;;WAGG;QACH,cAAc,CAAC,EAAE,OAAO,CAAC;QAEzB;;;WAGG;QACH,SAAS,CAAC,EAAE,MAAM,CAAC;QAEnB;;;WAGG;QACH,cAAc,CAAC,EAAE,OAAO,CAAC;QAEzB;;;;WAIG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;KACnB,CAAC;IAEF;;OAEG;IACH,WAAW,CAAC,EAAE;QACZ;;;WAGG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;QAElB;;;WAGG;QACH,eAAe,CAAC,EAAE,OAAO,CAAC;QAE1B;;;WAGG;QACH,aAAa,CAAC,EAAE,OAAO,CAAC;QAExB;;;WAGG;QACH,UAAU,CAAC,EAAE,OAAO,CAAC;QAErB;;;WAGG;QACH,QAAQ,CAAC,EAAE,OAAO,CAAC;KACpB,CAAC;IAEF;;OAEG;IACH,MAAM,CAAC,EAAE;QACP;;;WAGG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;QAElB;;WAEG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAEhB;;WAEG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAEhB;;WAEG;QACH,YAAY,CAAC,EAAE,MAAM,CAAC;QAEtB;;;WAGG;QACH,aAAa,CAAC,EAAE,OAAO,CAAC;KACzB,CAAC;CACH;AAED;;GAEG;AACH,eAAO,MAAM,aAAa,EAAE,QAAQ,CAAC,UAAU,CAqB9C,CAAC;AAOF;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,UAAU,GAAG,QAAQ,CAAC,UAAU,CAAC,CAiBzE;AAED;;;;GAIG;AACH,wBAAgB,aAAa,IAAI,QAAQ,CAAC,UAAU,CAAC,CAEpD;AAED;;;GAGG;AACH,wBAAsB,cAAc,IAAI,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,CAcpE"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../config.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,UAAU;IACzB;;OAEG;IACH,QAAQ,CAAC,EAAE;QACT;;;WAGG;QACH,cAAc,CAAC,EAAE,OAAO,CAAC;QAEzB;;;WAGG;QACH,SAAS,CAAC,EAAE,MAAM,CAAC;QAEnB;;;WAGG;QACH,cAAc,CAAC,EAAE,OAAO,CAAC;QAEzB;;;;WAIG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;KACnB,CAAC;IAEF;;OAEG;IACH,WAAW,CAAC,EAAE;QACZ;;;WAGG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;QAElB;;;WAGG;QACH,eAAe,CAAC,EAAE,OAAO,CAAC;QAE1B;;;WAGG;QACH,aAAa,CAAC,EAAE,OAAO,CAAC;QAExB;;;WAGG;QACH,UAAU,CAAC,EAAE,OAAO,CAAC;QAErB;;;WAGG;QACH,QAAQ,CAAC,EAAE,OAAO,CAAC;KACpB,CAAC;IAEF;;OAEG;IACH,MAAM,CAAC,EAAE;QACP;;;WAGG;QACH,OAAO,CAAC,EAAE,OAAO,CAAC;QAElB;;WAEG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAEhB;;WAEG;QACH,MAAM,CAAC,EAAE,MAAM,CAAC;QAEhB;;WAEG;QACH,YAAY,CAAC,EAAE,MAAM,CAAC;QAEtB;;;WAGG;QACH,aAAa,CAAC,EAAE,OAAO,CAAC;KACzB,CAAC;CACH;AAED;;GAEG;AACH,eAAO,MAAM,aAAa,EAAE,QAAQ,CAAC,UAAU,CAqB9C,CAAC;AAOF;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,UAAU,GAAG,QAAQ,CAAC,UAAU,CAAC,CAiBzE;AAED;;;;GAIG;AACH,wBAAgB,aAAa,IAAI,QAAQ,CAAC,UAAU,CAAC,CAEpD;AAED;;;GAGG;AACH,wBAAsB,cAAc,IAAI,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,CAgBpE"}

package/dist/config.js

@@ -130,7 +130,9 @@ async function loadZestConfig() {
         const configPath = process.cwd() + '/zest.config.ts';
         const userConfig = await Promise.resolve(`${configPath}`).then(s => __importStar(require(s)));
         if (userConfig.default) {
-            return userConfig.default;
+            // Update global loadedConfig so getZestConfig() returns the same values
+            loadedConfig = userConfig.default;
+            return loadedConfig;
         }
         return loadedConfig;
     }

package/dist/fixtures/fixtures.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fixtures.d.ts","sourceRoot":"","sources":["../../fixtures/fixtures.ts"],"names":[],"mappings":"AAKA,eAAO,MAAM,IAAI,6OAQf,CAAC;AAKH,OAAO,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC"}
+{"version":3,"file":"fixtures.d.ts","sourceRoot":"","sources":["../../fixtures/fixtures.ts"],"names":[],"mappings":"AAmCA,eAAO,MAAM,IAAI,6OAAsC,CAAC;AAExD,OAAO,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC"}

package/dist/fixtures/fixtures.js

@@ -4,14 +4,28 @@ exports.expect = exports.test = void 0;
 const test_1 = require("@playwright/test");
 const test_step_wrapper_1 = require("../utils/test-step-wrapper");
 let currentTestContext = null;
-exports.test = test_1.test.extend({
+const screenshotWrapperApplied = Symbol('zestScreenshotWrapperApplied');
+function enableStepScreenshots(targetTest) {
+    const testWithMeta = targetTest;
+    if (testWithMeta[screenshotWrapperApplied]) {
+        return targetTest;
+    }
+    (0, test_step_wrapper_1.wrapTestStepWithScreenshots)(targetTest, () => currentTestContext);
+    testWithMeta[screenshotWrapperApplied] = true;
+    const originalExtend = targetTest.extend.bind(targetTest);
+    targetTest.extend = ((...args) => {
+        const extendedTest = originalExtend(...args);
+        return enableStepScreenshots(extendedTest);
+    });
+    return targetTest;
+}
+const zestBaseTest = test_1.test.extend({
     page: async ({ page }, use, testInfo) => {
         currentTestContext = { testInfo, page };
         await use(page);
         currentTestContext = null;
     },
 });
-// Apply wrapper to test.step for automatic screenshots
-(0, test_step_wrapper_1.wrapTestStepWithScreenshots)(exports.test, () => currentTestContext);
+exports.test = enableStepScreenshots(zestBaseTest);
 var test_2 = require("@playwright/test");
 Object.defineProperty(exports, "expect", { enumerable: true, get: function () { return test_2.expect; } });

package/dist/utils/take-screenshots.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"take-screenshots.d.ts","sourceRoot":"","sources":["../../utils/take-screenshots.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAGvD;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAsB,uBAAuB,CAC3C,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,QAAQ,EAClB,SAAS,CAAC,EAAE,MAAM,EAClB,QAAQ,CAAC,EAAE,OAAO,GACjB,OAAO,CAAC,IAAI,CAAC,CA2Bf"}
+{"version":3,"file":"take-screenshots.d.ts","sourceRoot":"","sources":["../../utils/take-screenshots.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAGvD;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAsB,uBAAuB,CAC3C,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,QAAQ,EAClB,SAAS,CAAC,EAAE,MAAM,EAClB,QAAQ,CAAC,EAAE,OAAO,GACjB,OAAO,CAAC,IAAI,CAAC,CA4Bf"}

package/dist/utils/take-screenshots.js

@@ -32,6 +32,7 @@ async function takeScreenshotAfterStep(page, testInfo, stepTitle, hasError) {
                 return;
             }
             // Check if we should only take screenshots on failure
+            // If onlyOnFailure is true, only take screenshot when hasError is explicitly true
             if (config.screenshots.onlyOnFailure && !hasError) {
                 return;
             }

package/dist/utils/test-step-wrapper.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"test-step-wrapper.d.ts","sourceRoot":"","sources":["../../utils/test-step-wrapper.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,IAAI,EAAgB,IAAI,IAAI,IAAI,EAAY,MAAM,kBAAkB,CAAC;AAG7F;;;;GAIG;AACH,wBAAgB,2BAA2B,CACzC,IAAI,EAAE,OAAO,IAAI,EACjB,iBAAiB,EAAE,MAAM;IAAE,QAAQ,EAAE,QAAQ,CAAC;IAAC,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG,IAAI,GACjE,IAAI,CAmCN"}
+{"version":3,"file":"test-step-wrapper.d.ts","sourceRoot":"","sources":["../../utils/test-step-wrapper.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,IAAI,EAAgB,IAAI,IAAI,IAAI,EAAY,MAAM,kBAAkB,CAAC;AAG7F;;;;GAIG;AACH,wBAAgB,2BAA2B,CACzC,IAAI,EAAE,OAAO,IAAI,EACjB,iBAAiB,EAAE,MAAM;IAAE,QAAQ,EAAE,QAAQ,CAAC;IAAC,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG,IAAI,GACjE,IAAI,CAuCN"}

package/dist/utils/test-step-wrapper.js

@@ -12,6 +12,9 @@ function wrapTestStepWithScreenshots(test, getCurrentContext) {
     const originalTestStepSkip = test.step.skip.bind(test);
     const takeScreenshot = async (title, hasError = false) => {
         const context = getCurrentContext();
+        if (!context) {
+            return;
+        }
         try {
             await (0, take_screenshots_1.takeScreenshotAfterStep)(context.page, context.testInfo, title, hasError);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zest-pw/test",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "Advanced Playwright test framework with automatic screenshots, custom reporting, and Zephyr Scale integration",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",