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

Android blacboxtests fix, workaround for Python webdriver #2140

Merged
merged 1 commit into from
Dec 29, 2023

Conversation

kaidokert
Copy link
Member

@kaidokert kaidokert commented Dec 29, 2023

This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

@kaidokert kaidokert force-pushed the android_tests branch 2 times, most recently from 3d79e55 to 926ae93 Compare December 29, 2023 18:19
@kaidokert kaidokert changed the title Android tests Android blacboxtests fix, workaround for Python webdriver Dec 29, 2023
This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704
@kaidokert kaidokert marked this pull request as ready for review December 29, 2023 18:25
Copy link

codecov bot commented Dec 29, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (980c2a5) 58.54% compared to head (6bd8925) 58.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2140      +/-   ##
==========================================
+ Coverage   58.54%   58.56%   +0.02%     
==========================================
  Files        1912     1912              
  Lines       94578    94578              
==========================================
+ Hits        55370    55392      +22     
+ Misses      39208    39186      -22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kaidokert kaidokert merged commit 0f56fd5 into youtube:main Dec 29, 2023
374 checks passed
@kaidokert kaidokert added cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch labels Jan 2, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Jan 2, 2024
This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

(cherry picked from commit 0f56fd5)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Jan 2, 2024
This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

(cherry picked from commit 0f56fd5)
kaidokert added a commit that referenced this pull request Jan 2, 2024
This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

(cherry picked from commit 0f56fd5)
kaidokert added a commit that referenced this pull request Jan 2, 2024
This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

(cherry picked from commit 0f56fd5)
kaidokert added a commit that referenced this pull request Jan 2, 2024
… webdriver (#2150)

Refer to the original PR: #2140

This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

Co-authored-by: Kaido Kert <[email protected]>
kaidokert added a commit that referenced this pull request Jan 3, 2024
… webdriver (#2151)

Refer to the original PR: #2140

This makes sure that Webdriver Remote connection is initialized
with a timeout value before the timeout value gets used. Avoids
errors coming from incompatible urrlib/request/webdriver packages
using different global timeout values.

b/318024704

Co-authored-by: Kaido Kert <[email protected]>
@kaidokert kaidokert deleted the android_tests branch June 30, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch on_device
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants