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 the summary url to the PR e2e comment report #8374

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Adding the summary url to the PR e2e comment report #8374

merged 2 commits into from
Apr 30, 2024

Conversation

fungairino
Copy link
Collaborator

@fungairino fungairino commented Apr 30, 2024

What does this PR do?

Adds a quick link to the run summary from the e2e pr comment report.

Checklist

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

Copy link

codecov bot commented Apr 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.53%. Comparing base (20c8ee5) to head (7d22e5c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8374      +/-   ##
==========================================
+ Coverage   73.51%   73.53%   +0.01%     
==========================================
  Files        1333     1333              
  Lines       41245    41245              
  Branches     7677     7677              
==========================================
+ Hits        30322    30328       +6     
+ Misses      10923    10917       -6     

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

Copy link

github-actions bot commented Apr 30, 2024

Playwright test results - MV2

passed  33 passed
flaky  1 flaky
skipped  8 skipped

Details

report  Open report ↗︎
stats  42 tests across 16 suites
duration  7 minutes, 45 seconds
commit  7d22e5c

Flaky tests

chromeSetup › auth.setup.ts › authenticate

Skipped tests

chrome › tests/extensionConsoleActivation.spec.ts › can activate a mod with built-in integration
edge › tests/extensionConsoleActivation.spec.ts › can activate a mod with built-in integration
chrome › tests/runtime/sidebarController.spec.ts › sidebar controller › shows focus dialog in top-level frame
edge › tests/runtime/sidebarController.spec.ts › sidebar controller › shows focus dialog in top-level frame
chrome › tests/runtime/sidebarNavigation.spec.ts › sidebar mod panels are persistent during navigation
chrome › tests/runtime/sidebarNavigation.spec.ts › sidebar form and temporary panels are unavailable after navigation
edge › tests/runtime/sidebarNavigation.spec.ts › sidebar mod panels are persistent during navigation
edge › tests/runtime/sidebarNavigation.spec.ts › sidebar form and temporary panels are unavailable after navigation

Copy link

github-actions bot commented Apr 30, 2024

Playwright test results - MV3

passed  42 passed

Details

report  Open report ↗︎
stats  42 tests across 16 suites
duration  10 minutes, 49 seconds
commit  7d22e5c

Copy link

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

@fungairino fungairino merged commit 92fae7a into main Apr 30, 2024
33 checks passed
@fungairino fungairino deleted the e2e-pr branch April 30, 2024 17:21
@grahamlangford grahamlangford modified the milestones: 1.8.13, 1.8.14 Apr 30, 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.

3 participants