Activate/Deactivate User By ID (#95)

backend/app/graphql/types.py

@@ -57,6 +57,7 @@ class UserInfo(graphene.ObjectType):
     role_info = graphene.Field(RoleInfo)
     primary_contact = graphene.Field(Contact)
     onsite_contacts = graphene.List(Contact)
+    active = graphene.Boolean()
 
 
 class ContactInput(graphene.InputObjectType):
@@ -88,6 +89,7 @@ class UserInfoInput(graphene.InputObjectType):
     role_info = graphene.Field(RoleInfoInput)
     primary_contact = graphene.Field(ContactInput, required=True)
     onsite_contacts = graphene.List(ContactInput, required=True)
+    active = graphene.Boolean()
 
 
 class User(graphene.ObjectType):

backend/app/graphql/user_mutations.py

@@ -41,5 +41,47 @@ def mutate(self, info, requestor_id, id, userInfo):
             )
 
 
+class ActivateUserByID(Mutation):
+    class Arguments:
+        requestor_id = graphene.String(required=True)
+        id = graphene.String(required=True)
+
+    user = graphene.Field(User)
+
+    def mutate(self, info, requestor_id, id):
+        user_service = services["user_service"]
+        requestor_auth_id = user_service.get_auth_id_by_user_id(requestor_id)
+        requestor_role = user_service.get_user_role_by_auth_id(requestor_auth_id)
+
+        if requestor_role == "Admin":
+            activate_user_dto = user_service.activate_user_by_id(id)
+
+            return ActivateUserByID(
+                user=User(id=activate_user_dto.id, info=activate_user_dto.info)
+            )
+
+
+class DeactivateUserByID(Mutation):
+    class Arguments:
+        requestor_id = graphene.String(required=True)
+        id = graphene.String(required=True)
+
+    user = graphene.Field(User)
+
+    def mutate(self, info, requestor_id, id):
+        user_service = services["user_service"]
+        requestor_auth_id = user_service.get_auth_id_by_user_id(requestor_id)
+        requestor_role = user_service.get_user_role_by_auth_id(requestor_auth_id)
+
+        if requestor_role == "Admin" or requestor_id == id:
+            deactivate_user_dto = user_service.deactivate_user_by_id(id)
+
+            return DeactivateUserByID(
+                user=User(id=deactivate_user_dto.id, info=deactivate_user_dto.info)
+            )
+
+
 class UserMutations(MutationList):
     updateUserByID = UpdateUserByID.Field()
+    activateUserByID = ActivateUserByID.Field()
+    deactivateUserByID = DeactivateUserByID.Field()

backend/app/models/user_info.py

@@ -40,5 +40,6 @@ class UserInfo(mg.EmbeddedDocument):
     role_info = mg.EmbeddedDocumentField(RoleInfo)
     primary_contact = mg.EmbeddedDocumentField(Contact, required=True)
     onsite_contacts = mg.EmbeddedDocumentListField(Contact, required=True)
+    active = mg.BooleanField(default=True)
 
     meta = {"allow_inheritance": True}

backend/app/resources/validate_utils.py

@@ -58,6 +58,7 @@ def validate_userinfo(userinfo, error_list):
         "role_info",
         "primary_contact",
         "onsite_contacts",
+        "active",
     ]
     if not isinstance(userinfo, dict):
         error_list.append("The info supplied is not a dict.")
@@ -83,7 +84,9 @@ def validate_userinfo(userinfo, error_list):
             error_list = validate_role_info(
                 userinfo["role"], val, "info.role_info", error_list
             )
-        elif type(val) is not str:
+        elif key == "active" and type(val) is not bool:
+            error_list.append("The field info.active supplied is not a boolean.")
+        elif type(val) is not str and key != "active":
             error_list.append(f"The field info.{key} supplied is not a string.")
         elif val == "":
             error_list.append(

backend/app/services/implementations/onboarding_request_service.py

@@ -33,6 +33,7 @@ def create_onboarding_request(self, userInfo):
                 role_info=userInfo.role_info,
                 primary_contact=userInfo.primary_contact,
                 onsite_contacts=userInfo.onsite_contacts,
+                active=userInfo.active,
             )
             # Create OnboardingRequest object
             new_onboarding_request = OnboardingRequest(

backend/app/services/implementations/user_service.py

@@ -264,6 +264,48 @@ def update_user_by_id(self, user_id, update_user_dto):
         }
         return UserDTO(**kwargs)
 
