From 78b35e48107501b166efa0c30be37cc31da3be03 Mon Sep 17 00:00:00 2001
From: ilkin-overleaf <100852799+ilkin-overleaf@users.noreply.github.com>
Date: Mon, 10 Mar 2025 12:34:01 +0200
Subject: [PATCH] Merge pull request #24057 from
 overleaf/ii-group-management-improvements

[web] Group management improvements

GitOrigin-RevId: 01826bf37fa8d5e06595f660ccb42a8df00b63ae
---
 .../components/managers-table.tsx             |  6 ++-
 .../components/members-table/member-row.tsx   |  2 +-
 .../components/members-table/members-list.tsx |  6 +--
 .../group-management/components/user-row.tsx  |  2 +-
 .../bootstrap-5/components/group-members.scss |  8 +--
 .../stylesheets/components/group-members.less |  8 +--
 .../components/group-managers.spec.tsx        | 16 +++---
 .../components/group-members.spec.tsx         | 32 +++++------
 .../components/institution-managers.spec.tsx  | 16 +++---
 .../components/managed-group-members.spec.tsx | 20 +++----
 .../members-table/dropdown-button.spec.tsx    |  2 +-
 .../members-table/members-list.spec.tsx       | 53 ++++++++++---------
 .../components/publisher-managers.spec.tsx    | 14 ++---
 13 files changed, 95 insertions(+), 90 deletions(-)

