@nx/playwright 21.3.0-canary.20250715-cf994df → 21.3.0-canary.20250716-46d21c7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/playwright",
-  "version": "21.3.0-canary.20250715-cf994df",
+  "version": "21.3.0-canary.20250716-46d21c7",
   "type": "commonjs",
   "homepage": "https://nx.dev",
   "private": false,
@@ -35,9 +35,9 @@
   },
   "dependencies": {
     "@phenomnomnominal/tsquery": "~5.0.1",
-    "@nx/devkit": "21.3.0-canary.20250715-cf994df",
-    "@nx/eslint": "21.3.0-canary.20250715-cf994df",
-    "@nx/js": "21.3.0-canary.20250715-cf994df",
+    "@nx/devkit": "21.3.0-canary.20250716-46d21c7",
+    "@nx/eslint": "21.3.0-canary.20250716-46d21c7",
+    "@nx/js": "21.3.0-canary.20250716-46d21c7",
     "tslib": "^2.3.0",
     "minimatch": "9.0.3"
   },

package/src/plugins/plugin.js

@@ -142,11 +142,26 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
         // Playwright defaults to the following pattern.
         playwrightConfig.testMatch ??= '**/*.@(spec|test).?(c|m)[jt]s?(x)';
         const dependsOn = [];
-        await forEachTestFile((testFile) => {
+        const testFiles = await getAllTestFiles({
+            context,
+            path: testDir,
+            config: playwrightConfig,
+        });
+        for (const testFile of testFiles) {
             const outputSubfolder = (0, node_path_1.relative)(projectRoot, testFile)
                 .replace(/[\/\\]/g, '-')
                 .replace(/\./g, '-');
             const relativeSpecFilePath = (0, devkit_1.normalizePath)((0, node_path_1.relative)(projectRoot, testFile));
+            if (relativeSpecFilePath.includes('../')) {
+                throw new Error('@nx/playwright/plugin attempted to run tests outside of the project root. This is not supported and should not happen. Please open an issue at https://github.com/nrwl/nx/issues/new/choose with the following information:\n\n' +
+                    `\n\n${JSON.stringify({
+                        projectRoot,
+                        testFile,
+                        testFiles,
+                        context,
+                        config: playwrightConfig,
+                    }, null, 2)}`);
+            }
             const targetName = `${options.ciTargetName}--${relativeSpecFilePath}`;
             ciTargetGroup.push(targetName);
             targets[targetName] = {
@@ -175,11 +190,7 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
                 projects: 'self',
                 params: 'forward',
             });
-        }, {
-            context,
-            path: testDir,
-            config: playwrightConfig,
-        });
+        }
         targets[options.ciTargetName] ??= {};
         targets[options.ciTargetName] = {
             executor: 'nx:noop',
@@ -208,17 +219,13 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
     }
     return { targets, metadata };
 }
-async function forEachTestFile(cb, opts) {
+async function getAllTestFiles(opts) {
     const files = await (0, workspace_context_1.getFilesInDirectoryUsingContext)(opts.context.workspaceRoot, opts.path);
     const matcher = createMatcher(opts.config.testMatch);
     const ignoredMatcher = opts.config.testIgnore
         ? createMatcher(opts.config.testIgnore)
         : () => false;
-    for (const file of files) {
-        if (matcher(file) && !ignoredMatcher(file)) {
-            cb(file);
-        }
-    }
+    return files.filter((file) => matcher(file) && !ignoredMatcher(file));
 }
 function createMatcher(pattern) {
     if (Array.isArray(pattern)) {