Skip to content

Improve "Selenium2Driver::applyTimeouts" error handling for Selenium 3 support - #380

Merged
aik099 merged 1 commit into
minkphp:masterfrom
aik099:selenium3-timeout-fix
Jun 28, 2024
Merged

Improve "Selenium2Driver::applyTimeouts" error handling for Selenium 3 support#380
aik099 merged 1 commit into
minkphp:masterfrom
aik099:selenium3-timeout-fix

Conversation

@aik099

@aik099 aik099 commented Feb 18, 2024

Copy link
Copy Markdown
Member

Closes #379

Added code remains uncovered by tests because we're testing on Selenium 2 only.

@codecov

codecov Bot commented Feb 18, 2024

Copy link
Copy Markdown

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (4ca4083) 90.19% compared to head (2fe6b36) 89.80%.

Files Patch % Lines
src/Selenium2Driver.php 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #380      +/-   ##
============================================
- Coverage     90.19%   89.80%   -0.39%     
- Complexity      168      169       +1     
============================================
  Files             1        1              
  Lines           469      471       +2     
============================================
  Hits            423      423              
- Misses           46       48       +2     

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

@aik099 aik099 mentioned this pull request Feb 23, 2024
@aik099
aik099 requested a review from stof March 16, 2024 19:51
@aik099
aik099 merged commit 897e7d7 into minkphp:master Jun 28, 2024
@aik099
aik099 deleted the selenium3-timeout-fix branch June 28, 2024 09:18
@aik099

aik099 commented Jun 28, 2024

Copy link
Copy Markdown
Member Author

Thank you @stof . Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The "TimeoutTest::testInvalidTimeoutSettingThrowsException" test fails on Selenium 3.x

2 participants