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 race conditions / timeouts in app test #116

Open
mccalluc opened this issue Nov 1, 2024 · 0 comments
Open

Fix race conditions / timeouts in app test #116

mccalluc opened this issue Nov 1, 2024 · 0 comments
Labels
nice-to-have No discussion required, but not top priority

Comments

@mccalluc
Copy link
Contributor

mccalluc commented Nov 1, 2024

We could up the timeout... but that will mean a stall when the element really isn't there. This might also reflect a real usability problem: Users could stack up actions and the app would seem to be unresponsive.

  • Maybe we should lock controls until the last action is resolved?
  • Or maybe there is a way for higher priority actions to override lower priority?
       # -- Download results --
       page.get_by_role("button", name="Download results").click()
       expect_not_visible(pick_dataset_text)
>       expect_not_visible(perform_analysis_text)

Looking at the trace, it looks like it might have gotten bogged down in recalculating the >graphs and didn't respond to the mouseclick to move to the next page quickly enough. >Is there a risk of the application being unresponsive?

Originally posted by @mccalluc in #103 (comment)

@github-project-automation github-project-automation bot moved this to Pending in DP Wizard Nov 1, 2024
@mccalluc mccalluc added the nice-to-have No discussion required, but not top priority label Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nice-to-have No discussion required, but not top priority
Projects
Status: Pending
Development

No branches or pull requests

1 participant