Updating test to work with material icons and removing TODO comment a…

…nd adding better support for admin entered GO arguments.
CruGlobal · Oct 31, 2024 · d8fa224 · d8fa224
1 parent 4a14bf5
commit d8fa224
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/components/Announcements/AnnouncementAction/AnnouncementAction.test.tsx b/src/components/Announcements/AnnouncementAction/AnnouncementAction.test.tsx
@@ -50,13 +50,12 @@ const TestComponent: React.FC<AnnouncementActionProps> = ({
 describe('AnnouncementAction', () => {
   describe('Icon', () => {
     it('initial', () => {
-      const { getByRole, queryByText } = render(
+      const { getByRole } = render(
         <TestComponent actionStyle={ActionStyleEnum.Icon} />,
       );
 
-      expect(queryByText('Contacts')).not.toBeInTheDocument();
-
       const icon = getByRole('button', { name: 'Contacts' });
+      expect(icon.childNodes[0]).toHaveClass('material-icons');
       userEvent.click(icon);
       expect(handlePerformAction).toHaveBeenCalled();
     });

diff --git a/src/components/Announcements/Announcements.tsx b/src/components/Announcements/Announcements.tsx
@@ -96,8 +96,10 @@ const Announcement: React.FC = () => {
         switch (action.action) {
           case ActionEnum.Go:
             if (action.args) {
-              // TODO - Migrate the data on production, to convert to "Go" args to be to the new MPDX URL format
-              push(`/accountLists/${accountListId}/${action.args}`);
+              const args = action.args.startsWith('/')
+                ? action.args
+                : `/${action.args}`;
+              push(`/accountLists/${accountListId}${args}`);
             }
             break;
           case ActionEnum.AppealCreate: