From 71fcf9f168dd24fa1aff33b474aa38c4d0d4cc22 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=B4mulo=20Penido?= <romulo.penido@gmail.com>
Date: Wed, 10 Jul 2024 14:29:55 +0200
Subject: [PATCH] fix: only show course blocks in the search modal (#1148)

---
 src/search-modal/SearchUI.test.tsx | 9 +++++++--
 src/search-modal/SearchUI.tsx      | 5 ++++-
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/search-modal/SearchUI.test.tsx b/src/search-modal/SearchUI.test.tsx
index 7a62193e6d..7c0b0f1faa 100644
--- a/src/search-modal/SearchUI.test.tsx
+++ b/src/search-modal/SearchUI.test.tsx
@@ -175,7 +175,8 @@ describe('<SearchUI />', () => {
     expect(fetchMock).toHaveLastFetched((_url, req) => {
       const requestData = JSON.parse(req.body?.toString() ?? '');
       const requestedFilter = requestData?.queries[0].filter;
-      return requestedFilter?.[0] === 'context_key = "course-v1:org+test+123"';
+      return requestedFilter?.[0] === 'type = "course_block"'
+        && requestedFilter?.[1] === 'context_key = "course-v1:org+test+123"';
     });
     // Now we should see the results:
     expect(queryByText('Enter a keyword')).toBeNull();
@@ -398,7 +399,8 @@ describe('<SearchUI />', () => {
       expect(fetchMock).toHaveLastFetched((_url, req) => {
         const requestData = JSON.parse(req.body?.toString() ?? '');
         const requestedFilter = requestData?.queries[0].filter;
-        return (requestedFilter?.length === 1); // the filter is: 'context_key = "course-v1:org+test+123"'
+        // the filter is: ['type = "course_block"', 'context_key = "course-v1:org+test+123"']
+        return (requestedFilter?.length === 2);
       });
       // Now we should see the results:
       expect(getByText('6 results found')).toBeInTheDocument();
@@ -423,6 +425,7 @@ describe('<SearchUI />', () => {
         const requestData = JSON.parse(req.body?.toString() ?? '');
         const requestedFilter = requestData?.queries[0].filter;
         return JSON.stringify(requestedFilter) === JSON.stringify([
+          'type = "course_block"',
           'context_key = "course-v1:org+test+123"',
           ['block_type = problem'], // <-- the newly added filter, sent with the request
         ]);
@@ -448,6 +451,7 @@ describe('<SearchUI />', () => {
         const requestData = JSON.parse(req.body?.toString() ?? '');
         const requestedFilter = requestData?.queries?.[0]?.filter;
         return JSON.stringify(requestedFilter) === JSON.stringify([
+          'type = "course_block"',
           'context_key = "course-v1:org+test+123"',
           'tags.taxonomy = "ESDC Skills and Competencies"', // <-- the newly added filter, sent with the request
         ]);
@@ -481,6 +485,7 @@ describe('<SearchUI />', () => {
         const requestData = JSON.parse(req.body?.toString() ?? '');
         const requestedFilter = requestData?.queries?.[0]?.filter;
         return JSON.stringify(requestedFilter) === JSON.stringify([
+          'type = "course_block"',
           'context_key = "course-v1:org+test+123"',
           'tags.level0 = "ESDC Skills and Competencies > Abilities"',
         ]);
diff --git a/src/search-modal/SearchUI.tsx b/src/search-modal/SearchUI.tsx
index ce60d762b8..10406b13b8 100644
--- a/src/search-modal/SearchUI.tsx
+++ b/src/search-modal/SearchUI.tsx
@@ -27,7 +27,10 @@ const SearchUI: React.FC<{ courseId?: string, closeSearchModal?: () => void }> =
 
   return (
     <SearchContextProvider
-      extraFilter={searchThisCourse ? `context_key = "${props.courseId}"` : undefined}
+      extraFilter={[
+        'type = "course_block"',
+        ...(searchThisCourse ? [`context_key = "${props.courseId}"`] : []),
+      ]}
       closeSearchModal={props.closeSearchModal}
     >
       {/* We need to override z-index here or the <Dropdown.Menu> appears behind the <ModalDialog.Body>