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.

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

[BUG] Issue with coverage after istanbul conversion #7109

Closed
ExFlo opened this issue Jun 14, 2021 · 5 comments
Closed

[BUG] Issue with coverage after istanbul conversion #7109

ExFlo opened this issue Jun 14, 2021 · 5 comments

Comments

@ExFlo
Copy link

ExFlo commented Jun 14, 2021

Context:

  • Playwright Version:1.12.2
  • Operating System: Windows
  • Node.js version: 12.8.3
  • Browser: Chromium,
  • Extra:

Code Snippet

Repo to clone
reproducer

Describe the bug

I want to be able to merge the coverage generated by Playwright for each tests into one.
I have an issue using nyc to merge and generate the reports after the conversion to istanbul (as described by documentation).

I don't know if this is linked to coverage generated by Playwright or the other tools.

I am able when using other tool, to instrument the tests and to extract the coverage, to generate coverage but not in case of Playwright tools.

After cloning the repo run:
npm install; npm run e2e:playwright

to reproduce the issue.

Error message is:
"Invalid file coverage object, missing keys, found:path,all,statementMap,s,branchMap,b,fnMap,f"

Indeed the file generated have empty path which leads to the error.

Thx for the help.

BR

@ExFlo ExFlo changed the title [BUG] [BUG] Issue with coverage after istanbul conversion Jun 14, 2021
@mxschmitt
Copy link
Member

Hi, since you are using @playwright/test I would recommend that you wait for #7030. This gives you coverage report out of the box for all the browsers and not only or Chromium.

@mxschmitt
Copy link
Member

Closing as it gets addressed in #7030.

@fschwalm
Copy link

fschwalm commented Aug 3, 2021

Hello @ExFlo! Have you fixed this problem?
I'm facing the same problem.

thank you so much!

@ExFlo
Copy link
Author

ExFlo commented Aug 10, 2021

Hello @ExFlo! Have you fixed this problem?
I'm facing the same problem.

thank you so much!

Hi
No I am waiting for the coverage report system in @playwright/test as described above. Did not landed in 1.13 version...so still waiting.

@fschwalm
Copy link

@mxschmitt any updates?

thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants