fix(builtin): fix linker issue when running test with "local" tag on …

…osx & linux (#1835)

internal/linker/index.js

@@ -89,6 +89,10 @@ function resolveRoot(root, startCwd, isExecroot, runfiles) {
             return fromManifest;
         }
         else {
+            const maybe = path.resolve(`${symlinkRoot}/external/${root}`);
+            if (fs.existsSync(maybe)) {
+                return maybe;
+            }
             return path.resolve(`${startCwd}/../${root}`);
         }
     });

internal/linker/link_node_modules.ts

@@ -142,8 +142,17 @@ async function resolveRoot(
   if (fromManifest) {
     return fromManifest;
   } else {
-    // Under runfiles, the root will be one folder up from the startCwd `runfiles/my_wksp`.
-    // This is true whether legacy external runfiles are on or off.
+    const maybe = path.resolve(`${symlinkRoot}/external/${root}`);
+    if (fs.existsSync(maybe)) {
+      // Under runfiles, when not in the sandbox we must symlink node_modules down at the execroot
+      // `execroot/my_wksp/external/npm/node_modules` since `runfiles/npm/node_modules` will be a
+      // directory and not a symlink back to the root node_modules where we expect
+      // to resolve from. This case is tested in internal/linker/test/local.
+      return maybe;
+    }
+    // However, when in the sandbox, `execroot/my_wksp/external/npm/node_modules` does not exist,
+    // so we must symlink into `runfiles/npm/node_modules`. This directory exists whether legacy
+    // external runfiles are on or off.
     return path.resolve(`${startCwd}/../${root}`)
   }
 }

internal/linker/test/local/BUILD.bazel

@@ -0,0 +1,9 @@
+load("@npm_bazel_jasmine//:index.from_src.bzl", "jasmine_node_test")
+
+jasmine_node_test(
+    name = "test",
+    srcs = ["test.js"],
+    tags = ["local"],
+    templated_args = ["--nobazel_patch_module_resolver"],
+    deps = ["//internal/linker/test/local/fit"],
+)

internal/linker/test/local/fit/BUILD.bazel

@@ -0,0 +1,29 @@
+load("@build_bazel_rules_nodejs//:index.bzl", "pkg_npm")
+load("@npm//typescript:index.bzl", "tsc")
+
+tsc(
+    name = "fit_lib",
+    outs = [
+        "main.d.ts",
+        "main.js",
+    ],
+    args = [
+        "-p",
+        "$(execpath tsconfig.json)",
+        "--outDir",
+        # $(RULEDIR) is a shorthand for the dist/bin directory where Bazel requires we write outputs
+        "$(RULEDIR)",
+    ],
+    data = [
+        "main.ts",
+        "tsconfig.json",
+    ],
+)
+
+pkg_npm(
+    name = "fit",
+    package_name = "fit",
+    srcs = ["package.json"],
+    visibility = ["//internal/linker/test/local:__pkg__"],
+    deps = [":fit_lib"],
+)

internal/linker/test/local/fit/main.ts

@@ -0,0 +1 @@
+export const fit: string = 'fit';

internal/linker/test/local/fit/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "fit",
+  "main": "main.js",
+  "typings": "main.d.ts"
+}

internal/linker/test/local/fit/tsconfig.json

@@ -0,0 +1,5 @@
+{
+  "compilerOptions": {
+    "declaration": true
+  }
+}

internal/linker/test/local/test.js

@@ -0,0 +1,6 @@
+describe('linker', () => {
+  it('should work when job run with "local" tag', () => {
+    const fit = require('fit');
+    expect(fit.fit).toBe('fit');
+  });
+});