Use string/contains for new high cardinality string field filters (#29885)

e2e/test/scenarios/native-filters/sql-filters-source.cy.spec.js

@@ -223,6 +223,7 @@ describe("scenarios > filters > sql filters > values source", () => {
       SQLFilter.openTypePickerFromDefaultFilterType();
       SQLFilter.chooseType("Field Filter");
       FieldFilter.mapTo({ table: "Products", field: "Ean" });
+      FieldFilter.setWidgetType("String");
       setFilterQuestionSource({ question: "SQL source", field: "EAN" });
       saveQuestion("SQL filter");
 
@@ -304,6 +305,7 @@ describe("scenarios > filters > sql filters > values source", () => {
       SQLFilter.openTypePickerFromDefaultFilterType();
       SQLFilter.chooseType("Field Filter");
       FieldFilter.mapTo({ table: "Products", field: "Ean" });
+      FieldFilter.setWidgetType("String");
       setFilterListSource({ values: ["1018947080336", "7663515285824"] });
       saveQuestion("SQL filter");
 

frontend/src/metabase-lib/parameters/utils/template-tag-options.ts

@@ -45,7 +45,16 @@ export function getDefaultParameterWidgetType(tag: TemplateTag, field: Field) {
     options.some(option => option.type === widgetType)
   ) {
     return widgetType;
-  } else {
-    return options[0].type;
   }
+
+  const distinctCount = field.fingerprint?.global?.["distinct-count"];
+  if (
+    distinctCount != null &&
+    distinctCount > 20 &&
+    options.some(option => option.type === "string/contains")
+  ) {
+    return "string/contains";
+  }
+
+  return options[0].type;
 }

frontend/src/metabase-types/api/field.ts

@@ -4,7 +4,7 @@ import { TableId } from "./table";
 export type FieldId = number;
 
 export interface FieldFingerprint {
-  global: FieldGlobalFingerprint;
+  global?: FieldGlobalFingerprint;
   type?: FieldTypeFingerprint;
 }
 

frontend/src/metabase/query_builder/components/template_tags/TagEditorParam.unit.spec.tsx

@@ -107,7 +107,25 @@ describe("TagEditorParam", () => {
   });
 
   describe("tag dimension", () => {
-    it("should default to string/= for a new string field filter", async () => {
+    it("should default to string/= for a new low cardinality string field filter", async () => {
+      const tag = createMockTemplateTag({
+        type: "dimension",
+        dimension: undefined,
+        "widget-type": undefined,
+      });
+      const { setTemplateTag } = setup({ tag });
+
+      userEvent.click(await screen.findByText("People"));
+      userEvent.click(await screen.findByText("Source"));
+
+      expect(setTemplateTag).toHaveBeenCalledWith({
+        ...tag,
+        dimension: ["field", PEOPLE.SOURCE, null],
+        "widget-type": "string/=",
+      });
+    });
+
+    it("should default to string/contains for a new high cardinality string field filter", async () => {
       const tag = createMockTemplateTag({
         type: "dimension",
         dimension: undefined,
@@ -121,7 +139,7 @@ describe("TagEditorParam", () => {
       expect(setTemplateTag).toHaveBeenCalledWith({
         ...tag,
         dimension: ["field", PEOPLE.NAME, null],
-        "widget-type": "string/=",
+        "widget-type": "string/contains",
       });
     });