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 flaky Cypress tests #265

Merged
merged 4 commits into from
Nov 12, 2019
Merged

Fix flaky Cypress tests #265

merged 4 commits into from
Nov 12, 2019

Conversation

Azer0s
Copy link
Collaborator

@Azer0s Azer0s commented Sep 24, 2019

Description

There were some timing problems with Cypress while running in CI which didn't occur while running locally (probably due to the CI instance being slower than our local machines). I fixed this by using the built-in Cypress timeouts.

@Azer0s Azer0s requested review from kevinbader and mmacai September 24, 2019 13:06
@kevinbader
Copy link
Contributor

As discussed out of band, I'd prefer this over the current naming:

cy.assertEventReceived(<event object>)
cy.assertEventReceivedWithMessage(<text message>)

For this to work, I'd expect event-log and eventList to use the same name (i.e., one of them needs to be renamed).

@mmacai mmacai merged commit ea94d59 into master Nov 12, 2019
@mmacai mmacai deleted the fix/flaky-tests branch November 12, 2019 13:57
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