Conversation
MultitenancyHarmoniaIT has failed on every master run since #7413: the home page's verification asserts a <span> containing "Dashboard" through assertElementExistsByTypeAndContainsText, which fails when more than one element matches - "Found [2] elements ... but expected at most one." The second element is intended. #7413 gave the dashboard a breadcrumb crumb of its own in place of the empty trail it used to return, so the label is now on the page twice: the sidebar entry that opens the page, and the breadcrumb of the page it opened. The product is right and the assertion is too strict for what it checks - that the tenant can reach its home page - so the test asks for at least one match instead of exactly one. assertAnyElementExistsByTypeAndContainsText is the tolerant twin of the existing assertion, over the same frame-walking search with its "expected at most one" guard lifted; the strict method is unchanged, and it stays the default for anything that must be unique. Verified: MultitenancyHarmoniaIT green locally on H2 (3/3, the two failing tests among them), formatter:validate green with the cache wiped. Not covered here: BPMStarterTemplateIT timed out in the ui legs of run 35223726746 but passed at the same SHA in 35192425278, and the PostgreSQL slow leg's Liquibase "Could not acquire change log lock" cascade appeared once and not in the next run - both look like flakes rather than this break. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
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.
The break
MultitenancyHarmoniaIThas failed on every master run since #7413, in both the H2 and the PostgreSQLslowlegs — first seen in run 35192425278, still failing in the latest completed run:The second element is intended. #7413 made
breadcrumbTrailreturn aDashboardcrumb where it used to return an empty trail, so the label is on the home page twice: the sidebar entry that opens the page, and the breadcrumb of the page it opened.assertElementExistsByTypeAndContainsTextfails on more than one match, so the test broke on a correct product change.The change
The assertion checks that the tenant can reach its home page, which more than one match does not disprove — so the test asks for at least one match.
assertAnyElementExistsByTypeAndContainsTextis the tolerant twin of the existing assertion: the same frame-walking search with the "expected at most one" guard lifted, failing with a screenshot when nothing matches. The strict method is untouched and stays the default for anything that must be unique.Verified
MultitenancyHarmoniaITgreen locally on H2:Tests run: 3, Failures: 0, Errors: 0(the two tests that fail on master among them).mvn -T 1C formatter:validategreen with the formatter cache wiped first.Not this break
BPMStarterTemplateITtimed out in theuilegs of run 35223726746 but passed at the same SHA in 35192425278 — a flake, not addressed here.slowleg also hit a LiquibaseCould not acquire change log lockcascade in 35192425278 (which is what pushed that job past its 1h30m limit); it did not recur in the next run at the same SHA.🤖 Generated with Claude Code