Repository navigation
Add UI Testing Playground coverage - #4
Merged
Merged
Conversation
Round 6 diagnostics showed the real mechanics: Hidden Layers stacks a second "blueButton" layer on top of the still-present greenButton rather than replacing it, so the fix checks for the new layer instead of an id swap. Mouse Over's link keeps the same DOM node but swaps its class from text-primary to text-warning on hover, which broke the previous class-based locator; now located by its stable text instead. Sample App logs in correctly with "pwd" when tested directly, so the retry flow now re-enters the username as well as the password in case a failed attempt clears the form. Diagnoses Overlapped Element's remaining failure (JS value-setting returns empty immediately after a synchronous set+dispatch) at the correct 1920x1080 window size the real suite uses, comparing native input, JS set with/without dispatch, and the React native-setter trick, since two earlier guessed fixes for this one have both failed.
Round 7 diagnostics showed the field has no input/change listeners at all: a plain JavaScript property assignment (confirmed via Selenium's own attribute read immediately after) sets it reliably, while the "native setter" trick that's usually needed for React inputs actually broke it here, and native clicks/keystrokes fail because a permanently positioned element covers the field regardless of window size, not because of a narrow-viewport responsive issue. Also fixes the "#name" selector variable being silently swallowed as a Robot Framework comment (any cell starting with # is treated as one, regardless of section) by qualifying it with the input tag. Reverts the CI workflow's playground job back to the real suite and removes the temporary diagnostic file now that all 29 tests are green.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
robot --dryrunpasses locally for the full suiterobocop check/robocop format --checkcleantests/playground