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

Update e2e tests for MV3 #3766

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Update e2e tests for MV3 #3766

wants to merge 4 commits into from

Conversation

hyphenized
Copy link
Collaborator

@hyphenized hyphenized commented Jan 17, 2025

A few tests still need to be updated but this seems to be working for now.

Latest build: extension-builds-3766 (as of Fri, 17 Jan 2025 06:11:00 GMT).

Makes the new extension service worker available through the previously used background page fixture.
This provides direct access to the same commands executed in the github actions workflow
@hyphenized hyphenized self-assigned this Jan 17, 2025
@hyphenized hyphenized force-pushed the e2e-mv3 branch 2 times, most recently from 1753252 to 5b932d7 Compare January 17, 2025 05:25
Allows intercepting requests from the extension service worker(s) in playwright
@Shadowfiend
Copy link
Contributor

Oof incidentally looks like our upload-action usage is about to be killed on Jan 30 🙃

Looks like we're still seeing some light failures in the tests, might be timing-related though and seems like a better outcome than previously. I'd be down to merge this & keep rolling once the lint issues are resolved.

@hyphenized
Copy link
Collaborator Author

Oof incidentally looks like our upload-action usage is about to be killed on Jan 30 🙃

>.<

Looks like we're still seeing some light failures in the tests, might be timing-related though and seems like a better outcome than previously. I'd be down to merge this & keep rolling once the lint issues are resolved.

Yeah I agree.. let me see if I can get some of these resolved before merging this. For some reason my linter went on PTO 🤔

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