Skip to content

Conversation

@cgoldberg
Copy link
Member

@cgoldberg cgoldberg commented May 25, 2025

User description

💥 What does this PR do?

This PR increases the timeout to wait for devtools console messages in py/test/selenium/webdriver/common/devtools_tests.py because it keeps failing on RBE.

🔄 Types of changes

  • Internal test infrastructure

PR Type

Tests


Description

  • Increased timeout for devtools console message test

  • Aims to reduce test failures on RBE infrastructure


Changes walkthrough 📝

Relevant files
Tests
devtools_tests.py
Increase WebDriverWait timeout in devtools test                   

py/test/selenium/webdriver/common/devtools_tests.py

  • Increased WebDriverWait timeout from 5 to 10 seconds in console
    message test
  • Addresses intermittent test failures due to timing issues
  • +1/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @qodo-merge-pro
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 PR contains tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @selenium-ci selenium-ci added the C-py Python Bindings label May 25, 2025
    @qodo-merge-pro
    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

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

    Labels

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants