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

adding playwright report summary for PRs #8369

Closed
wants to merge 3 commits into from

Conversation

fungairino
Copy link
Collaborator

@fungairino fungairino commented Apr 29, 2024

What does this PR do?

  • Adds a step to the e2e ci job that will comment on the PR with the result of the test run.

Discussion

  • Distinguish mv2 and mv3 report?

Checklist

  • Add jest or playwright tests and/or storybook stories
  • Designate a primary reviewer @BLoe

Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.52%. Comparing base (6608569) to head (2b74482).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8369      +/-   ##
==========================================
+ Coverage   73.50%   73.52%   +0.02%     
==========================================
  Files        1330     1330              
  Lines       41142    41142              
  Branches     7657     7657              
==========================================
+ Hits        30241    30250       +9     
+ Misses      10901    10892       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Apr 29, 2024

Playwright test results

passed  38 passed

Details

stats  38 tests across 15 suites
duration  9 minutes, 42 seconds
commit  2b74482

@fungairino
Copy link
Collaborator Author

I'm going to try opening a new PR to verify this works

@fungairino fungairino closed this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant