fix(formatting): fix bad data formatting on visits count (#49)

SocialGouv · Apr 13, 2021 · 630b96f · 630b96f
1 parent edec5e4
commit 630b96f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/src/matomo/matomo-utils.test.ts b/src/matomo/matomo-utils.test.ts
@@ -13,7 +13,8 @@ type TestData =
   | (MatomoEventCategory & {
       otherKey: string;
     })[]
-  | number;
+  | number
+  | { value: number };
 
 describe("matomoUtils", () => {
   describe("getBulkRequestParamsFromConfig", () => {
@@ -86,7 +87,7 @@ describe("matomoUtils", () => {
         // eslint-disable-next-line @typescript-eslint/naming-convention
         [{ label: "label2", nb_events: 20, otherKey: "otherKey" }],
         [{ label: "label3", nb_events: 30, otherKey: "otherKey" }],
-        42,
+        { value: 42 },
       ];
 
       expect(createMatomoDataSanitizer(siteConfig)(matomoData)).toEqual([

diff --git a/src/matomo/matomo-utils.ts b/src/matomo/matomo-utils.ts
@@ -168,9 +168,14 @@ const formatMonthlyEvents = (config: MatomoEventConfig): ResultFormatter[] =>
       compose(map(formatMonthlyApiResult(config, date)), formatter)
     );
 
-const formatVisitsResponse = () => (
-  value: number
-): ArchifiltreCountStatistic => ({ label: "visitsCount", value });
+const formatVisitsResponse = () => ({
+  value,
+}: {
+  value: number;
+}): ArchifiltreCountStatistic => ({
+  label: "visitsCount",
+  value,
+});
 
 const formatVisitorCountriesResponse = () => (
   countries: MatomoUserCountry[]