Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

spec reporter don't have support for code coverage #47032

Closed
piranna opened this issue Mar 10, 2023 · 1 comment
Closed

spec reporter don't have support for code coverage #47032

piranna opened this issue Mar 10, 2023 · 1 comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. test_runner Issues and PRs related to the test runner subsystem.

Comments

@piranna
Copy link
Contributor

piranna commented Mar 10, 2023 •
edited
Loading

Version

v19.7.0

Platform

Linux executive 5.19.0-35-generic #36-Ubuntu SMP PREEMPT_DYNAMIC Fri Feb 3 18:36:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Subsystem

test runner

What steps will reproduce the bug?

Run a test with both --experimental-test-coverage and --test-reporter=spec arguments. When using default tap reporter, the --experimental-test-coverage shows proper tests code coverage info.

How often does it reproduce? Is there a required condition?

Always.

What is the expected behavior?

Tests code coverage info should be shown.

What do you see instead?

It's not shown.

Additional information

Since spec reporter is intended to provide an user readable output, I propose to use Istambul text reporter, that's the same being used by Jest, NYC, and by extension also c8.
@debadree25 debadree25 added the test_runner Issues and PRs related to the test runner subsystem. label Mar 10, 2023
@cjihrig
Copy link
Contributor

cjihrig commented Mar 10, 2023

Duplicate of #46665

@cjihrig cjihrig marked this as a duplicate of #46665 Mar 10, 2023
@piranna piranna mentioned this issue Mar 10, 2023
Closed
@aduh95 aduh95 added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Mar 10, 2023
@aduh95 aduh95 closed this as not planned Won't fix, can't repro, duplicate, stale Mar 10, 2023
@piranna piranna mentioned this issue Mar 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. test_runner Issues and PRs related to the test runner subsystem.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@piranna @cjihrig @aduh95 @debadree25