feat: video gallery status filter wording

openedx · Oct 14, 2023 · 08ea104 · 08ea104
1 parent 1983f2b
commit 08ea104
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/src/editors/containers/VideoGallery/index.test.jsx b/src/editors/containers/VideoGallery/index.test.jsx
@@ -165,7 +165,7 @@ describe('VideoGallery', () => {
 
       await act(async () => {
         fireEvent.click(screen.getByRole('button', {
-          name: 'Video status',
+          name: 'Filter by video status',
         }));
       });
       await act(async () => {

diff --git a/src/editors/containers/VideoGallery/messages.js b/src/editors/containers/VideoGallery/messages.js
@@ -55,10 +55,10 @@ export const messages = {
   },
 
   // Filter Dropdown
-  filterByVideoStatusNone: {
-    id: 'authoring.selectvideomodal.filter.videostatusnone.label',
-    defaultMessage: 'Video status',
-    description: 'Dropdown label for filter by video status (none)',
+  filterByVideoStatusTitle: {
+    id: 'authoring.selectvideomodal.filter.title.label',
+    defaultMessage: 'Filter by video status',
+    description: 'Title for video status filter dropdown',
   },
   filterByVideoStatusUploading: {
     id: 'authoring.selectvideomodal.filter.videostatusuploading.label',

diff --git a/src/editors/containers/VideoGallery/utils.js b/src/editors/containers/VideoGallery/utils.js
@@ -29,7 +29,7 @@ export const filterKeys = StrictDict({
 });
 
 export const filterMessages = StrictDict({
-  title: messages[messageKeys.filterByVideoStatusNone],
+  title: messages[messageKeys.filterByVideoStatusTitle],
   uploading: messages[messageKeys.filterByVideoStatusUploading],
   processing: messages[messageKeys.filterByVideoStatusProcessing],
   ready: messages[messageKeys.filterByVideoStatusReady],

diff --git a/src/editors/sharedComponents/SelectionModal/SearchSort.test.jsx b/src/editors/sharedComponents/SelectionModal/SearchSort.test.jsx
@@ -72,7 +72,7 @@ describe('SearchSort component', () => {
   test('adds a filter option for each filterKet', async () => {
     const { getByRole } = getComponent();
     await act(() => {
-      fireEvent.click(screen.getByRole('button', { name: /video status/i }));
+      fireEvent.click(screen.getByRole('button', { name: /Filter by video status/i }));
     });
     Object.keys(filterMessages)
       .forEach((key) => {