selenium
selenium copied to clipboard
SeleniumHQ
[dotnet][rb][java][js][py] Automated Browser Version Update
This is an automated pull request to update pinned browsers and drivers
Merge after verify the new browser versions properly passing the tests and no bugs need to be filed
PR Description updated to latest commit (https://github.com/SeleniumHQ/selenium/commit/4eb007f18854ec0bb50818e164ccaac54f553b83)
- [ ] Copy walkthrough table to "Files Changed" Tab
![qodo-code-review[bot] avatar](https://avatars.githubusercontent.com/in/484649?v=4 "Published by a pub.dev verified publisher"){kind=small}
PR Review 🔍
| ⏱️ Estimated effort to review [1-5] |
2, because the changes are straightforward and involve only updates to URLs and hashes for browser versions. The structure and logic of the code remain unchanged. |
| 🧪 Relevant tests |
No |
| ⚡ Possible issues |
Possible Bug: Ensure that the new browser versions and drivers are compatible with existing tests and functionalities. This should be verified as suggested in the PR description. |
| 🔒 Security concerns |
No |
PR Code Suggestions ✨
| Category | Suggestion | Score |
| Enhancement |
Implement automated verification of URLs for browser archives to ensure their availability and correctnessConsider adding a verification step to check the availability and correctness of the URLs Suggestion importance[1-10]: 7Why: Adding automated verification for URLs in a CI/CD pipeline is a valuable enhancement for maintaining the reliability and correctness of external dependencies. | 7 |
| Possible issue |
Verify the integrity and correctness of the URL for the
| 3 |
| Security |
Re-verify the
| 3 |
CI Failure Feedback 🧐
(Checks updated until commit https://github.com/SeleniumHQ/selenium/commit/72a45e4b7c1dc596e9f16ad55e250f6a7f70e51c)
|
Action: Ruby / Local Tests (edge, windows) / Local Tests (edge, windows) |
|
Failed stage: Run Bazel [❌] |
|
Failed test name: Selenium::WebDriver::BiDi |
|
Failure summary: The action failed because the test Selenium::WebDriver::BiDi gets session status failed because it was expected to raise an error due to a pending test guard, but no error was raised. Selenium::WebDriver::BiDi does not close BiDi session if at least one window is opened failed because it was expected to raise an error due to a pending test guard, but no error was raised. Selenium::WebDriver::BiDi closes BiDi session if last window is closed failed because it was expected to raise an error due to a pending test guard, but no error was raised. |
Relevant error logs: |
✨ CI feedback usage guide:
The CI feedback tool (/checks) automatically triggers when a PR has a failed check.
The tool analyzes the failed checks and provides several feedbacks:
- Failed stage
- Failed test name
- Failure summary
- Relevant error logs
In addition to being automatically triggered, the tool can also be invoked manually by commenting on a PR:
/checks "https://github.com/{repo_name}/actions/runs/{run_number}/job/{job_number}"
where {repo_name} is the name of the repository, {run_number} is the run number of the failed check, and {job_number} is the job number of the failed check.
Configuration options
-
enable_auto_checks_feedback- if set to true, the tool will automatically provide feedback when a check is failed. Default is true. -
excluded_checks_list- a list of checks to exclude from the feedback, for example: ["check1", "check2"]. Default is an empty list. -
enable_help_text- if set to true, the tool will provide a help message with the feedback. Default is true. -
persistent_comment- if set to true, the tool will overwrite a previous checks comment with the new feedback. Default is true. -
final_update_message- ifpersistent_commentis true and updating a previous checks message, the tool will also create a new message: "Persistent checks updated to latest commit". Default is true.
See more information about the checks tool in the docs.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 58.72%. Comparing base (
57f8398) to head (2d6744b). Report is 355 commits behind head on trunk.
:exclamation: Current head 2d6744b differs from pull request most recent head 72a45e4
Please upload reports for the commit 72a45e4 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## trunk #13942 +/- ##
==========================================
+ Coverage 58.48% 58.72% +0.23%
==========================================
Files 86 86
Lines 5270 5298 +28
Branches 220 227 +7
==========================================
+ Hits 3082 3111 +29
+ Misses 1968 1960 -8
- Partials 220 227 +7
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
![codecov[bot] avatar](https://avatars.githubusercontent.com/in/254?v=4 "Published by a pub.dev verified publisher"){kind=small}