fix: Remove never published filter from component picker

src/library-authoring/LibraryAuthoringPage.tsx

@@ -37,6 +37,7 @@ import {
   SearchKeywordsField,
   SearchSortWidget,
   TypesFilterData,
+  PublishStatus,
 } from '../search-manager';
 import LibraryContent from './LibraryContent';
 import { LibrarySidebar } from './library-sidebar';
@@ -259,6 +260,11 @@ const LibraryAuthoringPage = ({
     <Tab key={contentType} eventKey={contentType} title={tabTitles[contentType]} />
   ));
 
+  const publishedFilters = [PublishStatus.Published, PublishStatus.Modified];
+  if (!showOnlyPublished) {
+    publishedFilters.push(PublishStatus.NeverPublished);
+  }
+
   return (
     <div className="d-flex">
       <div className="flex-grow-1">
@@ -299,7 +305,7 @@ const LibraryAuthoringPage = ({
               <SearchKeywordsField className="mr-3" />
               <FilterByTags />
               {!(insideCollections || insideUnits) && <FilterByBlockType />}
-              <FilterByPublished />
+              <FilterByPublished visibleFilters={publishedFilters} />
               <ClearFiltersButton />
               <ActionRow.Spacer />
               <SearchSortWidget />

src/library-authoring/component-picker/ComponentPicker.test.tsx

@@ -302,4 +302,20 @@ describe('<ComponentPicker />', () => {
     expect(screen.queryByRole('tab', { name: /collections/i })).not.toBeInTheDocument();
     expect(screen.queryByRole('tab', { name: /components/i })).not.toBeInTheDocument();
   });
+
+  it('should not display never published filter', async () => {
+    render(<ComponentPicker />);
+
+    expect(await screen.findByText('Test Library 1')).toBeInTheDocument();
+    fireEvent.click(screen.getByDisplayValue(/lib:sampletaxonomyorg1:tl1/i));
+
+    // Wait for the content library to load
+    const filterButton = await screen.findByRole('button', { name: /publish status/i });
+    fireEvent.click(filterButton);
+
+    // Verify the filters. Note: It's hard to verify the `published` filter,
+    // because there are many components with that text on the screen, but that's not the important thing.
+    expect(screen.getByText(/modified since publish/i)).toBeInTheDocument();
+    expect(screen.queryByText(/never published/i)).not.toBeInTheDocument();
+  });
 });

src/search-manager/FilterByPublished.tsx

@@ -10,12 +10,18 @@ import { FormattedMessage, useIntl } from '@edx/frontend-platform/i18n';
 import messages from './messages';
 import SearchFilterWidget from './SearchFilterWidget';
 import { useSearchContext } from './SearchManager';
-import { PublishStatus } from './data/api';
+import { allPublishFilters, PublishStatus } from './data/api';
+
+interface FilterByPublishedProps {
+  visibleFilters?: PublishStatus[],
+}
 
 /**
  * A button with a dropdown that allows filtering the current search by publish status
  */
-const FilterByPublished: React.FC<Record<never, never>> = () => {
+const FilterByPublished = ({
+  visibleFilters = allPublishFilters,
+}: FilterByPublishedProps) => {
   const intl = useIntl();
   const {
     publishStatus,
@@ -42,6 +48,25 @@ const FilterByPublished: React.FC<Record<never, never>> = () => {
   };
   const appliedFilters = publishStatusFilter.map(mode => ({ label: modeToLabel[mode] }));
 
+  const filterLabels = {
+    [PublishStatus.Published]: intl.formatMessage(messages.publishStatusPublished),
+    [PublishStatus.Modified]: intl.formatMessage(messages.publishStatusModified),
+    [PublishStatus.NeverPublished]: intl.formatMessage(messages.publishStatusNeverPublished),
+  };
+
+  const visibleFiltersToRender = visibleFilters.map((filter) => (
+    <MenuItem
+      as={Form.Checkbox}
+      value={filter}
+      onChange={() => { toggleFilterMode(filter); }}
+    >
+      <div>
+        {filterLabels[filter]}
+        <Badge variant="light" pill>{publishStatus[filter] ?? 0}</Badge>
+      </div>
+    </MenuItem>
+  ));
+
   return (
     <SearchFilterWidget
       appliedFilters={appliedFilters}
@@ -55,36 +80,7 @@ const FilterByPublished: React.FC<Record<never, never>> = () => {
           value={publishStatusFilter}
         >
           <Menu className="block-type-refinement-menu" style={{ boxShadow: 'none' }}>
-            <MenuItem
-              as={Form.Checkbox}
-              value={PublishStatus.Published}
-              onChange={() => { toggleFilterMode(PublishStatus.Published); }}
-            >
-              <div>
-                {intl.formatMessage(messages.publishStatusPublished)}
-                <Badge variant="light" pill>{publishStatus[PublishStatus.Published] ?? 0}</Badge>
-              </div>
-            </MenuItem>
-            <MenuItem
-              as={Form.Checkbox}
-              value={PublishStatus.Modified}
-              onChange={() => { toggleFilterMode(PublishStatus.Modified); }}
-            >
-              <div>
-                {intl.formatMessage(messages.publishStatusModified)}
-                <Badge variant="light" pill>{publishStatus[PublishStatus.Modified] ?? 0}</Badge>
-              </div>
-            </MenuItem>
-            <MenuItem
-              as={Form.Checkbox}
-              value={PublishStatus.NeverPublished}
-              onChange={() => { toggleFilterMode(PublishStatus.NeverPublished); }}
-            >
-              <div>
-                {intl.formatMessage(messages.publishStatusNeverPublished)}
-                <Badge variant="light" pill>{publishStatus[PublishStatus.NeverPublished] ?? 0}</Badge>
-              </div>
-            </MenuItem>
+            {visibleFiltersToRender}
           </Menu>
         </Form.CheckboxSet>
       </Form.Group>

src/search-manager/data/api.ts

@@ -31,6 +31,8 @@ export enum PublishStatus {
   NeverPublished = 'never',
 }
 
+export const allPublishFilters: PublishStatus[] = Object.values(PublishStatus);
+
 /**
  * Get the content search configuration from the CMS.
  */