Skip to content

Conversation

@BalusC
Copy link
Member

@BalusC BalusC commented Aug 15, 2025

#1590 for eclipse-ee4j/mojarra#5601

Ran entire TCK. Only one test failed and the particular test explicitly checked presence of onchange attribute instead of actual outcome of the behavior. So improved it and added two more use cases.

Rest of TCK all green for eclipse-ee4j/mojarra#5601

@arjantijms arjantijms merged commit f8bd694 into fix_flaky_tests Aug 19, 2025
1 check passed
@BalusC
Copy link
Member Author

BalusC commented Aug 19, 2025

@arjantijms in the future please only merge PR when it targets a release branch, or leave merging up to the PR creator. This PR targeted "fix flaky tests" branch which was in turn not merged into a release branch yet. It should have been merged first. Because the "fix flaky tests" branch has been completed in a different branch by someone else, that branch is closed off. But as the current PR is already merged into there and thus never ended up in a release branch, I basically have to redo the current branch/PR instead of simply rebasing the branch/PR. That wouldn't have been necessary if it wasn't merged.

@BalusC
Copy link
Member Author

BalusC commented Sep 5, 2025

Superseded by #2055

@BalusC BalusC deleted the issue_1590_fix_failing_tests branch September 5, 2025 12:33
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.

4 participants