PR feedback

vercel · Jan 30, 2025 · 525f3c2 · 525f3c2
1 parent ff78679
commit 525f3c2
Showing 1 changed file with 7 additions and 18 deletions.
diff --git a/crates/turborepo-globwalk/src/lib.rs b/crates/turborepo-globwalk/src/lib.rs
@@ -895,30 +895,19 @@ mod test {
         ; "nested packages work")]
     #[test_case(&[
             "/external/file.txt",
-            "/repos/some-app/apps/docs/package.json",
-            "/repos/some-app/apps/web/package.json",
-            "/repos/some-app/bower_components/readline/package.json",
-            "/repos/some-app/examples/package.json",
-            "/repos/some-app/node_modules/gulp/bower_components/readline/package.json",
-            "/repos/some-app/node_modules/react/package.json",
             "/repos/some-app/package.json",
+            "/repos/some-app/index.js",
+            "/repos/some-app/just-a-dir/index.js",
             "/repos/some-app/packages/xzibit/package.json",
-            "/repos/some-app/packages/xzibit/node_modules/street-legal/package.json",
-            "/repos/some-app/packages/xzibit/node_modules/paint-colors/package.json",
-            "/repos/some-app/packages/xzibit/packages/yo-dawg/package.json",
-            "/repos/some-app/packages/xzibit/packages/yo-dawg/node_modules/meme/package.json",
-            "/repos/some-app/packages/xzibit/packages/yo-dawg/node_modules/yo-dawg/package.json",
+            "/repos/some-app/packages/xzibit/index.js",
             "/repos/some-app/packages/colors/package.json",
-            "/repos/some-app/packages/faker/package.json",
-            "/repos/some-app/packages/left-pad/package.json",
-            "/repos/some-app/test/mocks/spanish-inquisition/package.json",
-            "/repos/some-app/tests/mocks/spanish-inquisition/package.json",
+            "/repos/some-app/packages/colors/index.js",
         ],
         "/repos/some-app/",
-        &["**/package.json"],
+        &["**/*.js"],
         &[],
-        &["/repos/some-app/package.json"],
-        &["/repos/some-app/package.json"],
+        &["/repos/some-app/index.js", "/repos/some-app/just-a-dir/index.js"],
+        &["/repos/some-app/index.js", "/repos/some-app/just-a-dir/index.js"],
         Settings::default().ignore_nested_packages()
         ; "ignore nested workspaces setting only matches top level package")]
     #[test_case(&[