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

[Bug]: Skipped test has weird 0 in report #29861

Closed
dimkin-eu opened this issue Mar 8, 2024 · 1 comment · Fixed by #30078
Closed

[Bug]: Skipped test has weird 0 in report #29861

dimkin-eu opened this issue Mar 8, 2024 · 1 comment · Fixed by #30078
Labels
feature-html-reporter open-to-a-pull-request The feature request looks good, we are open to reviewing a PR

Comments

@dimkin-eu
Copy link

Version

1.42.1

Steps to reproduce

open report of skipped test
Screenshot 2024-03-08 at 19 03 54

Expected behavior

There could be either 0s or nothing

Actual behavior

0 which scares a bit

Additional context

No response

Environment

System:
    OS: macOS 14.3.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 6.49 GB / 32.00 GB
  Binaries:
    Node: 20.9.0 - /usr/local/bin/node
    npm: 10.1.0 - /usr/local/bin/npm
    pnpm: 8.9.0 - /usr/local/bin/pnpm
  Languages:
    Bash: 3.2.57 - /bin/bash
@dgozman dgozman added open-to-a-pull-request The feature request looks good, we are open to reviewing a PR feature-html-reporter labels Mar 8, 2024
@shriphad-rao
Copy link
Contributor

Hi!
I'll work on this
Can you please assign it to me?

@shriphad-rao shriphad-rao mentioned this issue Mar 23, 2024
Merged
yury-s pushed a commit that referenced this issue Mar 25, 2024
@shriphad-rao
fix: time in ms for skip test (#30078)
82804cc 
Fixes #29861
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-html-reporter open-to-a-pull-request The feature request looks good, we are open to reviewing a PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: time in ms for skip test shriphad-rao/playwright
3 participants
@dimkin-eu @dgozman @shriphad-rao