8000 Gateway: Integration Playwright Test Stability. by sk-enya · Pull Request #555 · bobanetwork/gateway · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Gateway: Integration Playwright Test Stability. #555

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 63 commits into from
Jul 16, 2024
Merged

Gateway: Integration Playwright Test Stability. #555

merged 63 commits into from
Jul 16, 2024

Conversation

sk-enya
Copy link
Collaborator
@sk-enya sk-enya commented Jun 1, 2024 •
edited
Loading

📋
closes: #582

Setting up integration test for bridging.
Screenshot 2024-07-13 at 10 28 54 PM

sk-enya added 26 commits May 27, 2024 03:22
@sk-enya
change:
b2c89b2 
- added test with playwright
@sk-enya
added testid to label of address
4888a31
@sk-enya
change:
63b24e8 
- removed cypress and corresponding package with synpress
- updated networkservices
- updated wallet.service with type fixes
@sk-enya
change: test spanshot fixes with fix for workflow dispatch.
5fdca79
@sk-enya
change:
32b0242 
- fix flaky connect behaviour
- change build script.
@sk-enya
change:
7610db9 
- moved fixture to specific file.
- added reports generation type to playwright
- added dotenv to read the env params and account details.
- updated the spec to review and connect MM.
@sk-enya
change:
1e2d350 
- updated testid to grab the element in e2e specs
- written integration test for deposit eth on sepolia.
@sk-enya
- written integration test for deposit eth on sepolia.
0004d89
@sk-enya
change:
b9028cf 
- update env params
- update ui params
- update report
- update readme scripts.
@sk-enya
update success modal design
32beef1
@sk-enya
modal fixes
eefb264
@sk-enya
test timeout fixes to wait and pass the test for all scenario
91fdeab
@sk-enya
updated spec file name
0d59b25
@sk-enya
change:
44891a3 
- fetch lookup to be removed
- prepare useLookupPrice hook and calling whenever needed instead of calling on bridge page repeatedly.
@sk-enya
change: removed lookup price reducer, action, selector.
750bbc6
@sk-enya
cleanup import
975ebd4
@sk-enya
change:
4c8a0ef 
- formating
- rpc url update for sepolia
- added integration
10000
 test for boba deposit.
@sk-enya
change:
90c892b 
- running only single test
@sk-enya
change:
9ba163a 
- package script to the class with utility function so it be easy to read, reuse and update.
- add cleanup script.
- updated testId to select the element from page.
- created lib with action to invoke from test.
@sk-enya
moove common code to separate functions
4cb3432
@sk-enya
change:
6f158dc 
- removed loop of checking transfer balance on depositERC20.
- add type inference
- added report generate script.
- test cleanup and getting test to be working!
@sk-enya
change:
9feff6d 
- added tests case for withdrawal flow check on classic.
- updated spec and moved to separate files based on it's usages.
- update id to data-testid for switcher
- update snapshosts unit test
- vertical step fix button text!
@sk-enya
- update snapshosts unit test
153625b
@sk-enya
fix broken test issues of timeout.
224c474
@sk-enya
change:
ecbd3d7 
- getting withdrawal test to be up and running!
@sk-enya
change: light bridge deposit.
8210a62
@sk-enya sk-enya requested review from wsdt, alvaro-ricotta and souradeep-das and removed request for wsdt June 1, 2024 00:46
@sk-enya sk-enya requested a review from boyuan-chen July 14, 2024 10:05
@souradeep-das
Copy link
Contributor

could we make the ci tests pass as well?

souradeep-das
souradeep-das approved these changes Jul 15, 2024
View reviewed changes
alvaro-ricotta
alvaro-ricotta approved these changes Jul 16, 2024
View reviewed changes
Copy link
Contributor
@alvaro-ricotta alvaro-ricotta left a comment

Choose a reason for hiding this comment

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

Looks good.

@sk-enya sk-enya merged commit 55c8676 into main Jul 16, 2024
3 of 5 checks passed
@sk-enya sk-enya deleted the feature/integration-test-playwright branch August 29, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@souradeep-das souradeep-das souradeep-das approved these changes

@boyuan-chen boyuan-chen boyuan-chen approved these changes

@alvaro-ricotta alvaro-ricotta alvaro-ricotta approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

E2E Test Scenario Consideration
4 participants
@sk-enya @souradeep-das @boyuan-chen @alvaro-ricotta
0