Merge pull request #855 from CruGlobal/845-lock-staff-profile-blocks

[EVENT-845] Allow admins to edit profile fields
CruGlobal · May 1, 2024 · 680e601 · 680e601
2 parents 1fe57f8 + 7ea1da2
commit 680e601
Show file tree

Hide file tree

Showing 2 changed files with 56 additions and 2 deletions.
diff --git a/app/scripts/directives/blocks.js b/app/scripts/directives/blocks.js
@@ -27,7 +27,10 @@ angular.module('confRegistrationWebApp').directive('nameQuestion', function () {
 
       const user = $rootScope.globalUser();
       $scope.lockedStaffProfileBlock = Boolean(
-        user && user.employeeId && $scope.block.profileType === 'NAME',
+        !$scope.adminEditRegistrant &&
+          user &&
+          user.employeeId &&
+          $scope.block.profileType === 'NAME',
       );
     },
   };
@@ -72,7 +75,10 @@ angular
       controller: function ($rootScope, $scope) {
         const user = $rootScope.globalUser();
         $scope.lockedStaffProfileBlock = Boolean(
-          user && user.employeeId && $scope.block.profileType === 'EMAIL',
+          !$scope.adminEditRegistrant &&
+            user &&
+            user.employeeId &&
+            $scope.block.profileType === 'EMAIL',
         );
       },
     };

diff --git a/test/spec/directives/block.spec.js b/test/spec/directives/block.spec.js
@@ -84,6 +84,30 @@ describe('Directive: blocks', () => {
           expect($scope.lockedStaffProfileBlock).toBe(false);
         });
       });
+
+      describe('when an admin is editing', () => {
+        beforeEach(() => {
+          $scope.adminEditRegistrant = {};
+          spyOn($rootScope, 'globalUser').and.returnValue({
+            employeeId: '0123456',
+          });
+        });
+
+        it('is false when the profile type is NAME', () => {
+          $compile('<name-question></name-question>')($scope);
+          $scope.$digest();
+
+          expect($scope.lockedStaffProfileBlock).toBe(false);
+        });
+
+        it('is false when the profile type is not NAME', () => {
+          $scope.block.profileType = null;
+          $compile('<name-question></name-question>')($scope);
+          $scope.$digest();
+
+          expect($scope.lockedStaffProfileBlock).toBe(false);
+        });
+      });
     });
   });
 
@@ -146,6 +170,30 @@ describe('Directive: blocks', () => {
         });
       });
 
+      describe('when an admin is editing', () => {
+        beforeEach(() => {
+          $scope.adminEditRegistrant = {};
+          spyOn($rootScope, 'globalUser').and.returnValue({
+            employeeId: '0123456',
+          });
+        });
+
+        it('is false when the profile type is EMAIL', () => {
+          $compile('<email-question></email-question>')($scope);
+          $scope.$digest();
+
+          expect($scope.lockedStaffProfileBlock).toBe(false);
+        });
+
+        it('is false when the profile type is not EMAIL', () => {
+          $scope.block.profileType = null;
+          $compile('<email-question></email-question>')($scope);
+          $scope.$digest();
+
+          expect($scope.lockedStaffProfileBlock).toBe(false);
+        });
+      });
+
       describe('with no profile', () => {
         beforeEach(() => {
           spyOn($rootScope, 'globalUser').and.returnValue(null);