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

[wallet] e2e back to green #1210

Merged
merged 17 commits into from
Oct 7, 2019
Merged

[wallet] e2e back to green #1210

merged 17 commits into from
Oct 7, 2019

Conversation

martinvol
Copy link
Contributor

Description

  • Made test faster running the emulator in parallel with the build
  • Emulator runs without the boot animation and without a window
  • App throws an error if DEFAULT_SYNC_MODE is not set in the .env files, previously the app would crash
  • Disable a couple of tests that broke due app updates for the moment

Tested

itself.

Other changes

Related issues

Backwards compatibility

packages/mobile/src/config.ts Outdated Show resolved Hide resolved
packages/mobile/e2e/Transfer.spec.js Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Oct 5, 2019

Codecov Report

Merging #1210 into master will increase coverage by 0.01%.
The diff coverage is 62.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1210      +/-   ##
==========================================
+ Coverage   66.45%   66.47%   +0.01%     
==========================================
  Files         262      262              
  Lines        7647     7648       +1     
  Branches      440      441       +1     
==========================================
+ Hits         5082     5084       +2     
+ Misses       2466     2464       -2     
- Partials       99      100       +1
Flag Coverage Δ
#mobile 66.47% <62.5%> (+0.01%) ⬆️
Impacted Files Coverage Δ
packages/mobile/src/web3/testnets.ts 0% <ø> (ø) ⬆️
packages/mobile/src/geth/network-config.ts 0% <0%> (ø) ⬆️
packages/mobile/src/geth/geth.ts 25.92% <100%> (ø) ⬆️
packages/mobile/src/web3/contracts.ts 52.17% <100%> (-1.02%) ⬇️
packages/mobile/src/config.ts 92.5% <75%> (-1.95%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25feb78...4a82196. Read the comment docs.

@martinvol martinvol added the automerge Have PR merge automatically when checks pass label Oct 7, 2019
@ashishb ashishb merged commit 33084de into master Oct 7, 2019
@ashishb ashishb deleted the martinvol/fixed_e2e branch October 7, 2019 13:46
aaronmgdr added a commit that referenced this pull request Oct 8, 2019
* master: (35 commits)
  [Wallet] Fix top of emojis cut off in the activity feed (#1243)
  Adding a contract to store minimum required client version (#1081)
  Revert "Feature #909 proxy delegatecall (#1152)" (#1241)
  Use ContractKit to get addresses for Blockchain API (#1175)
  Feature #909 proxy delegatecall (#1152)
  Fix Faucet done message (#1217)
  Updated SETUP.md with new yarn process (#1224)
  Adding `increaseAllowance` and `decreaseAllowance` methods (#1196)
  extracting function signatures (#1061)
  Fix integration hardcode (#1208)
  Fixing flaky governance test (#1155)
  Restore CI branch (#1223)
  [wallet] e2e back to green (#1210)
  [Wallet] Implement new import wallet flow designs (#1209)
  [Wallet] Fix disable conditions for butons on Enter Invite screen (#1214)
  [protocol] Rename infrastructureFraction to proposerFraction (#1174)
  [ck] Proper promise treatment to avoid UnhandledPromises (#1219)
  [ck] Transform StableToken parameters from fixidity format (#1218)
  [wallet]Store encrypted local signing key (#1188)
  2019-10-03 alfajores deployment (#1200)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Revive Wallet app E2E tests
3 participants