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

[uSvFQccG] Make ExportCypherTest more robust. #481

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

Lojjs
Copy link
Contributor

@Lojjs Lojjs commented Aug 28, 2023

Things like the order in UNWIND lists and the exact ids could vary, which is not a problem for the functionality but cause flakiness of the test.

I focused on the test that was marked as flaky, but this could be done for more of the export tests as a follow-up cleanup/remove risk of flakiness task if we like to.

Things like the order in UNWIND lists and the exact ids could vary, which is not a problem for the functionality but cause flakiness of the test.
Copy link
Contributor

@gem-neo4j gem-neo4j left a comment

Choose a reason for hiding this comment

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

I think this is a good way to deal with it :) Maybe make a cleanup card somewhere in APOC trello that points here so if we do get a flaky test card for another area we have links pointing to this solution?

Another thought, and I know this wasn't to do with this PR, but I find tests naming ids of github issues hard to understand, would be nice if we either had a comment explaining the original issue or a more explanatory name, nothing for you to do here, just a rant :P

@Lojjs Lojjs merged commit 6e4bf9e into dev Aug 29, 2023
11 of 12 checks passed
@Lojjs Lojjs deleted the dev-export-cypher-flaky branch August 29, 2023 07:14
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.

2 participants