fixup! feat(builtin): enable coverage on nodejs_test

internal/node/node.bzl

@@ -156,6 +156,8 @@ def _nodejs_binary_impl(ctx):
             env_vars += "export %s=\"%s\"\n" % (k, ctx.var[k])
 
     # indicates that this was run with `bazel coverage`
+    # and that we should collect and store coverage
+    # TODO: store coverage in the location bazel tells us
     if ctx.configuration.coverage_enabled:
         # TODO: not sure if $RANDOM is the best var here
         # but we need a new dir between runs

internal/node/node_launcher.sh

@@ -148,6 +148,7 @@ if [ "${EXPECTED_EXIT_CODE}" -eq "0" ]; then
   # If we had merely forked a child process here, we'd be responsible
   # for forwarding those OS interactions.
   exec "${node}" "${NODE_OPTIONS[@]}" "${script}" "${ARGS[@]}"
+  # exec terminates execution of this shell script, nothing later will run.
 fi
 
 set +e