Account e2e test #1099
Merged
Account e2e test #1099
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@boodland is attempting to deploy a commit to the Chayn Team on Vercel. A member of the Team first needs to authorize it. |
|
@boodland sorry - some conflicts have come up! I don't know what your env variables look like but make sure when you run your tests, the NEXT_PUBLIC_API_URL no longer has a trailing slash. Also I had to update your checkUrl function to only look at pathnames as the domain is not always known on feature branches. Hope that makes sense. I am hoping the tests will be less flakey now! |
* Move meet-the-team page to new cypress test location * Add join us section test * Add our free services section test * Add why bloom section test * Add our bloom team section test * Add our themes section test * Add testimonial section test
* Add ids to form fields * Add admin dashboard tests
Bumps [react-cookie-consent](https://github.com/Mastermindzh/react-cookie-consent) from 8.0.1 to 9.0.0. - [Release notes](https://github.com/Mastermindzh/react-cookie-consent/releases) - [Changelog](https://github.com/Mastermindzh/react-cookie-consent/blob/master/CHANGELOG.md) - [Commits](Mastermindzh/react-cookie-consent@8.0.1...9.0.0) --- updated-dependencies: - dependency-name: react-cookie-consent dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
|
@eleanorreem conflict fixed 🙂, sorry for the long commit history, I just rebase. I think my env var is correct, just to be sure, it is |
Bloom frontend
|
||||||||||||||||||||||||||||||||||
| Project |
Bloom frontend
|
| Branch Review |
develop
|
| Run status |
|
| Run duration | 03m 34s |
| Commit |
|
| Committer | Alberto Arias |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
1
|
Flaky
|
0
|
Pending
|
4
|
Skipped
|
0
|
Passing
|
76
|
| View all changes introduced in this branch ↗︎ | |
Tests for review
cypress/integration/tests/user-disable-service-emails.cy.tsx • 1 failed test
| Test | Artifacts | |
|---|---|---|
| User disable service emails page should display > header section |
Test Replay
Screenshots
|
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue link / number:
#1089
What changes did you make?
Moved welcome e2e tests within tests folder
Created account e2e tests for english version
Why did you make the changes?
So welcome tests are in the right folder
So the page is covered with test and help us to validate it is working when making changes or migrating it to app router
Did you run tests?
Yes