@gooddata/sdk-e2e-utils 11.63.0-alpha.1 → 11.63.0-alpha.3

package/esm/create-test/feature-hub/mock.d.ts

@@ -1,7 +1,14 @@
 import type { Page } from "@playwright/test";
 import type { IFeatureHubEnvironment, IFeatureHubFeature } from "./types.js";
 export declare function makeFeatureHubResponse(features: IFeatureHubFeature[]): IFeatureHubEnvironment[];
-/** FeatureHub mock — each test gets a fresh page, so route dies with it. */
+/**
+ * FeatureHub mock — each test gets a fresh page, so route dies with it.
+ *
+ * The app reads FeatureHub once, on first load, so the mock must be in place before the page navigates
+ * anywhere. Registering it afterwards (e.g. `featureFlags` on an inner describe/test while an outer
+ * describe's `beforeEach` already called `page.goto`) would leave the app running on the previous
+ * features, which fails in confusing ways. Fail fast instead.
+ */
 export declare function registerFeatureHubMock(page: Page, features: IFeatureHubFeature[]): Promise<void>;
 /** Remove the FeatureHub mock registered by {@link registerFeatureHubMock}. */
 export declare function unregisterFeatureHubMock(page: Page): Promise<void>;

package/esm/create-test/feature-hub/mock.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"mock.d.ts","sourceRoot":"","sources":["../../../src/create-test/feature-hub/mock.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAE7C,OAAO,KAAK,EAAE,sBAAsB,EAAE,kBAAkB,EAAE,MAAM,YAAY,CAAC;AAI7E,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,kBAAkB,EAAE,GAAG,sBAAsB,EAAE,CAO/F;AAED,4EAA4E;AAC5E,wBAAsB,sBAAsB,CAAC,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,kBAAkB,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAWtG;AAED,+EAA+E;AAC/E,wBAAsB,wBAAwB,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAExE;AAED,iFAAiF;AACjF,wBAAsB,wBAAwB,CAC1C,IAAI,EAAE,IAAI,EACV,kBAAkB,EAAE,kBAAkB,EAAE,GACzC,OAAO,CAAC,IAAI,CAAC,CAGf"}
+{"version":3,"file":"mock.d.ts","sourceRoot":"","sources":["../../../src/create-test/feature-hub/mock.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAE7C,OAAO,KAAK,EAAE,sBAAsB,EAAE,kBAAkB,EAAE,MAAM,YAAY,CAAC;AAI7E,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,kBAAkB,EAAE,GAAG,sBAAsB,EAAE,CAO/F;AAED;;;;;;;GAOG;AACH,wBAAsB,sBAAsB,CAAC,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,kBAAkB,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAmBtG;AAED,+EAA+E;AAC/E,wBAAsB,wBAAwB,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAExE;AAED,iFAAiF;AACjF,wBAAsB,wBAAwB,CAC1C,IAAI,EAAE,IAAI,EACV,kBAAkB,EAAE,kBAAkB,EAAE,GACzC,OAAO,CAAC,IAAI,CAAC,CAGf"}

package/esm/create-test/feature-hub/mock.js

@@ -8,8 +8,20 @@ export function makeFeatureHubResponse(features) {
         },
     ];
 }
-/** FeatureHub mock — each test gets a fresh page, so route dies with it. */
+/**
+ * FeatureHub mock — each test gets a fresh page, so route dies with it.
+ *
+ * The app reads FeatureHub once, on first load, so the mock must be in place before the page navigates
+ * anywhere. Registering it afterwards (e.g. `featureFlags` on an inner describe/test while an outer
+ * describe's `beforeEach` already called `page.goto`) would leave the app running on the previous
+ * features, which fails in confusing ways. Fail fast instead.
+ */
 export async function registerFeatureHubMock(page, features) {
+    if (page.url() !== "about:blank") {
+        throw new Error(`@gooddata/sdk-e2e-utils: FeatureHub mock registered after the page navigated to "${page.url()}". ` +
+            "Features are read on first load, so the mock has no effect now. Register `featureFlags` " +
+            "on a describe/test whose hooks run before any `page.goto`, or move the navigation after it.");
+    }
     const body = JSON.stringify(makeFeatureHubResponse(features));
     await page.route(FEATURE_HUB_ROUTE, async (route) => {
         await route.fulfill({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gooddata/sdk-e2e-utils",
-  "version": "11.63.0-alpha.1",
+  "version": "11.63.0-alpha.3",
   "description": "GoodData utility functions for Playwright E2E tests",
   "license": "MIT",
   "author": "GoodData",
@@ -50,8 +50,8 @@
     "oxlint-plugin-eslint": "1.82.0",
     "oxlint-tsgolint": "7.0.2001",
     "typescript": "7.0.2",
-    "@gooddata/oxlint-config": "11.63.0-alpha.1",
-    "@gooddata/eslint-config": "11.63.0-alpha.1"
+    "@gooddata/eslint-config": "11.63.0-alpha.3",
+    "@gooddata/oxlint-config": "11.63.0-alpha.3"
   },
   "scripts": {
     "_phase:build": "npm run build",