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 condition in PF4 progressbar #1448

Merged

Conversation

dosas
Copy link
Collaborator

@dosas dosas commented Jun 26, 2024

when the progress bar disappears between the
is_displayed query and the current_progress query
a NoSuchElement exception is raised

@dosas dosas requested a review from sambible June 26, 2024 12:46
@dosas dosas added CherryPick PR needs CherryPick to previous branches 6.15.z labels Jun 26, 2024
@omkarkhatavkar
Copy link
Collaborator

Can one of the admins verify this patch?

airgun/widgets.py Outdated Show resolved Hide resolved
@dosas dosas force-pushed the fix-progress-bar-race-condition branch from 913a897 to 6653c45 Compare July 4, 2024 11:53
@dosas dosas requested a review from omkarkhatavkar July 4, 2024 11:54
airgun/widgets.py Outdated Show resolved Hide resolved
when the progress bar disappears between the
is_displayed query and the current_progress query
a NoSuchElement exception is raised
@dosas dosas force-pushed the fix-progress-bar-race-condition branch from 6653c45 to 452139b Compare July 4, 2024 12:33
@omkarkhatavkar
Copy link
Collaborator

@dosas Do you know any test that I can confirm for this feature?

@dosas
Copy link
Collaborator Author

dosas commented Jul 16, 2024

@dosas Do you know any test that I can confirm for this feature?

It is used when publishing 'new' content views so should be tests/foreman/ui/test_contenview.py

@vijaysawant
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k test_file_cv_display

@vijaysawant
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py

@omkarkhatavkar
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k test_file_cv_display

Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

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

Good change. I've noticed this when working with CV Publish modal, so this will be nice for CV tests going forward.

@sambible sambible merged commit 1661b6c into SatelliteQE:master Jul 23, 2024
7 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 23, 2024
when the progress bar disappears between the
is_displayed query and the current_progress query
a NoSuchElement exception is raised

(cherry picked from commit 1661b6c)
dosas added a commit to ATIX-AG/airgun that referenced this pull request Jul 31, 2024
when the progress bar disappears between the
is_displayed query and the current_progress query
a NoSuchElement exception is raised

(cherry picked from commit 1661b6c)
ogajduse pushed a commit that referenced this pull request Aug 16, 2024
when the progress bar disappears between the
is_displayed query and the current_progress query
a NoSuchElement exception is raised

(cherry picked from commit 1661b6c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants