Skip to content

fix(java): wait for footer by locator#143

Merged
jsugg merged 1 commit into
masterfrom
agent/fix-grid-edge-footer-wait
Jul 17, 2026
Merged

fix(java): wait for footer by locator#143
jsugg merged 1 commit into
masterfrom
agent/fix-grid-edge-footer-wait

Conversation

@jsugg

@jsugg jsugg commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Poll the footer locator inside the explicit wait instead of resolving it eagerly.
  • Add a deterministic delayed-footer regression test to the Java regression suite.

Root cause

The previous page-open condition called driver.findElement while constructing the expected condition. A transiently absent footer on Selenium Grid Edge therefore failed before WebDriverWait could poll.

Validation

  • Java 25 container: delayed-footer regression test passed.
  • Local Selenium Grid Edge: delayed-footer and WindowsTest.validateNewTab passed (2 tests, 0 failures).
  • python3 tools/check-scenarios.py --write-matrix passed with no generated diff.

Avoid eager footer lookup before explicit-wait polling on Grid Edge.
@jsugg
jsugg marked this pull request as ready for review July 17, 2026 01:58
@jsugg
jsugg merged commit 5b1e6a7 into master Jul 17, 2026
7 checks passed
@jsugg
jsugg deleted the agent/fix-grid-edge-footer-wait branch July 17, 2026 01:59
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