fix: Added usernameInIdp property to update user schema (#292)

This change fixes authentication of users who are added by admin from ServiceWorkbench UI directly

addons/addon-base-raas/packages/base-raas-services/lib/schema/update-user.json

@@ -10,6 +10,12 @@
       "type": "string",
       "pattern": "^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\\.[a-zA-Z0-9-]+)*$"
     },
+    "usernameInIdp": {
+      "type": "string",
+      "minLength": 3,
+      "maxLength": 300,
+      "pattern": "^[A-Za-z0-9-_.]+$|^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\\.[a-zA-Z0-9-]+)*$"
+    },
     "firstName": {
       "type": "string",
       "maxLength": 500

addons/addon-base-raas/packages/base-raas-services/lib/user/__tests__/user-service.test.js

@@ -69,6 +69,56 @@ describe('UserService', () => {
     service.assertAuthorized = jest.fn();
   });
 
+  describe('updateUser', () => {
+    it('should not fail validation when usernameInIdp in schema', async () => {
+      const user = {
+        email: '[email protected]',
+        usernameInIdp: 'example',
+        uid: 'user1',
+        rev: 0,
+      };
+
+      // mocked functions
+      service.toUserType = jest.fn(() => {
+        return { userType: 'root' };
+      });
+      service.getUserByPrincipal = jest.fn(() => {
+        return null;
+      });
+      service.findUser = jest.fn(() => {
+        return user;
+      });
+      await service.updateUser({}, user);
+    });
+
+    it('should fail validation when unknown property in schema', async () => {
+      const user = {
+        email: '[email protected]',
+        usernameInIdp: 'example',
+        uid: 'user1',
+        rev: 0,
+        unknown: 'unknownProperty',
+      };
+
+      // mocked functions
+      service.toUserType = jest.fn(() => {
+        return { userType: 'root' };
+      });
+      service.getUserByPrincipal = jest.fn(() => {
+        return null;
+      });
+      service.findUser = jest.fn(() => {
+        return user;
+      });
+      try {
+        await service.updateUser({}, user);
+        expect.fail('Expected to throw error validation errors');
+      } catch (err) {
+        expect(err.message).toEqual('Input has validation errors');
+      }
+    });
+  });
+
   describe('createUsers', () => {
     it('should try to create a user', async () => {
       const user = {

addons/addon-base/packages/services/lib/schema/update-user.json

@@ -6,6 +6,12 @@
     "uid": {
       "type": "string"
     },
+    "usernameInIdp": {
+      "type": "string",
+      "minLength": 3,
+      "maxLength": 300,
+      "pattern": "^[A-Za-z0-9-_.]+$|^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\\.[a-zA-Z0-9-]+)*$"
+    },
     "email": {
       "type": "string",
       "pattern": "^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\\.[a-zA-Z0-9-]+)*$"

addons/addon-base/packages/services/lib/user/__tests__/user-service.test.js

@@ -243,6 +243,7 @@ describe('UserService', () => {
       // BUILD
       const toUpdate = {
         uid,
+        usernameInIdp: 'user1',
         rev: 2,
       };