-
Notifications
You must be signed in to change notification settings - Fork 326
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
Clean up integration tests and add listeners for backend calls #11847
Merged
+2,479
−2,357
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
1a64456
Remove section headers from integration tests
somebody1234 d5eee57
Add `calls` to E2E tests
somebody1234 3a22990
Adjust tests
somebody1234 7f9e042
Fix type errors
somebody1234 0072570
Use named imports in Dashboard integration tests
somebody1234 490602e
WIP: `goToSettingsTabActions`
somebody1234 c7f4ea0
Update `userSettings.spec` to use new API
somebody1234 b7daa29
Finish updating `userSettings.spec` to use new API
somebody1234 62a9427
Update `organizationSettings.spec` to use new API
somebody1234 75efa0a
Switch integration tests to use named imports
somebody1234 554e174
Add `DrivePageActions.withSearchBar` to switch `assetSearchBar.spec` …
somebody1234 e049e33
Basic conversion of E2E tests to new structure
somebody1234 3538721
Fix circular imports in integration tests
somebody1234 e1cbb0e
Merge branch 'develop' into wip/sb/more-integration-tests
somebody1234 2e086f0
Fix Playwright imports
somebody1234 20af42c
Formatting
somebody1234 9b2d078
Use `page` from `actions.step`
somebody1234 8179f17
Fix broken integration tests
somebody1234 abcb91b
Fix integration tests
somebody1234 78a515a
Merge branch 'develop' into wip/sb/more-integration-tests
somebody1234 37edeb0
Fix more integration tests
somebody1234 9f9982a
Fix more integration tests
somebody1234 cea2aeb
Fix test errors
somebody1234 a9f45ea
Consistent formatting
somebody1234 8581f73
Fix lat integration test
somebody1234 f10d637
Fix
somebody1234 c796cde
Track backend calls in `organizationSettings.spec`
somebody1234 10ecb07
Add more docs for integration tests
somebody1234 6c0db07
Uncomment tests and skip them instead
somebody1234 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix more integration tests
commit 37edeb030e5f25fc34350ecaa104f4b95d2e7839
There are no files selected for viewing
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
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
btw, why we need to
setup.slow()
here?