[e2e] Enabling/Disabling token conversion on testnets
Explanation
New e2e testcase for checking that "Enabling/Disabling Token Conversion" toggle works correctly. This means testing that:
- shows conversion on test network when toggle is activated
- don't show conversion on test network when deactivated
More Information
- Resolves #14606
Manual Testing Steps
- Build test environment: yarn build:test
- Run test
yarn test:e2e:single test/e2e/tests/advanced-settings.spec.js --browser=firefoxyarn test:e2e:single test/e2e/tests/advanced-settings.spec.js --browser=chrome
Pre-Merge Checklist
- [X ] PR template is filled out
- [ ] IF this PR fixes a bug, a test that would have caught the bug has been added
- [X ] PR is linked to the appropriate GitHub issue
- [ ] PR has been added to the appropriate release Milestone
+ If there are functional changes:
- [ ] Manual testing complete & passed
- [ ] "Extension QA Board" label has been applied
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.
Builds ready [db1212f]
- builds: chrome, firefox, opera
- builds (beta): chrome, firefox, opera
- builds (flask): chrome, firefox, opera
- build viz: Build System
- mv3: Background Module Init Stats
- mv3: UI Init Stats
- mv3: Module Load Stats
- mv3: Bundle Size Stats
- code coverage: Report
- storybook: Storybook
- all artifacts
Page Load Metrics (1703 ± 43 ms)
| Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
|---|---|---|---|---|---|---|---|
| Chrome | Home | firstPaint | 96 | 1826 | 202 | 373 | 179 |
| domContentLoaded | 1516 | 1844 | 1690 | 85 | 41 | ||
| load | 1536 | 1916 | 1703 | 90 | 43 | ||
| domInteractive | 1516 | 1844 | 1690 | 85 | 41 |
@adonesky1 what do you mean by modify these tests so that there is a secondary currency for the test nets? So we are testing both scenarios, if it is present when activated and if it is not present when deactivated. Do you mean something else?
it('should show conversion on test networks when activated'it('should not show conversion on test networks when deactivated'
For the mocking part, that is a great point, I could test only Localnetwork with API response mocked, instead of testing real testnets with real endpoint calls. Ty!!
Builds ready [3dfaff7]
- builds: chrome, firefox, opera
- builds (beta): chrome, firefox, opera
- builds (flask): chrome, firefox, opera
- build viz: Build System
- mv3: Background Module Init Stats
- mv3: UI Init Stats
- mv3: Module Load Stats
- mv3: Bundle Size Stats
- code coverage: Report
- storybook: Storybook
- all artifacts
Page Load Metrics (1771 ± 50 ms)
| Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
|---|---|---|---|---|---|---|---|
| Chrome | Home | firstPaint | 89 | 205 | 118 | 27 | 13 |
| domContentLoaded | 1582 | 1998 | 1749 | 117 | 56 | ||
| load | 1630 | 1998 | 1771 | 103 | 50 | ||
| domInteractive | 1582 | 1997 | 1749 | 117 | 56 |
Builds ready [b7dccd1]
- builds: chrome, firefox, opera
- builds (beta): chrome, firefox, opera
- builds (flask): chrome, firefox, opera
- build viz: Build System
- mv3: Background Module Init Stats
- mv3: UI Init Stats
- mv3: Module Load Stats
- mv3: Bundle Size Stats
- code coverage: Report
- storybook: Storybook
- all artifacts
Page Load Metrics (1823 ± 68 ms)
| Platform | Page | Metric | Min (ms) | Max (ms) | Average (ms) | StandardDeviation (ms) | MarginOfError (ms) |
|---|---|---|---|---|---|---|---|
| Chrome | Home | firstPaint | 93 | 1900 | 207 | 389 | 187 |
| domContentLoaded | 1575 | 2244 | 1799 | 134 | 65 | ||
| load | 1677 | 2244 | 1823 | 141 | 68 | ||
| domInteractive | 1575 | 2244 | 1799 | 134 | 65 |
thank you for your great suggestions @adonesky1 !!