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

Makefile: Update Cockpit lib to 140b7e62f5cfcad5ad9105867474526ef68351db #1484

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 16, 2023

@github-actions github-actions bot added the bot label Nov 16, 2023
@github-actions github-actions bot changed the title [no-test] Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Nov 16, 2023
@martinpitt martinpitt force-pushed the cockpit-lib-update-cockpit-lib-20231116-022509 branch from 810f5df to 49c9cf7 Compare November 16, 2023 02:25
@martinpitt martinpitt self-assigned this Nov 16, 2023
@martinpitt
Copy link
Member

Test shows multiple dialogs on top of each other, see cockpit-project/cockpit#19595 . Let's fix!

martinpitt and others added 2 commits November 16, 2023 10:31
Close the error dialog in expectSearchErrorForNotExistingImage() and the
image create dialog at the end of _testBasic() before clicking on the
main page to open a new one. That is cheating, and also a race
condition.

Also wait for the successful download dialog to close before moving on
to the next one.
@martinpitt martinpitt force-pushed the cockpit-lib-update-cockpit-lib-20231116-022509 branch from 49c9cf7 to 9cc6ce7 Compare November 16, 2023 09:32
@martinpitt martinpitt changed the title Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Makefile: Update Cockpit lib to 140b7e62f5cfcad5ad9105867474526ef68351db Nov 16, 2023
Copy link
Member

@jelly jelly left a comment

Choose a reason for hiding this comment

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

Thanks!

@jelly jelly merged commit 2f0cbc6 into main Nov 16, 2023
33 checks passed
@jelly jelly deleted the cockpit-lib-update-cockpit-lib-20231116-022509 branch November 16, 2023 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants