Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XWIKI-22680: Regression on createAndDeleteUser #3683

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

Sereza7
Copy link
Contributor

@Sereza7 Sereza7 commented Nov 25, 2024

Jira URL

https://jira.xwiki.org/browse/XWIKI-22680

Fix for the CI regressions: https://ci.xwiki.org/job/XWiki/job/xwiki-platform/job/stable-16.10.x/9/testReport/junit/org.xwiki.invitation.test.docker/AllIT$NestedInvitationIT/Platform_Builds___main___integration_tests___IT_for_xwiki_platform_core_xwiki_platform_invitation_xwiki_platform_invitation_test_xwiki_platform_invitation_test_docker___Build_for_IT_for_xwiki_platform_core_xwiki_platform_invitation_xwiki_platform_invitation_test_xwiki_platform_invitation_test_docker___nonAdminCanSend_TestUtils_/ and https://ci.xwiki.org/job/XWiki/job/xwiki-platform/job/stable-16.10.x/lastCompletedBuild/testReport/org.xwiki.invitation.test.docker/AllIT$NestedInvitationIT/Platform_Builds___main___integration_tests___IT_for_xwiki_platform_core_xwiki_platform_invitation_xwiki_platform_invitation_test_xwiki_platform_invitation_test_docker___Build_for_IT_for_xwiki_platform_core_xwiki_platform_invitation_xwiki_platform_invitation_test_xwiki_platform_invitation_test_docker___guestActionsOnNonexistantMessage_TestUtils_/

Changes

Description

  • Removed the waiting for forms that do not use livevalidation.

Clarifications

  • I failed to consider those forms before. Thanks to luck, the conditions to wait did not influence the invitation form. However, with the latest merge, the condition were not enough. In order to solve this, I used the presence of a validation message to check if the form relied on the livevalidation script. Without this validation message, there is no need for a wait because there's nothing stopping the form from being submitted.

Screenshots & Video

None, test only change.

Executed Tests

After building the changes with mvn clean install -f xwiki-platform-core/xwiki-platform-test/xwiki-platform-test-ui, I could pass the tests locally: mvn clean install -f xwiki-platform-core/xwiki-platform-invitation/xwiki-platform-invitation-test/xwiki-platform-invitation-test-docker -Dit.test=InvitationIT#guestActionsOnNonexistantMessage and mvn clean install -f xwiki-platform-core/xwiki-platform-administration/xwiki-platform-administration-test/xwiki-platform-administration-test-docker -Dit.test=RegisterIT#registerExistingUser.

Expected merging strategy

* Removed the waiting for forms that do not use livevalidation.
@surli surli merged commit 777a296 into xwiki:master Nov 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants