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

E2E Tests: Make App Wall tests more resilient. Split out App Routes test. #3072

Merged
merged 7 commits into from
Sep 27, 2018

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented Sep 27, 2018

The app wall tests fail if there are more than 9 apps that already exist.

This PR makes them more resilient.

It also splits out the App Routes tests so we can pinpoint where the unreliability is.

@nwmac nwmac self-assigned this Sep 27, 2018
@cfdreddbot
Copy link

Hey nwmac!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@nwmac nwmac changed the title E2E Tests: Make App Wall tests more resilient E2E Tests: Make App Wall tests more resilient. Split out App Routes test. Sep 27, 2018
@codecov
Copy link

codecov bot commented Sep 27, 2018

Codecov Report

Merging #3072 into v2-master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           v2-master    #3072   +/-   ##
==========================================
  Coverage      71.01%   71.01%           
==========================================
  Files            607      607           
  Lines          26258    26258           
  Branches        5940     5940           
==========================================
  Hits           18648    18648           
  Misses          7610     7610

@richard-cox richard-cox merged commit 41bd69f into v2-master Sep 27, 2018
@richard-cox richard-cox deleted the e2e-resilience branch September 27, 2018 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants