fix: fail on request errors for triggering retry

package.json

@@ -135,7 +135,7 @@
     "operator:harbor": "node dist/operator/harbor.js",
     "operator:keycloak-dev": "NODE_TLS_REJECT_UNAUTHORIZED=0 ts-node-dev ./src/operator/keycloak.ts",
     "operator:keycloak": "node dist/operator/keycloak.js",
-    "test": "NODE_ENV=test mocha -r ts-node/register -r ts-custom-error --exit src/**/*.test.*"
+    "test": "NODE_ENV=test mocha -r ts-node/register -r ts-custom-error --exit src/*.test.* src/**/*.test.*"
   },
   "standard-version": {
     "skip": {

src/operator/keycloak.test.ts

@@ -1,6 +1,6 @@
 import { expect } from 'chai'
 import sinon from 'sinon'
-import { addUserGroups, removeUserGroups } from './keycloak'
+import { updateUserGroups } from './keycloak'
 
 describe('Keycloak User Group Management', () => {
   let api: any
@@ -28,13 +28,17 @@ describe('Keycloak User Group Management', () => {
 
   describe('removeUserGroups', () => {
     it('should remove user from groups not in teamGroups', async () => {
+      const groupsById = {
+        group1: 'group1-id',
+        group2: 'group2-id',
+      }
       const existingUserGroups = [
         { name: 'group1', id: 'group1-id' },
         { name: 'group2', id: 'group2-id' },
       ]
       api.users.realmUsersIdGroupsGet.resolves({ body: existingUserGroups })
 
-      await removeUserGroups(api, existingUser, ['group1'])
+      await updateUserGroups(api, existingUser, groupsById, ['group1'])
 
       expect(api.users.realmUsersIdGroupsGroupIdDelete.calledWith(keycloakRealm, 'user-id', 'group2-id')).to.be.true
       expect(api.users.realmUsersIdGroupsGroupIdDelete.calledWith(keycloakRealm, 'user-id', 'group1-id')).to.be.false
@@ -43,15 +47,14 @@ describe('Keycloak User Group Management', () => {
 
   describe('addUserGroups', () => {
     it('should add user to groups in teamGroups if not already present', async () => {
-      const currentKeycloakGroups = [
-        { name: 'group1', id: 'group1-id' },
-        { name: 'group2', id: 'group2-id' },
-      ]
+      const groupsById = {
+        group1: 'group1-id',
+        group2: 'group2-id',
+      }
       const existingUserGroups = [{ name: 'group1', id: 'group1-id' }]
-      api.groups.realmGroupsGet.resolves({ body: currentKeycloakGroups })
       api.users.realmUsersIdGroupsGet.resolves({ body: existingUserGroups })
 
-      await addUserGroups(api, existingUser, ['group1', 'group2'])
+      await updateUserGroups(api, existingUser, groupsById, ['group1', 'group2'])
 
       expect(api.users.realmUsersIdGroupsGroupIdPut.calledWith(keycloakRealm, 'user-id', 'group2-id')).to.be.true
       expect(api.users.realmUsersIdGroupsGroupIdPut.calledWith(keycloakRealm, 'user-id', 'group1-id')).to.be.false