[REF] functions: create helper for COUNTUNIQUE

This commit is a step to introduce the spreadsheet pivot. The function
COUNTUNIQUE is used in the pivot table to count the unique values in a
range. The helper function is created to avoid code duplication.

Task: 3748717
Part-of: #4025
Co-authored-by: rrahir <rar@odoo.com>
Co-authored-by: Pierre Rousseau <pro@odoo.com>

src/functions/helper_math.ts

@@ -1,6 +1,21 @@
-import { Arg, Locale } from "../types";
-import { reduceNumbers } from "./helpers";
+import { Arg, FPayload, Locale } from "../types";
+import { reduceAny, reduceNumbers } from "./helpers";
+
+export function isDefined(data: FPayload | undefined): boolean {
+  if (data === undefined) {
+    return false;
+  }
+  const { value } = data;
+  if (value === null || value === "") {
+    return false;
+  }
+  return true;
+}
 
 export function sum(values: Arg[], locale: Locale): number {
   return reduceNumbers(values, (acc, a) => acc + a, 0, locale);
 }
+
+export function countUnique(args: Arg[]): number {
+  return reduceAny(args, (acc, a) => (isDefined(a) ? acc.add(a?.value) : acc), new Set()).size;
+}

src/functions/module_math.ts

@@ -11,7 +11,7 @@ import {
 import { CellErrorType } from "../types/errors";
 import { arg } from "./arguments";
 import { assertPositive } from "./helper_assert";
-import { sum } from "./helper_math";
+import { countUnique, isDefined, sum } from "./helper_math";
 import { getUnitMatrix } from "./helper_matrices";
 import {
   assert,
@@ -520,17 +520,6 @@ export const COUNTIFS = {
 // COUNTUNIQUE
 // -----------------------------------------------------------------------------
 
-function isDefined(data: FPayload | undefined): boolean {
-  if (data === undefined) {
-    return false;
-  }
-  const { value } = data;
-  if (value === null || value === "") {
-    return false;
-  }
-  return true;
-}
-
 export const COUNTUNIQUE = {
   description: _t("Counts number of unique values in a range."),
   args: [
@@ -542,7 +531,7 @@ export const COUNTUNIQUE = {
   ],
   returns: ["NUMBER"],
   compute: function (...args: Arg[]): number {
-    return reduceAny(args, (acc, a) => (isDefined(a) ? acc.add(a?.value) : acc), new Set()).size;
+    return countUnique(args);
   },
 } satisfies AddFunctionDescription;