Remove dead code (#42823)

enterprise/frontend/src/metabase-enterprise/content_verification/index.ts

@@ -3,20 +3,13 @@ import { hasPremiumFeature } from "metabase-enterprise/settings";
 
 import { ModelFilterControls } from "./ModelFilterControls";
 import { VerifiedFilter } from "./VerifiedFilter";
-import {
-  availableModelFilters,
-  sortCollectionsByVerification,
-  sortModelsByVerification,
-  useModelFilterSettings,
-} from "./utils";
+import { availableModelFilters, useModelFilterSettings } from "./utils";
 
 if (hasPremiumFeature("content_verification")) {
   Object.assign(PLUGIN_CONTENT_VERIFICATION, {
     VerifiedFilter,
     ModelFilterControls,
     availableModelFilters,
-    sortModelsByVerification,
-    sortCollectionsByVerification,
     useModelFilterSettings,
   });
 }

enterprise/frontend/src/metabase-enterprise/content_verification/utils.ts

@@ -6,34 +6,6 @@ import type {
   AvailableModelFilters,
 } from "metabase/browse/utils";
 import { useUserSetting } from "metabase/common/hooks";
-import type { CollectionEssentials, SearchResult } from "metabase-types/api";
-
-export const sortCollectionsByVerification = (
-  collection1: CollectionEssentials,
-  collection2: CollectionEssentials,
-) => {
-  const isCollection1Official = collection1.authority_level === "official";
-  const isCollection2Official = collection2.authority_level === "official";
-  if (isCollection1Official && !isCollection2Official) {
-    return -1;
-  }
-  if (isCollection2Official && !isCollection1Official) {
-    return 1;
-  }
-  return 0;
-};
-
-export const sortModelsByVerification = (a: SearchResult, b: SearchResult) => {
-  const aVerified = a.moderated_status === "verified";
-  const bVerified = b.moderated_status === "verified";
-  if (aVerified && !bVerified) {
-    return -1;
-  }
-  if (!aVerified && bVerified) {
-    return 1;
-  }
-  return 0;
-};
 
 export const availableModelFilters: AvailableModelFilters = {
   onlyShowVerifiedModels: {

enterprise/frontend/src/metabase-enterprise/content_verification/utils.unit.spec.ts

 import { createMockModelResult } from "metabase/browse/test-utils";
-import type { CollectionEssentials } from "metabase-types/api";
 
-import { availableModelFilters, sortCollectionsByVerification } from "./utils";
+import { availableModelFilters } from "./utils";
 
 describe("Utilities related to content verification", () => {
-  it("include a function that sorts verified collections before unverified collections", () => {
-    const unsorted: CollectionEssentials[] = [
-      {
-        id: 99,
-        authority_level: "official",
-        name: "Collection Zulu - verified",
-      },
-      {
-        id: 1,
-        authority_level: null,
-        name: "Collection Alpha - unverified",
-      },
-    ];
-    const sortFunction = (a: CollectionEssentials, b: CollectionEssentials) =>
-      sortCollectionsByVerification(a, b) || a.name.localeCompare(b.name);
-    const sorted = unsorted.sort(sortFunction);
-    expect(sorted[0].name).toBe("Collection Zulu - verified");
-    expect(sorted[1].name).toBe("Collection Alpha - unverified");
-  });
   it("include a constant that defines a filter for only showing verified models", () => {
     const models = [
       createMockModelResult({