Skip to content
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

Integration tests #44

Merged
merged 40 commits into from
Mar 12, 2020
Merged

Integration tests #44

merged 40 commits into from
Mar 12, 2020

Conversation

petarbaykov
Copy link
Contributor

@petarbaykov petarbaykov commented Mar 11, 2020

  • aex2.js spec sometimes pass and sometime not (I think is problem with cypress) and runs only in headed browser
  • names and settings pages don't have tests, because I am not sure if the UI will remain the same (currently is same as waellet UI)

petarbaykov and others added 30 commits March 9, 2020 13:59
* Send INIT_RPC message only if IS_EXTENSION
* Test cases for import account page
* Add more cy commands
* Run build before start tests
* Cleanup cypress config
* Increase cypress getting element timeout
* Amount send more tests
* More tests cases for tip page
* Extract some helpers function in file so they can be tested or reused in tests
* Test cases for pending txs
* Test cases for last visited route
* Extract mock login method for reuse in tests and extension
* Create build folder for extension that will run in tests browser
* Load extension before in tests browser before browser is started
@mradkov mradkov self-requested a review March 12, 2020 11:07
Copy link
Contributor

@mradkov mradkov left a comment

Choose a reason for hiding this comment

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

Thanks for this initial test coverage!

@mradkov mradkov merged commit 0fa3aa9 into master Mar 12, 2020
@davidyuk davidyuk deleted the tests branch March 16, 2020 13:49
@petarbaykov petarbaykov mentioned this pull request Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants