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

WIP: Playwright with shared repo, compared to open Playwright PR #235

Closed

Conversation

mickmister
Copy link
Contributor

@mickmister mickmister commented Nov 17, 2023

Summary

This PR is built off of Playwright PR #228, and is pointing to that branch for the sake of comparing the difference between them This PR uses a submodule defined at mattermost-community/mattermost-plugin-e2e-test-utils#1 to centralize the Playwright bootstrapping code.

PR comparing to master #236

Ticket Link

@mickmister mickmister requested a review from larkox as a code owner November 17, 2023 03:50
@mickmister mickmister changed the title Playwright with shared repo Playwright with shared repo, compared to open Playwright PR Nov 17, 2023
@codecov-commenter
Copy link

codecov-commenter commented Nov 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (feat/adding-playwright-e2e@dd496af). Click here to learn what that means.

Additional details and impacted files
@@                     Coverage Diff                      @@
##             feat/adding-playwright-e2e    #235   +/-   ##
============================================================
  Coverage                              ?   6.42%           
============================================================
  Files                                 ?      11           
  Lines                                 ?    1712           
  Branches                              ?       0           
============================================================
  Hits                                  ?     110           
  Misses                                ?    1594           
  Partials                              ?       8           

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

@mickmister mickmister changed the title Playwright with shared repo, compared to open Playwright PR WIP: Playwright with shared repo, compared to open Playwright PR Nov 20, 2023
@mickmister mickmister marked this pull request as draft November 20, 2023 02:15
@mickmister
Copy link
Contributor Author

Closing since #228 was merged

@mickmister mickmister closed this Nov 20, 2023
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

Successfully merging this pull request may close these issues.

2 participants