fix: remove flag check

openedx-unsupported · Feb 16, 2024 · b7af486 · b7af486
1 parent 2d2fe7e
commit b7af486
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 43 deletions.
diff --git a/src/library-authoring/studio-header-wrapper/specs/utils.spec.js b/src/library-authoring/studio-header-wrapper/specs/utils.spec.js
@@ -1,4 +1,3 @@
-import { getConfig, setConfig } from '@edx/frontend-platform';
 import { LOADING_STATUS } from '../../common';
 import { getMainMenuDropdown, getOutlineLink } from '../utils';
 
@@ -24,11 +23,18 @@ describe('studio header wrapper utils', () => {
     });
   });
   describe('getMainMenuDropdown', () => {
-    it('should return an array of length 1', () => {
+    it('should return an array of length 1 and correct items', () => {
       const libraryId = 'testId';
       const loadingStatus = LOADING_STATUS.LOADED;
-      const dropdownArray = getMainMenuDropdown(loadingStatus, libraryId, { formatMessage: jest.fn() });
+      const dropdownArray = getMainMenuDropdown(loadingStatus, libraryId, intl);
       expect(dropdownArray).toHaveLength(1);
+      const subItemTitles = dropdownArray[0].items.map(item => item.title);
+      expect(subItemTitles).toEqual([
+        'Details',
+        'User access',
+        'Export Tags',
+        'Import',
+      ]);
     });
     it('should return an empty array', () => {
       const libraryId = 'testId';
@@ -37,38 +43,4 @@ describe('studio header wrapper utils', () => {
       expect(dropdownArray).toHaveLength(0);
     });
   });
-
-  it('should show the export tags sub item when the flag is true', () => {
-    const libraryId = 'testId';
-    const loadingStatus = LOADING_STATUS.LOADED;
-    setConfig({
-      ...getConfig(),
-      ENABLE_TAGGING_TAXONOMY_PAGES: 'true',
-    });
-    const dropdownArray = getMainMenuDropdown(loadingStatus, libraryId, intl);
-    expect(dropdownArray).toHaveLength(1);
-    const subItemTitles = dropdownArray[0].items.map(item => item.title);
-    expect(subItemTitles).toEqual([
-      'Details',
-      'User access',
-      'Export Tags',
-      'Import',
-    ]);
-  });
-  it('should not show the export tags sub item when the flag is false', () => {
-    const libraryId = 'testId';
-    const loadingStatus = LOADING_STATUS.LOADED;
-    setConfig({
-      ...getConfig(),
-      ENABLE_TAGGING_TAXONOMY_PAGES: 'false',
-    });
-    const dropdownArray = getMainMenuDropdown(loadingStatus, libraryId, intl);
-    expect(dropdownArray).toHaveLength(1);
-    const subItemTitles = dropdownArray[0].items.map(item => item.title);
-    expect(subItemTitles).toEqual([
-      'Details',
-      'User access',
-      'Import',
-    ]);
-  });
 });
diff --git a/src/library-authoring/studio-header-wrapper/utils.js b/src/library-authoring/studio-header-wrapper/utils.js
@@ -24,12 +24,10 @@ export const getMainMenuDropdown = (loadingStatus, libraryId, intl) => {
             href: ROUTES.Detail.ACCESS_SLUG(libraryId),
             title: intl.formatMessage(messages['library.header.settings.access']),
           },
-          ...(getConfig().ENABLE_TAGGING_TAXONOMY_PAGES === 'true' ? [
-            {
-              href: `${getConfig().STUDIO_BASE_URL}/api/content_tagging/v1/object_tags/${libraryId}/export/`,
-              title: intl.formatMessage(messages['library.header.settings.exportTags']),
-            },
-          ] : []),
+          {
+            href: `${getConfig().STUDIO_BASE_URL}/api/content_tagging/v1/object_tags/${libraryId}/export/`,
+            title: intl.formatMessage(messages['library.header.settings.exportTags']),
+          },
           {
             href: ROUTES.Detail.IMPORT_SLUG(libraryId),
             title: intl.formatMessage(messages['library.header.settings.import']),