diff --git a/services/web/frontend/js/features/group-management/components/managers-table.tsx b/services/web/frontend/js/features/group-management/components/managers-table.tsx
index d5950f6578..d19fedce69 100644
--- a/services/web/frontend/js/features/group-management/components/managers-table.tsx
+++ b/services/web/frontend/js/features/group-management/components/managers-table.tsx
@@ -174,8 +174,10 @@ export function ManagersTable({
             <div className="row-spaced-small">
               <ErrorAlert error={removeMemberError} />
               <OLTable
-                className="managed-users-table managed-users-list structured-list"
-                data-testid="managed-users-table"
+                className="managed-entities-table managed-entities-list structured-list"
+                container={false}
+                hover
+                data-testid="managed-entities-table"
               >
                 <thead>
                   <tr>
diff --git a/services/web/frontend/js/features/group-management/components/members-table/member-row.tsx b/services/web/frontend/js/features/group-management/components/members-table/member-row.tsx
index 27a50c728f..8fec350a3b 100644
--- a/services/web/frontend/js/features/group-management/components/members-table/member-row.tsx
+++ b/services/web/frontend/js/features/group-management/components/members-table/member-row.tsx
@@ -35,7 +35,7 @@ export default function MemberRow({
   const groupSSOActive = getMeta('ol-groupSSOActive')
 
   return (
-    <tr className="managed-user-row">
+    <tr className="managed-entity-row">
       <SelectUserCheckbox user={user} />
       <td
         className={classnames('cell-email', {
diff --git a/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx b/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
index 6fdc81bb25..2fe9339dda 100644
--- a/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
+++ b/services/web/frontend/js/features/group-management/components/members-table/members-list.tsx
@@ -40,9 +40,9 @@ export default function MembersList({ groupId }: ManagedUsersListProps) {
       )}
       <OLTable
         className={classNames(
-          'managed-users-table',
+          'managed-entities-table',
           'structured-list',
-          'managed-users-list',
+          'managed-entities-list',
           {
             'managed-users-active': managedUsersActive,
             'group-sso-active': groupSSOActive,
@@ -50,7 +50,7 @@ export default function MembersList({ groupId }: ManagedUsersListProps) {
         )}
         container={false}
         hover
-        data-testid="managed-users-table"
+        data-testid="managed-entities-table"
       >
         <thead>
           <tr>
diff --git a/services/web/frontend/js/features/group-management/components/user-row.tsx b/services/web/frontend/js/features/group-management/components/user-row.tsx
index fd7fea4b53..013dee1499 100644
--- a/services/web/frontend/js/features/group-management/components/user-row.tsx
+++ b/services/web/frontend/js/features/group-management/components/user-row.tsx
@@ -34,7 +34,7 @@ export default function UserRow({
   )
 
   return (
-    <tr key={`user-${user.email}`} className="managed-user-row">
+    <tr key={`user-${user.email}`} className="managed-entity-row">
       <td className="cell-checkbox">
         <BootstrapVersionSwitcher
           bs3={
diff --git a/services/web/frontend/stylesheets/bootstrap-5/components/group-members.scss b/services/web/frontend/stylesheets/bootstrap-5/components/group-members.scss
index 1c7ba2bde5..bfe5efb677 100644
--- a/services/web/frontend/stylesheets/bootstrap-5/components/group-members.scss
+++ b/services/web/frontend/stylesheets/bootstrap-5/components/group-members.scss
@@ -1,13 +1,13 @@
 /* Styles for group-subscription members view */
 
-.structured-list.managed-users-list {
+.structured-list.managed-entities-list {
   /* Override scrolling behaviour on structured-list */
   overflow: initial;
   overflow-y: initial;
   overflow-x: initial;
 }
 
-.managed-users-list {
+.managed-entities-list {
   vertical-align: middle;
 
   .security-state-invite-pending {
@@ -22,7 +22,7 @@
     color: var(--content-danger);
   }
 
-  .managed-user-row {
+  .managed-entity-row {
     overflow-wrap: break-word;
   }
 
@@ -32,7 +32,7 @@
   }
 }
 
-.managed-users-table {
+.managed-entities-table {
   width: 100%;
   table-layout: fixed;
 
diff --git a/services/web/frontend/stylesheets/components/group-members.less b/services/web/frontend/stylesheets/components/group-members.less
index 88c6d9d0f9..a164d42032 100644
--- a/services/web/frontend/stylesheets/components/group-members.less
+++ b/services/web/frontend/stylesheets/components/group-members.less
@@ -1,13 +1,13 @@
 /* Styles for group-subscription members view */
 
-.structured-list.managed-users-list {
+.structured-list.managed-entities-list {
   /* Override scrolling behaviour on structured-list */
   overflow: initial;
   overflow-y: initial;
   overflow-x: initial;
 }
 
-.managed-users-list {
+.managed-entities-list {
   .security-state-invite-pending {
     color: @text-muted;
   }
@@ -17,7 +17,7 @@
   .security-state-not-managed {
     color: @red;
   }
-  .managed-user-row {
+  .managed-entity-row {
     overflow-wrap: break-word;
   }
 
@@ -61,7 +61,7 @@
   }
 }
 
-.managed-users-table {
+.managed-entities-table {
   width: 100%;
   table-layout: fixed;
 
diff --git a/services/web/test/frontend/features/group-management/components/group-managers.spec.tsx b/services/web/test/frontend/features/group-management/components/group-managers.spec.tsx
index d91660f437..ef7aea1b95 100644
--- a/services/web/test/frontend/features/group-management/components/group-managers.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/group-managers.spec.tsx
@@ -37,7 +37,7 @@ describe('group managers', function () {
   it('renders the group management page', function () {
     cy.findByRole('heading', { name: /my awesome team/i, level: 1 })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -72,7 +72,7 @@ describe('group managers', function () {
       cy.findByRole('button').click()
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(3)').within(() => {
@@ -101,7 +101,7 @@ describe('group managers', function () {
   })
 
   it('checks the select all checkbox', function () {
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -112,13 +112,13 @@ describe('group managers', function () {
         })
       })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('thead')
       .within(() => {
         cy.findByLabelText(/select all/i).check()
       })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -135,7 +135,7 @@ describe('group managers', function () {
       statusCode: 200,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -145,7 +145,7 @@ describe('group managers', function () {
 
     cy.findByRole('button', { name: /remove manager/i }).click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -162,7 +162,7 @@ describe('group managers', function () {
       statusCode: 500,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
diff --git a/services/web/test/frontend/features/group-management/components/group-members.spec.tsx b/services/web/test/frontend/features/group-management/components/group-members.spec.tsx
index 32c506e17e..f2bcb21b97 100644
--- a/services/web/test/frontend/features/group-management/components/group-members.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/group-members.spec.tsx
@@ -64,7 +64,7 @@ describe('GroupMembers', function () {
         'You have added 2 of 10 available members'
       )
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -100,7 +100,7 @@ describe('GroupMembers', function () {
       cy.get('.form-control').type('someone.else@test.com')
       cy.get('.add-more-members-form button').click()
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(3)').within(() => {
@@ -130,7 +130,7 @@ describe('GroupMembers', function () {
     })
 
     it('checks the select all checkbox', function () {
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -143,7 +143,7 @@ describe('GroupMembers', function () {
 
       cy.get('.select-all').click()
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -160,7 +160,7 @@ describe('GroupMembers', function () {
         statusCode: 200,
       })
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -171,7 +171,7 @@ describe('GroupMembers', function () {
       cy.get('button').contains('Remove from group').click()
 
       cy.get('small').contains('You have added 1 of 10 available members')
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -188,7 +188,7 @@ describe('GroupMembers', function () {
         statusCode: 500,
       })
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -257,7 +257,7 @@ describe('GroupMembers', function () {
       cy.get('h1').contains('My Awesome Team')
       cy.get('small').contains('You have added 3 of 10 available members')
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -304,7 +304,7 @@ describe('GroupMembers', function () {
       cy.get('.form-control').type('someone.else@test.com')
       cy.get('.add-more-members-form button').click()
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(4)').within(() => {
@@ -336,7 +336,7 @@ describe('GroupMembers', function () {
     })
 
     it('checks the select all checkbox', function () {
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -349,7 +349,7 @@ describe('GroupMembers', function () {
 
       cy.get('.select-all').click()
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -368,7 +368,7 @@ describe('GroupMembers', function () {
         statusCode: 200,
       })
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -379,7 +379,7 @@ describe('GroupMembers', function () {
       cy.get('button').contains('Remove from group').click()
 
       cy.get('small').contains('You have added 2 of 10 available members')
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -395,7 +395,7 @@ describe('GroupMembers', function () {
         statusCode: 200,
       })
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           // no checkbox should be shown for 'Claire Jennings', a managed user
@@ -410,7 +410,7 @@ describe('GroupMembers', function () {
         statusCode: 500,
       })
 
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(1)').within(() => {
@@ -480,7 +480,7 @@ describe('GroupMembers', function () {
     })
 
     it('should display the Security column', function () {
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.get('tr:nth-child(2)').within(() => {
diff --git a/services/web/test/frontend/features/group-management/components/institution-managers.spec.tsx b/services/web/test/frontend/features/group-management/components/institution-managers.spec.tsx
index 8acbb5d0f6..bf365d1291 100644
--- a/services/web/test/frontend/features/group-management/components/institution-managers.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/institution-managers.spec.tsx
@@ -37,7 +37,7 @@ describe('institution managers', function () {
   it('renders the institution management page', function () {
     cy.findByRole('heading', { name: /my awesome institution/i, level: 1 })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -72,7 +72,7 @@ describe('institution managers', function () {
       cy.findByRole('button').click()
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(3)').within(() => {
@@ -101,7 +101,7 @@ describe('institution managers', function () {
   })
 
   it('checks the select all checkbox', function () {
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -112,13 +112,13 @@ describe('institution managers', function () {
         })
       })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('thead')
       .within(() => {
         cy.findByLabelText(/select all/i).check()
       })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -135,7 +135,7 @@ describe('institution managers', function () {
       statusCode: 200,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -145,7 +145,7 @@ describe('institution managers', function () {
 
     cy.findByRole('button', { name: /remove manager/i }).click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -162,7 +162,7 @@ describe('institution managers', function () {
       statusCode: 500,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
diff --git a/services/web/test/frontend/features/group-management/components/managed-group-members.spec.tsx b/services/web/test/frontend/features/group-management/components/managed-group-members.spec.tsx
index d1f48a76cb..0a97f43483 100644
--- a/services/web/test/frontend/features/group-management/components/managed-group-members.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/managed-group-members.spec.tsx
@@ -86,7 +86,7 @@ describe('group members, with managed users', function () {
     cy.get('h1').contains('My Awesome Team')
     cy.get('small').contains('You have added 3 of 10 available members')
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -134,7 +134,7 @@ describe('group members, with managed users', function () {
     cy.get('.form-control').type('someone.else@test.com')
     cy.get('.add-more-members-form button').click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(4)').within(() => {
@@ -166,7 +166,7 @@ describe('group members, with managed users', function () {
   })
 
   it('checks the select all checkbox', function () {
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -179,7 +179,7 @@ describe('group members, with managed users', function () {
 
     cy.get('.select-all').click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -198,7 +198,7 @@ describe('group members, with managed users', function () {
       statusCode: 200,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -209,7 +209,7 @@ describe('group members, with managed users', function () {
     cy.get('button').contains('Remove from group').click()
 
     cy.get('small').contains('You have added 2 of 10 available members')
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -225,7 +225,7 @@ describe('group members, with managed users', function () {
       statusCode: 200,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         // no checkbox should be shown for 'Claire Jennings', a managed user
@@ -240,7 +240,7 @@ describe('group members, with managed users', function () {
       statusCode: 500,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -275,7 +275,7 @@ describe('Group members when group SSO is enabled', function () {
       win.metaAttributesCache.set('ol-groupSSOActive', false)
     })
     mountGroupMembersProvider()
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(2)').within(() => {
@@ -295,7 +295,7 @@ describe('Group members when group SSO is enabled', function () {
       win.metaAttributesCache.set('ol-groupSSOActive', true)
     })
     mountGroupMembersProvider()
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(2)').within(() => {
diff --git a/services/web/test/frontend/features/group-management/components/members-table/dropdown-button.spec.tsx b/services/web/test/frontend/features/group-management/components/members-table/dropdown-button.spec.tsx
index 9963777bcb..9e59cb84c4 100644
--- a/services/web/test/frontend/features/group-management/components/members-table/dropdown-button.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/members-table/dropdown-button.spec.tsx
@@ -7,7 +7,7 @@ import { User } from '../../../../../../types/group-management/user'
 
 function Wrapper({ children }: PropsWithChildren<Record<string, unknown>>) {
   return (
-    <ul className="managed-users-list">
+    <ul className="managed-entities-list">
       <span
         className="managed-users-actions"
         style={{ display: 'flex', width: '100%', justifyContent: 'flex-end' }}
diff --git a/services/web/test/frontend/features/group-management/components/members-table/members-list.spec.tsx b/services/web/test/frontend/features/group-management/components/members-table/members-list.spec.tsx
index 2741f67110..f3c53972a7 100644
--- a/services/web/test/frontend/features/group-management/components/members-table/members-list.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/members-table/members-list.spec.tsx
@@ -52,16 +52,16 @@ describe('MembersList', function () {
       mountManagedUsersList()
 
       // Select-all checkbox
-      cy.findByTestId('managed-users-table').within(() => {
+      cy.findByTestId('managed-entities-table').within(() => {
         cy.findByTestId('select-all-checkbox')
       })
-      cy.findByTestId('managed-users-table').should('contain.text', 'Email')
-      cy.findByTestId('managed-users-table').should('contain.text', 'Name')
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should('contain.text', 'Email')
+      cy.findByTestId('managed-entities-table').should('contain.text', 'Name')
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         'Last Active'
       )
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'not.contain.text',
         'Security'
       )
@@ -73,48 +73,51 @@ describe('MembersList', function () {
       mountManagedUsersList()
 
       // Select-all checkbox
-      cy.findByTestId('managed-users-table').within(() => {
+      cy.findByTestId('managed-entities-table').within(() => {
         cy.findByTestId('select-all-checkbox')
       })
 
-      cy.findByTestId('managed-users-table').should('contain.text', 'Email')
-      cy.findByTestId('managed-users-table').should('contain.text', 'Name')
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should('contain.text', 'Email')
+      cy.findByTestId('managed-entities-table').should('contain.text', 'Name')
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         'Last Active'
       )
-      cy.findByTestId('managed-users-table').should('contain.text', 'Security')
+      cy.findByTestId('managed-entities-table').should(
+        'contain.text',
+        'Security'
+      )
     })
 
     it('should render the list of users', function () {
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.findAllByRole('row').should('have.length', 2)
         })
       // First user
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[0].email
       )
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[0].first_name
       )
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[0].last_name
       )
       // Second user
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[1].email
       )
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[1].first_name
       )
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         users[1].last_name
       )
@@ -134,11 +137,11 @@ describe('MembersList', function () {
     })
 
     it('should render the list, with a "no members" message', function () {
-      cy.findByTestId('managed-users-table').should(
+      cy.findByTestId('managed-entities-table').should(
         'contain.text',
         'No members'
       )
-      cy.findByTestId('managed-users-table')
+      cy.findByTestId('managed-entities-table')
         .find('tbody')
         .within(() => {
           cy.findAllByRole('row')
@@ -223,7 +226,7 @@ describe('MembersList', function () {
     describe('unlinking user', function () {
       beforeEach(function () {
         mountManagedUsersList()
-        cy.findByTestId('managed-users-table')
+        cy.findByTestId('managed-entities-table')
           .find('tbody')
           .within(() => {
             cy.get('tr:nth-child(3)').within(() => {
@@ -242,7 +245,7 @@ describe('MembersList', function () {
           'contain.text',
           `SSO reauthentication request has been sent to ${USER_LINKED.email}`
         )
-        cy.findByTestId('managed-users-table')
+        cy.findByTestId('managed-entities-table')
           .find('tbody')
           .within(() => {
             cy.get('tr:nth-child(3)').within(() => {
@@ -262,7 +265,7 @@ describe('MembersList', function () {
 
       describe('when user is not managed', function () {
         beforeEach(function () {
-          cy.findByTestId('managed-users-table')
+          cy.findByTestId('managed-entities-table')
             .find('tbody')
             .within(() => {
               cy.get('tr:nth-child(3)').within(() => {
@@ -282,7 +285,7 @@ describe('MembersList', function () {
             'contain.text',
             `SSO reauthentication request has been sent to ${USER_LINKED.email}`
           )
-          cy.findByTestId('managed-users-table')
+          cy.findByTestId('managed-entities-table')
             .find('tbody')
             .within(() => {
               cy.get('tr:nth-child(3)').within(() => {
@@ -295,7 +298,7 @@ describe('MembersList', function () {
 
       describe('when user is managed', function () {
         beforeEach(function () {
-          cy.findByTestId('managed-users-table')
+          cy.findByTestId('managed-entities-table')
             .find('tbody')
             .within(() => {
               cy.get('tr:nth-child(4)').within(() => {
@@ -315,7 +318,7 @@ describe('MembersList', function () {
             'contain.text',
             `SSO reauthentication request has been sent to ${USER_LINKED_AND_MANAGED.email}`
           )
-          cy.findByTestId('managed-users-table')
+          cy.findByTestId('managed-entities-table')
             .find('tbody')
             .within(() => {
               cy.get('tr:nth-child(4)').within(() => {
diff --git a/services/web/test/frontend/features/group-management/components/publisher-managers.spec.tsx b/services/web/test/frontend/features/group-management/components/publisher-managers.spec.tsx
index b39fe9e2d2..12c1765842 100644
--- a/services/web/test/frontend/features/group-management/components/publisher-managers.spec.tsx
+++ b/services/web/test/frontend/features/group-management/components/publisher-managers.spec.tsx
@@ -37,7 +37,7 @@ describe('publisher managers', function () {
   it('renders the publisher management page', function () {
     cy.findByRole('heading', { name: /my awesome publisher/i, level: 1 })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -72,7 +72,7 @@ describe('publisher managers', function () {
       cy.findByRole('button').click()
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(3)').within(() => {
@@ -101,7 +101,7 @@ describe('publisher managers', function () {
   })
 
   it('checks the select all checkbox', function () {
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -114,7 +114,7 @@ describe('publisher managers', function () {
 
     cy.get('.select-all').click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -131,7 +131,7 @@ describe('publisher managers', function () {
       statusCode: 200,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -141,7 +141,7 @@ describe('publisher managers', function () {
 
     cy.findByRole('button', { name: 'Remove manager' }).click()
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {
@@ -158,7 +158,7 @@ describe('publisher managers', function () {
       statusCode: 500,
     })
 
-    cy.findByTestId('managed-users-table')
+    cy.findByTestId('managed-entities-table')
       .find('tbody')
       .within(() => {
         cy.get('tr:nth-child(1)').within(() => {