Skip to content

Fixed failing e2e tests and made minor adjustments#5498

Closed
Andepande wants to merge 5 commits into
mainfrom
e2e-appium/Fix-failing-tests
Closed

Fixed failing e2e tests and made minor adjustments#5498
Andepande wants to merge 5 commits into
mainfrom
e2e-appium/Fix-failing-tests

Conversation

@Andepande

Copy link
Copy Markdown
Contributor

Development & PR Process

  1. Follow MetaMask Mobile Coding Standards
  2. Add release-xx label to identify the PR slated for a upcoming release (will be used in release discussion)
  3. Add needs-dev-review label when work is completed
  4. Add needs-qa label when dev review is completed
  5. Add QA Passed label when QA has signed off

Description

A number of e2e/appium tests were failing, this PR fixes those failures with some minor adjustments to the running of wdio tests

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

@Andepande Andepande requested a review from a team as a code owner January 13, 2023 16:58
Comment thread wdio/features/step-definitions/send-flow.steps.js

@cortisiko cortisiko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

left a couple of thangsss, take a look when you get a chance @Andepande

Comment thread wdio.conf.js Outdated
Comment thread wdio/features/step-definitions/add-networks.steps.js
Comment thread wdio/features/NetworkFlow.feature
Comment thread wdio/features/CreatingWalletAccount.feature Outdated
Comment thread wdio/features/screen-objects/CommonScreen.objects.js Outdated
Comment thread wdio/features/ExploreWizard.feature Outdated
Comment thread wdio/features/ExploreWizard.feature Outdated

@cortisiko cortisiko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

small feedback

Comment thread wdio/features/testIDs/Screens/WalletView.testIds.js
Comment thread wdio/features/step-definitions/send-flow.steps.js
Comment thread wdio/features/NetworkFlow.feature
When I tap on the navbar network title button
And I tap on <NETWORK> on Networks list to switch
And I tap on Got it in the network education modal
And I tap on Goerli Test Network on Networks list to switch

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can we revert the changes made in this file? This feature file was created to be the stepping stone for the Send tokens feature file.

And "CUSTOM NETWORKS" tab is displayed on networks screen
When I tap on the "CUSTOM NETWORKS" tab
Then Add button is disabled
# Then Add button is disabled

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

probably remove this comment?

And "CUSTOM NETWORKS" tab is displayed on networks screen
When I tap on the "CUSTOM NETWORKS" tab
Then Add button is disabled
# Then Add button is disabled

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Probably remove this comment

@Andepande Andepande closed this Jan 18, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants