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

Feat/e2e pipeline #1238

Closed
wants to merge 25 commits into from
Closed

Feat/e2e pipeline #1238

wants to merge 25 commits into from

Conversation

JonathanConn
Copy link

WIP

Changes

  • feat: Added e2e pipeline

…d more explorer page testing, started creating fixtures for cleanup, added skeleton for injected wallets
…nd creating playwright fixtures. This will allow for cleaner tests
… easier to update tests in the futue as the ui changes, ec/ V2 -> v3
…or functions, fixed gitignore, added a seperate script for playwright for future modiciation
@vercel
Copy link

vercel bot commented Aug 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web3modal-gallery ❌ Failed (Inspect) Aug 1, 2023 7:13pm
web3modal-laboratory ❌ Failed (Inspect) Aug 1, 2023 7:13pm

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 1, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: e79f92a
Status: ✅  Deploy successful!
Preview URL: https://e5099b63.web3modal-dev.pages.dev
Branch Preview URL: https://feat-e2e-pipeline.web3modal-dev.pages.dev

View logs

@@ -0,0 +1,51 @@
name: Pull request E2E tests

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are you ignoring my previous PR feedback here?

Please either say you disagree with the feedback or agree with it but don't ignore it.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not ignore your pervious feedback
This draft pr was never meant to be reviewed
I was using this PR to test solutions to an issue I linked in our slack
I agree with the changes you suggested and they will be there in the final version

on:
pull_request:
branches:
- V2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feels like a regression to the previous iteration where you also ran upon push

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The previous iteration did not exist, I never wrote a previous iteration
That was auto generated by playwright
I later removed it to try and fix the confusion

@@ -0,0 +1,51 @@
name: Pull request E2E tests
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason you're narrowing it down to PRs now?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed with JS team yesterday, was told to run at the same time as other PR checks

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