Correct addGroup.feature and add new text cases to other group tests #31322
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.
Description
addGroup.feature
-group-id
->group_id
groupExists()
to correctly test for groups that have unusual characters in their name.#
?
%
in group names to all the places where groups are manipulated -addToGroup
deleteGroup
removeFromGroup
- for both API v1 and v2Now we have a full range of test cases for group names that do work properly with the existing Provisioning API v1 and v2. This will help to ensure there are no regressions when we fix the "slash in group name" case.
Motivation and Context
I wondered why the
addGroup.feature
test cases for group names containing a slash were passing. Because actually there is a problem with that - see issue #31015 and PR #31017I discovered that there was a typo in the column name of the examples table. Actually the scenario outline was running over-and-over and repeatedly testing adding a group with the literal name
<group_id>
instead of substituting in the group id in each example table row.When I make that fix, then a lot of the group creations fail, because the group should be URL-encoded when the group name needs to be sent in the URL (e.g. when checking if the group exists).
After fixing that, I still have to remove the entries from the examples table that have a slash in the group name, because those really do fail. The other PR addresses fixing that problem.
How Has This Been Tested?
Local acceptance test runs of group tests
Types of changes
Checklist: