[rb] fix select being able to select options hidden by css rules

[FFederi]

🔗 Related Issues

Related to issue: [🚀 Feature]: Unifying Select Class Across All Bindings

💥 What does this PR do?

Modifies select behaviour to make it the same as python and Java bindings.
In particular, makes it so selecting an option hidden by CSS rules raises an exception.

🔧 Implementation Notes

I couldn't find (or understand) a general way to check visibility of an element, so I added a simple check on CSS values like it's been done with Java and python.

🔄 Types of changes

• Bug fix (backwards compatible)

[qodo-code-review]

PR Type

Bug fix

---

Description

• Prevents selecting/deselecting options hidden by CSS rules

• Adds visibility check using CSS properties (visibility, display, opacity)

• Raises ElementNotInteractableError for invisible options

• Aligns Ruby bindings behavior with Python and Java implementations

---

File Walkthrough

	Relevant files
Bug fix	select.rb Add visibility checks to select/deselect operations            rb/lib/selenium/webdriver/support/select.rb Added css_property_and_visible? helper method to check element visibility Modified select_option to raise ElementNotInteractableError for invisible options Modified deselect_option to raise ElementNotInteractableError for invisible options Checks CSS properties (visibility, display, opacity) against hidden values +19/-0
Tests	select_spec.rb Add tests for invisible option selection behavior                rb/spec/integration/selenium/webdriver/select_spec.rb Added multi_invisible fixture for testing invisible select elements Added test case for selecting invisible options by text Added test case for deselecting invisible options by text Both test cases verify ElementNotInteractableError is raised +17/-0

---

[selenium-ci]

Thank you, @FFederi for this code suggestion.

The support packages contain example code that many users find helpful, but they do not necessarily represent
the best practices for using Selenium, and the Selenium team is not currently merging changes to them.

After reviewing the change, unless it is a critical fix or a feature that is needed for Selenium
to work, we will likely close the PR.

We actively encourage people to add the wrapper and helper code that makes sense for them to their own frameworks.
If you have any questions, please contact us

[qodo-code-review]

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🟡	🎫 #15265 🟢 Make `Select` class behavior consistent across language bindings (tracking item: Ruby). Ensure selection by visible text (and "contains visible text" where applicable) does not allow selecting options hidden by CSS, considering the CSS properties visibility, display, and opacity and the values hidden, none, 0, 0.0. ⚪ Confirm that Ruby’s "contains visible text" selection path (if supported in this binding/version) also routes through the same option selection logic and therefore raises the same exception for CSS-hidden options in real browser runs.
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
🟢	Generic: Comprehensive Audit Trails Objective: To create a detailed and reliable record of critical system actions for security analysis and compliance. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Error Handling Objective: To prevent the leakage of sensitive system information through error messages while providing sufficient detail for internal debugging. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Logging Practices Objective: To ensure logs are useful for debugging and auditing without exposing sensitive information like PII, PHI, or cardholder data. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Security-First Input Validation and Data Handling Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent vulnerabilities Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
⚪	Generic: Meaningful Naming and Self-Documenting Code Objective: Ensure all identifiers clearly express their purpose and intent, making code self-documenting Status: Potentially misleading name: The method css_property_and_visible? implies a full visibility check but only tests a small set of CSS property values, which may confuse readers about its actual behavior. Referred Code def css_property_and_visible?(element) css_value_candidates = %w[hidden none 0 0.0].to_set css_property_candidates = %w[visibility display opacity] css_property_candidates.none? do |property| css_value_candidates.include?(element.css_value(property)) end end Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Robust Error Handling and Edge Case Management Objective: Ensure comprehensive error handling that provides meaningful context and graceful degradation Status: Incomplete visibility checks: The new CSS-based invisibility detection in css_property_and_visible? may miss edge cases (e.g., other opacity string formats, inherited styles, or other visibility-affecting CSS values), potentially allowing still-non-interactable options through or blocking valid ones. Referred Code def css_property_and_visible?(element) css_value_candidates = %w[hidden none 0 0.0].to_set css_property_candidates = %w[visibility display opacity] css_property_candidates.none? do |property| css_value_candidates.include?(element.css_value(property)) end end Learn more about managing compliance generic rules or creating your own custom rules

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Improve visibility check for correctness Improve the css_property_and_visible? method by checking each CSS property (display, visibility, opacity) against its specific "hiding" value for a more correct and robust visibility check. rb/lib/selenium/webdriver/support/select.rb [280-287] def css_property_and_visible?(element) - css_value_candidates = %w[hidden none 0 0.0].to_set - css_property_candidates = %w[visibility display opacity] + return false if element.css_value('display') == 'none' + return false if element.css_value('visibility') == 'hidden' + return false if element.css_value('opacity').to_f.zero? - css_property_candidates.none? do |property| - css_value_candidates.include?(element.css_value(property)) - end + true end Apply / Chat Suggestion importance[1-10]: 8 __ Why: The suggestion correctly identifies that the visibility check logic is flawed by using a single set of values for different CSS properties, and proposes a more robust and correct implementation.	Medium
More