jasmine_node_test cannot find c8 even though @npm//c8 is passed in deps

[dylanShark]

🐞 bug report

Affected Rule

The issue is caused by the rule:

jasmine_node_test

Is this a regression?

Not sure

Description

Hi team, we are currently using rules_nodejs 3.4.2. We try to invoke jasmine_node_test like the following:

jasmine_node_test(
    name = "some_test",
    srcs = [
        ":js_src",
    ],
    deps = ["@npm//c8",]
)        

and when running bazel coverage on this target, we get error ERROR: c8 npm package is required for bazel coverage. I also tried putting "@npm//c8", in srcs but it didn't work either. I thought this was fixed in #2100 which should have been included in 3.4.2, could someone please take a look?

🔬 Minimal Reproduction

🔥 Exception or Error

ERROR: c8 npm package is required for bazel coverage

🌍 Your Environment

Operating System:

  
macos 10.15.7
  

Output of bazel version:

  
Build label: 3.7.1
  

Rules_nodejs version:

(Please check that you have matching versions between WORKSPACE file and @bazel/* npm packages.)

  
3.4.2
  

Anything else relevant?

[github-actions]

This issue has been automatically marked as stale because it has not had any activity for 90 days. It will be closed if no further activity occurs in two weeks. Collaborators can add a "cleanup" or "need: discussion" label to keep it open indefinitely. Thanks for your contributions to rules_nodejs!

[github-actions]

This issue was automatically closed because it went two weeks without a reply since it was labeled "Can Close?"