Skip to content

Fix failing Switch E2E tests#22698

Closed
CodingItWrong wants to merge 2 commits intofacebook:masterfrom
CodingItWrong:switch-e2e-flake
Closed

Fix failing Switch E2E tests#22698
CodingItWrong wants to merge 2 commits intofacebook:masterfrom
CodingItWrong:switch-e2e-flake

Conversation

@CodingItWrong
Copy link
Copy Markdown
Contributor

Switch E2E tests have been failing on master, although they pass locally.

Changelog:

Help reviewers and the release process by writing your own changelog entry. See http://facebook.github.io/react-native/docs/contributing#changelog for an example.

[General] [Fixed] - Fix failing Switch E2E tests

Test Plan:

Switch E2E tests should pass on CI

This is an attempt to prevent random failures of Switch tests
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Tests This PR adds or edits a test case. Import Started This pull request has been imported. This does not imply the PR has been approved. labels Dec 18, 2018
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@elicwhite
Copy link
Copy Markdown
Member

This is sad, it more than doubles the time each test takes to run.

@react-native-bot
Copy link
Copy Markdown
Collaborator

@CodingItWrong merged commit bb09866 into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Dec 19, 2018
@react-native-bot react-native-bot added the Merged This PR has been merged. label Dec 19, 2018
grabbou pushed a commit that referenced this pull request Jan 3, 2019
Summary:
Switch E2E tests have been failing on master, although they pass locally.

Changelog:
----------

Help reviewers and the release process by writing your own changelog entry. See http://facebook.github.io/react-native/docs/contributing#changelog for an example.

[General] [Fixed] - Fix failing Switch E2E tests
Pull Request resolved: #22698

Differential Revision: D13511917

Pulled By: hramos

fbshipit-source-id: b2f5cac1927531a855699e34e1a37036773f0aad
@hramos hramos removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Feb 6, 2019
t-nanava pushed a commit to microsoft/react-native-macos that referenced this pull request Jun 17, 2019
Summary:
Switch E2E tests have been failing on master, although they pass locally.

Changelog:
----------

Help reviewers and the release process by writing your own changelog entry. See http://facebook.github.io/react-native/docs/contributing#changelog for an example.

[General] [Fixed] - Fix failing Switch E2E tests
Pull Request resolved: facebook#22698

Differential Revision: D13511917

Pulled By: hramos

fbshipit-source-id: b2f5cac1927531a855699e34e1a37036773f0aad
rozele pushed a commit to microsoft/react-native-windows that referenced this pull request Jul 18, 2019
Summary:
Switch E2E tests have been failing on master, although they pass locally.

Changelog:
----------

Help reviewers and the release process by writing your own changelog entry. See http://facebook.github.io/react-native/docs/contributing#changelog for an example.

[General] [Fixed] - Fix failing Switch E2E tests
Pull Request resolved: facebook/react-native#22698

Differential Revision: D13511917

Pulled By: hramos

fbshipit-source-id: b2f5cac1927531a855699e34e1a37036773f0aad
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Tests This PR adds or edits a test case.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants