Skip to content

[build] Skip failing Firefox Beta tests#17234

Merged
cgoldberg merged 1 commit into
SeleniumHQ:trunkfrom
cgoldberg:skip-firefox-beta-failures
Mar 17, 2026
Merged

[build] Skip failing Firefox Beta tests#17234
cgoldberg merged 1 commit into
SeleniumHQ:trunkfrom
cgoldberg:skip-firefox-beta-failures

Conversation

@cgoldberg

Copy link
Copy Markdown
Member

💥 What does this PR do?

This PR adds the following Bazel targets to ./.skipped-tests:

-//java/test/org/openqa/selenium:AlertsTest-firefox-beta
-//java/test/org/openqa/selenium:FormHandlingTest-firefox-beta
-//java/test/org/openqa/selenium:WebScriptTest-firefox-beta
-//java/test/org/openqa/selenium/bidi/browser:BrowserCommandsTest-firefox-beta
-//java/test/org/openqa/selenium/bidi/browser:BrowserCommandsTest-firefox-beta-remote
-//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextInspectorTest-firefox-beta
-//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextInspectorTest-firefox-beta-remote
-//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextTest-firefox-beta
-//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextTest-firefox-beta-remote
-//rb/spec/integration/selenium/webdriver/bidi:browsing_context-firefox-beta-bidi

These are currently failing on RBE in trunk and on every PR branch.

💡 Additional Considerations

Someone needs to figure out if these failures are due to a legitimate bug in Firefox Beta that will be fixed, or if we need to fix something in selenium code to accommodate changes in the upcoming Firefox release. Once fixed in Firefox or on our end, these should be re-enabled.

🔄 Types of changes

  • Test

@qodo-code-review

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Skip failing Firefox Beta tests across Java and Ruby

🧪 Tests

Grey Divider

Walkthroughs

Description
• Skip 10 failing Firefox Beta tests in .skipped-tests
• Tests span Java and Ruby bindings across multiple modules
• Failures occur on RBE in trunk and PR branches
• Requires investigation into Firefox Beta compatibility issues

Grey Divider

File Changes

1. .skipped-tests ⚙️ Configuration changes +10/-0

Add Firefox Beta failing tests to skip list

• Added 10 Bazel test targets for Firefox Beta to skip list
• Includes Java tests for alerts, form handling, web scripts
• Includes BiDi browser and browsing context tests
• Includes Ruby BiDi browsing context integration test

.skipped-tests


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@cgoldberg cgoldberg merged commit 401f754 into SeleniumHQ:trunk Mar 17, 2026
17 checks passed
@cgoldberg cgoldberg deleted the skip-firefox-beta-failures branch March 17, 2026 22:31
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.

1 participant