We should have integration tests that use the Engine MockServer to verify behaviour when engine requests fail in the context of user/group syncing.
- Integration tests for when the user/group sync fails. We should verify that role cleanup doesn't take place in this scenario and also test the backing off retry logic
- When a user hasn't been imported during sync, optimize can directly fetch the user from the engine. We should test for when that fetch from the engine fails.
If you have any questions about setting expectations and verifying requests with MockServer, ping Joshua Windels. The documentation is also pretty good https://www.mock-server.com/mock_server/creating_expectations.html and https://www.mock-server.com/mock_server/verification.html