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

Flaky test: unit_tests/gui/test_main_window::test_that_run_dialog_can_be_closed_after_used_to_open_plots #7071

Closed
jonathan-eq opened this issue Feb 1, 2024 · 1 comment · Fixed by #7077
Assignees
Labels

Comments

@jonathan-eq
Copy link
Contributor

jonathan-eq commented Feb 1, 2024

Test is timing out when waiting for run_dialog.done_button to be enabled. The button is enabled when the worker in the QThread is finished, but this is sometimes not done. I am not sure why the thread does not finish, maybe it just needs longer timeout?

            run_dialog = wait_for_child(gui, qtbot, RunDialog)
    
            # Ensure that once the run dialog is opened
            # another simulation cannot be started
            assert not start_simulation.isEnabled()
    
            # Change simulation mode and ensure that
            # another experiment still cannot be started
            for ind in range(simulation_mode.count()):
                simulation_mode.setCurrentIndex(ind)
                assert not start_simulation.isEnabled()
    
            # The user expects to be able to open e.g. the even viewer
            # while the run dialog is open
            assert not run_dialog.isModal()
    
            qtbot.mouseClick(run_dialog.plot_button, Qt.LeftButton)
>           qtbot.waitUntil(run_dialog.done_button.isVisible, timeout=20000)
E           pytestqt.exceptions.TimeoutError: waitUntil timed out in 20000 milliseconds

To reproduce

Run tests in parallell.
I used command pytest -x tests/unit_tests/gui/test_main_window.py -k plot -n 12 --count 50

Environment

  • OS: macOS
  • ERT/Komodo release: main ERT
  • Python version: 3.10
  • Remote/HPC execution involved: no
@jonathan-eq jonathan-eq added the bug label Feb 1, 2024
@jonathan-eq
Copy link
Contributor Author

The run_esmda fixture has a timeout of 200000, but we only have 20000. Maybe increasing this would help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant