Re: [I] Add ScimGroup tests for PatchGenerator, (previously)UpdateRequest [directory-scimple]

["joshuapsteele (via GitHub)" <gi...@apache.org>]

joshuapsteele commented on issue #427:
URL: https://github.com/apache/directory-scimple/issues/427#issuecomment-1840934004

   @bdemers I'm having issues getting the `GroupIT` tests (or any ITs, for that matter) to run locally. `./mvnw package` seems to skip right over them, unless I've missed something! (Perhaps this merits another issue.)
   
   Anyway, the important point in this thread is that we need a test that covers the following PATCH operation to remove a user from a group. Instead of specifying the user ID in the `path` (as is currently covered in the tests), Azure, for example, specifies the user ID in the `value` like so:
   
   ```
       String patchBody = "{" +
         "\"schemas\": [\"urn:ietf:params:scim:api:messages:2.0:PatchOp\"]," +
         "\"Operations\": [{" +
         "\"op\": \"remove\"," +
         "\"path\": \"members\"," +
         "\"value\": [{\"value\": \"" + userId + "\"}]" +
         "}]}";
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@directory.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@directory.apache.org
For additional commands, e-mail: dev-help@directory.apache.org