-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Migrate select slider to playwright #8703
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
|
||
# Move mouse to 500, 0 pixels on the screen to simulate dragging left | ||
app.mouse.move(500, 0) | ||
app.mouse.up() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: This triggers a rerun or? maybe we add a wait_for_app_run(app)
here as well to make this more obvious. The current setup should be pretty save regarding flakiness, but we had a couple of cases where adding this also improved flakiness
e2e_playwright/st_select_slider.py
Outdated
def format_option(option): | ||
return f"Color: {option}" | ||
|
||
|
||
w1 = st.select_slider( | ||
"Label 1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: In many other widget tests we have been using the labels to give some indication of what the widgets is supposed to be tested for:
streamlit/e2e_playwright/st_text_input.py
Line 33 in 7f24fc0
v6 = st.text_input("text input 6 (disabled)", "default text", disabled=True) |
Describe your changes
GitHub Issue Link (if applicable)
Testing Plan
Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.