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: Work around flakiness loading proposals #3342

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Conversation

dskloetd
Copy link
Contributor

Motivation

We've seen flakiness with e2e/neurons.spec.ts for a while. I think I finally figured out what happens.

The NnsProposals component loads neurons and proposals at the same time.
But once neurons are loaded, it loads proposals again. So it's possible
that the component goes back into loading state immediately after
proposals are loaded.

I was able to reproduce this by adding an artificial delay in listNeurons. Then you consistently see the skeletons reappear after the disappear.

Unfortunately the component is very complicated so it's not easy to fix without a major cleanup.
So until we have time to clean this up, we should work around the issue to avoid flakiness with the test.

Changes

Once proposals are loaded, wait 1 second and wait again for proposals to be loaded if necessary.

Tests

Still pass. Hopefully less flaky in the future.

Todos

  • Add entry to changelog (if necessary).
    not necessary

@dskloetd dskloetd marked this pull request as ready for review September 21, 2023 08:00
@dskloetd dskloetd requested a review from lmuntaner September 21, 2023 08:00
Copy link
Contributor

@lmuntaner lmuntaner left a comment

Choose a reason for hiding this comment

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

I agree with trying this. If this fixes it, then we can spend time refactoring NnsProposals.

@dskloetd dskloetd added this pull request to the merge queue Sep 21, 2023
Merged via the queue into main with commit a84933f Sep 21, 2023
38 checks passed
@dskloetd dskloetd deleted the kloet/e2e-neurons branch September 21, 2023 09: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.

2 participants