+    def activate_user_by_id(self, user_id):
+        try:
+            user = User.objects(id=user_id).first()
+            if not user:
+                raise Exception(f"user_id {user_id} not found")
+
+            user.info.active = True  # activate user
+
+            user.save()  # save changes
+
+            updated_user_dict = (
+                UserService.__user_to_serializable_dict_and_remove_auth_id(user)
+            )
+            kwargs = {"id": updated_user_dict["id"], "info": updated_user_dict["info"]}
+
+            return UserDTO(**kwargs)
+
+        except Exception as e:
+            self.logger.error(f"Failed to activate user. Reason = {e}")
+            raise e
+
+    def deactivate_user_by_id(self, user_id):
+        try:
+            user = User.objects(id=user_id).first()
+            if not user:
+                raise Exception(f"user_id {user_id} not found")
+
+            user.info.active = False  # deactivate user
+
+            user.save()  # save changes
+
+            updated_user_dict = (
+                UserService.__user_to_serializable_dict_and_remove_auth_id(user)
+            )
+            kwargs = {"id": updated_user_dict["id"], "info": updated_user_dict["info"]}
+
+            return UserDTO(**kwargs)
+
+        except Exception as e:
+            self.logger.error(f"Failed to deactivate user. Reason = {e}")
+            raise e
+
     def delete_user_by_id(self, user_id):
         try:
             deleted_user = User.objects(id=user_id).modify(remove=True, new=False)

backend/app/services/interfaces/user_service.py

@@ -116,6 +116,32 @@ def update_user_by_id(self, user_id, user):
         """
         pass
 
+    @abstractmethod
+    def activate_user_by_id(self, user_id):
+        """
+        Activate a user
+
+        :param user_id: user's id
+        :type user_id: str
+        :return: the activated user
+        :rtype: UserDTO
+        :raises Exception: if user update fails
+        """
+        pass
+
+    @abstractmethod
+    def deactivate_user_by_id(self, user_id):
+        """
+        Deactivate a user
+
+        :param user_id: user's id
+        :type user_id: str
+        :return: the deactivated user
+        :rtype: UserDTO
+        :raises Exception: if user update fails
+        """
+        pass
+
     @abstractmethod
     def delete_user_by_id(self, user_id):
         """

backend/tests/graphql/mock_test_data.py

@@ -19,6 +19,7 @@
         {"name": "abc", "phone": "123-456-7890", "email": "[email protected]"},
         {"name": "Jane Doe", "phone": "111-222-3333", "email": "[email protected]"},
     ],
+    "active": True,
 }
 
 MOCK_INFO1_CAMEL = {
@@ -42,6 +43,7 @@
         {"name": "abc", "phone": "123-456-7890", "email": "[email protected]"},
         {"name": "Jane Doe", "phone": "111-222-3333", "email": "[email protected]"},
     ],
+    "active": True,
 }
 
 MOCK_INFO2_SNAKE = {
@@ -66,6 +68,7 @@
         {"name": "def", "phone": "098-765-4321", "email": "[email protected]"},
         {"name": "John Doe", "phone": "444-555-6666", "email": "[email protected]"},
     ],
+    "active": True,
 }
 
 MOCK_INFO2_CAMEL = {
@@ -90,6 +93,7 @@
         {"name": "def", "phone": "098-765-4321", "email": "[email protected]"},
         {"name": "John Doe", "phone": "444-555-6666", "email": "[email protected]"},
     ],
+    "active": True,
 }
 
 MOCK_INFO3_SNAKE = {
@@ -108,6 +112,7 @@
         {"name": "ghi", "phone": "135-792-4680", "email": "[email protected]"},
         {"name": "Jack Doe", "phone": "777-888-999", "email": "[email protected]"},
     ],
+    "active": False,
 }
 
 MOCK_INFO3_CAMEL = {
@@ -126,6 +131,7 @@
         {"name": "ghi", "phone": "135-792-4680", "email": "[email protected]"},
         {"name": "Jack Doe", "phone": "777-888-999", "email": "[email protected]"},
     ],
+    "active": False,
 }
 
 MOCK_USER1_SNAKE = {