fix: Fix broken tests after changes made

openedx-unsupported · Jan 8, 2024 · 83a24fe · 83a24fe
1 parent c720f64
commit 83a24fe
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 6 deletions.
diff --git a/src/library-authoring/author-library/LibraryAuthoringPage.jsx b/src/library-authoring/author-library/LibraryAuthoringPage.jsx
@@ -130,12 +130,15 @@ export const BlockPreviewBase = ({
           />
           <OverlayTrigger
             placement="top"
-            overlay={
-              <Tooltip id="more-actions-tooltip">More Actions</Tooltip>
-            }
+            overlay={(
+              <Tooltip id="more-actions-tooltip">
+                {intl.formatMessage(messages['library.detail.block.more_actions'])}
+              </Tooltip>
+            )}
           >
             <Dropdown>
               <Dropdown.Toggle
+                aria-label={intl.formatMessage(messages['library.detail.block.more_actions'])}
                 as={IconButton}
                 src={MoreVert}
                 iconAs={Icon}

diff --git a/src/library-authoring/author-library/messages.js b/src/library-authoring/author-library/messages.js
@@ -165,6 +165,11 @@ const messages = defineMessages({
     defaultMessage: 'Manage tags',
     description: 'Aria label for manage tags button',
   },
+  'library.detail.block.more_actions': {
+    id: 'library.detail.block.more_actions',
+    defaultMessage: 'More actions',
+    description: 'Aria label for more actions button',
+  },
   'library.detail.block.copy': {
     id: 'library.detail.block.copy',
     defaultMessage: 'Copy',

diff --git a/src/library-authoring/author-library/specs/LibraryAuthoringPage.spec.jsx b/src/library-authoring/author-library/specs/LibraryAuthoringPage.spec.jsx
@@ -360,11 +360,16 @@ testSuite('<LibraryAuthoringPageContainer />', () => {
     const library = libraryFactory();
     const block = blockFactory(undefined, { library });
     await render(library, genState(library, [block]));
-    const del = screen.getByLabelText('Delete');
+    const moreActionsButton = screen.getByLabelText('More actions');
     act(() => {
-      del.click();
+      moreActionsButton.click();
     });
-    const yes = await screen.findByText('Delete');
+    const deleteAction = await screen.getByLabelText('Delete');
+    act(() => {
+      deleteAction.click();
+    });
+    const deleteElems = await screen.findAllByText('Delete');
+    const yes = deleteElems[0];
     act(() => {
       yes.click();
     });