test: add test coverage for npm_package_bin configuration_env_vars

And test coverage for using --action_env=FOO=BAR to pass env variables to npm_package_bin & nodejs_binary

.bazelrc

@@ -24,6 +24,7 @@ build --noincompatible_no_support_tools_in_action_inputs
 # Define environment value used by some tests such as //internal/npm_install/test:bazel_bin_test
 build --define=SOME_TEST_ENV=some_value
 test --define=SOME_TEST_ENV=some_value
+build --action_env=SOME_OTHER_ENV=some_other_value
 
 ###############################
 # Remote Build Execution support

.circleci/config.yml

@@ -10,7 +10,7 @@
 ## IMPORTANT
 # If you change the `default_docker_image` version, also change the `cache_key` version
 var_1: &default_docker_image circleci/node:10.16
-var_3: &cache_key node-0.16-{{ checksum "yarn.lock" }}
+var_3: &cache_key node-0.16-{{ checksum "yarn.lock" }}-v2
 
 var_4: &init_environment
   run:

internal/node/test/BUILD.bazel

@@ -70,7 +70,10 @@ nodejs_binary(
 
 nodejs_test(
     name = "define_var",
-    configuration_env_vars = ["SOME_TEST_ENV"],
+    configuration_env_vars = [
+        "SOME_TEST_ENV",
+        "SOME_OTHER_ENV",
+    ],
     data = glob(["*.spec.js"]),
     entry_point = ":define.spec.js",
 )
@@ -271,6 +274,7 @@ npm_package_bin(
         "$@",
         "$(SOME_TEST_ENV)",
     ],
+    configuration_env_vars = ["SOME_OTHER_ENV"],
     tool = ":expand_variables",
 )
 

internal/node/test/define.spec.js

@@ -6,3 +6,11 @@ if (process.env['SOME_TEST_ENV'] !== 'some_value') {
   console.error('should accept vars; must be run with --define=SOME_TEST_ENV=some_value');
   process.exitCode = 1;
 }
+
+// Similar to above but testing another env variable that is expected to be specified with
+// `--action_env=SOME_OTHER_ENV=some_other_value`.
+if (process.env['SOME_OTHER_ENV'] !== 'some_other_value') {
+  console.error(
+      'should accept vars; must be run with --action_env=SOME_OTHER_ENV=some_other_value');
+  process.exitCode = 1;
+}

internal/node/test/expand_variables.golden

@@ -1,3 +1,4 @@
 [
-  "some_value"
+  "some_value",
+  "some_other_value"
 ]

internal/node/test/expand_variables.js

@@ -1,4 +1,5 @@
 const fs = require('fs');
 const args = process.argv.slice(2);
 const outfile = args.shift();
+args.push(process.env['SOME_OTHER_ENV']);
 fs.writeFileSync(outfile, JSON.stringify(args, null, 2), 'utf-8');

internal/npm_install/test/bazel_bin_test.sh

@@ -12,8 +12,9 @@ source "${RUNFILES_DIR:-/dev/null}/$f" 2>/dev/null || \
 # --- end runfiles.bash initialization v2 ---
 
 readonly OUT=$($(rlocation "npm/testy/bin/testy.sh"))
+readonly EXPECTED="Hello some_value && some_other_value"
 
-if [ "$OUT" != "Hello some_value" ]; then
-  echo "Expected output 'Hello world' but was '$OUT'"
+if [ "${OUT}" != "${EXPECTED}" ]; then
+  echo "Expected output '${EXPECTED}' but was '${OUT}'"
   exit 1
 fi

tools/npm_packages/testy/index.js

@@ -1 +1 @@
-console.log(`Hello ${process.env['SOME_TEST_ENV']}`);
+console.log(`Hello ${process.env['SOME_TEST_ENV']} && ${process.env['SOME_OTHER_ENV']}`);

tools/npm_packages/testy/package.json

@@ -7,7 +7,7 @@
   "bazelBin": {
     "testy": {
       "additionalAttributes": {
-        "configuration_env_vars": "[\"SOME_TEST_ENV\"]"
+        "configuration_env_vars": "[\"SOME_TEST_ENV\", \"SOME_OTHER_ENV\"]"
       }
     }
   }