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

fix(cypress): more time for website:start:prod to finish in cypress script #1744

Merged
merged 1 commit into from
Sep 5, 2019

Conversation

spencercorwin
Copy link
Collaborator

Description of the Change

Bumps the time from 3 minutes to 5 minutes for the website to startup in the runCypress script. 3 minutes wasn't enough time to finish the website:start:prod script, thus causing the Cypress tests to fail when run in --local mode.

Test Plan

Run yarn test:cypress-express --local locally and see the Cypress express test pass.

Alternate Designs

None.

Benefits

Passing tests.

Possible Drawbacks

None.

Applicable Issues

#1740

@spencercorwin
Copy link
Collaborator Author

bors r+

bors bot added a commit that referenced this pull request Sep 5, 2019
1744: fix(cypress): more time for website:start:prod to finish in cypress script r=spencercorwin a=spencercorwin

### Description of the Change

Bumps the time from 3 minutes to 5 minutes for the website to startup in the `runCypress` script. 3 minutes wasn't enough time to finish the `website:start:prod` script, thus causing the Cypress tests to fail when run in `--local` mode.

### Test Plan

Run `yarn test:cypress-express --local` locally and see the Cypress express test pass.

### Alternate Designs

None.

### Benefits

Passing tests.

### Possible Drawbacks

None.

### Applicable Issues

#1740 

Co-authored-by: Spencer Corwin <spencercorwin@icloud.com>
@bors
Copy link
Contributor

bors bot commented Sep 5, 2019

@bors bors bot merged commit 05b0058 into neo-one-suite:master Sep 5, 2019
@spencercorwin spencercorwin deleted the fix-cypress branch October 23, 2019 00:42
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.

1 participant