feat: [IOCOM-2100] Common FIMS start API

scripts/generate-api-models.sh

@@ -2,7 +2,7 @@
 
 IO_BACKEND_VERSION=v16.7.4-RELEASE
 # need to change after merge on io-services-metadata
-IO_SERVICES_METADATA_VERSION=1.0.59
+IO_SERVICES_METADATA_VERSION=1.0.60
 # Session manager version
 IO_SESSION_MANAGER_VERSION=1.4.0
 

ts/components/ui/Markdown/handlers/__test__/link.test.ts

@@ -2,9 +2,7 @@ import * as E from "fp-ts/lib/Either";
 import {
   deriveCustomHandledLink,
   isHttpsLink,
-  isIoFIMSLink,
-  isIoInternalLink,
-  removeFIMSPrefixFromUrl
+  isIoInternalLink
 } from "../link";
 
 const loadingCases: ReadonlyArray<
@@ -73,19 +71,6 @@ const loadingCases: ReadonlyArray<
   ]
 ];
 
-const fimsCases: ReadonlyArray<
-  [input: string, expectedResult: ReturnType<typeof removeFIMSPrefixFromUrl>]
-> = [
-  [
-    "iosso://https://italia.io/main/messages?messageId=4&serviceId=5",
-    "https://italia.io/main/messages?messageId=4&serviceId=5"
-  ],
-  [
-    "iOsSo://https://italia.io/main/messages?messageId=4&serviceId=5",
-    "https://italia.io/main/messages?messageId=4&serviceId=5"
-  ]
-];
-
 describe("deriveCustomHandledLink", () => {
   test.each(loadingCases)(
     "given %p as argument, returns %p",
@@ -96,16 +81,6 @@ describe("deriveCustomHandledLink", () => {
   );
 });
 
-describe("removeFIMSPrefixFromUrl", () => {
-  test.each(fimsCases)(
-    "given %p as argument, returns %p",
-    (firstArg, expectedResult) => {
-      const result = removeFIMSPrefixFromUrl(firstArg);
-      expect(result).toEqual(expectedResult);
-    }
-  );
-});
-
 describe("isHttpsLink", () => {
   ["https://", "hTtPs://", "HTTPS://"].forEach(protocol => {
     it(`should return true for '${protocol}'`, () => {
@@ -139,39 +114,6 @@ describe("isHttpsLink", () => {
   });
 });
 
-describe("isIoFIMSLink", () => {
-  ["iosso://", "iOsSo://", "IOSSO://"].forEach(protocol => {
-    it(`should return true for '${protocol}'`, () => {
-      const isIOFIMSLink = isIoFIMSLink(`${protocol}whatever`);
-      expect(isIOFIMSLink).toBe(true);
-    });
-  });
-  [
-    "iosso:/",
-    "iosso:",
-    "iosso",
-    "https://",
-    "http://",
-    "ioit://",
-    "iohandledlink://",
-    "clipboard://",
-    "clipboard:",
-    "sms://",
-    "sms:",
-    "tel://",
-    "tel:",
-    "mailto://",
-    "mailto:",
-    "copy://",
-    "copy:"
-  ].forEach(protocol => {
-    it(`should return false for '${protocol}'`, () => {
-      const isIOFIMSLink = isIoFIMSLink(`${protocol}whatever`);
-      expect(isIOFIMSLink).toBe(false);
-    });
-  });
-});
-
 describe("isIoInternalLink", () => {
   ["ioit://", "iOiT://", "IOIT://"].forEach(protocol => {
     it(`should return true for '${protocol}'`, () => {

ts/components/ui/Markdown/handlers/link.ts

@@ -2,28 +2,15 @@ import { IOToast } from "@pagopa/io-app-design-system";
 import * as E from "fp-ts/lib/Either";
 import * as t from "io-ts";
 import I18n from "../../../../i18n";
-import {
-  IO_FIMS_LINK_PREFIX,
-  IO_FIMS_LINK_PROTOCOL,
-  IO_INTERNAL_LINK_PREFIX
-} from "../../../../utils/navigation";
+import { IO_INTERNAL_LINK_PREFIX } from "../../../../utils/navigation";
 import { openWebUrl } from "../../../../utils/url";
 
 export const isIoInternalLink = (href: string): boolean =>
   href.toLowerCase().startsWith(IO_INTERNAL_LINK_PREFIX);
 
-export const isIoFIMSLink = (href: string): boolean =>
-  href.toLowerCase().startsWith(IO_FIMS_LINK_PREFIX);
-
 export const isHttpsLink = (href: string): boolean =>
   href.toLowerCase().startsWith("https://");
 
-export const removeFIMSPrefixFromUrl = (fimsUrlWithProtocol: string) => {
-  // eslint-disable-next-line no-useless-escape
-  const regexp = new RegExp(`^${IO_FIMS_LINK_PROTOCOL}\/\/`, "i");
-  return fimsUrlWithProtocol.replace(regexp, "");
-};
-
 /**
  * a dedicated codec for CustomHandledLink
  * ex: iohandledlink://tel:1234567 -> {url: tel:1234567, type: tel, value:1234567}

ts/features/fims/common/analytics/__tests__/index.test.ts

@@ -16,6 +16,7 @@ import { ServiceId } from "../../../../../../definitions/backend/ServiceId";
 import { ServicePublic } from "../../../../../../definitions/backend/ServicePublic";
 import * as mixpanel from "../../../../../mixpanel";
 import { GlobalState } from "../../../../../store/reducers/types";
+import { MESSAGES_ROUTES } from "../../../../messages/navigation/routes";
 import * as serviceSelectors from "../../../../services/details/store/reducers";
 import * as fimsAuthenticationSelectors from "../../../singleSignOn/store/selectors";
 
@@ -26,9 +27,6 @@ const organizationNames = [undefined, "organization name"];
 const referenceReason = "The reason";
 const referenceServiceId = "01J9RSWBB4VSHVRJSY33XGA6YH" as ServiceId;
 const serviceNames = [undefined, "service name"];
-const sources: ReadonlyArray<
-  "message_detail" | "service_detail" | "credential_detail"
-> = ["message_detail", "service_detail", "credential_detail"] as const;
 
 describe("trackAuthenticationStart", () => {
   beforeEach(() => {
@@ -38,38 +36,37 @@ describe("trackAuthenticationStart", () => {
   organizationFiscalCodes.forEach(organizationFiscalCode =>
     organizationNames.forEach(organizationName =>
       serviceNames.forEach(serviceName =>
-        sources.forEach(source =>
-          it(`should match event name, and expected parameters for ${
-            organizationFiscalCode ? "defined   " : "undefined "
-          } organization fiscal code, ${
-            organizationName ? "defined   " : "undefined "
-          } organization name, ${
-            serviceName ? "defined   " : "undefined "
-          } service name, ${source} source, `, () => {
-            const mixpanelTrackMock = generateMixpanelTrackMock();
-            void trackAuthenticationStart(
-              referenceServiceId,
-              serviceName,
-              organizationName,
-              organizationFiscalCode,
-              referenceCtaLabel,
-              source
-            );
-            expect(mixpanelTrackMock.mock.calls.length).toBe(1);
-            expect(mixpanelTrackMock.mock.calls[0].length).toBe(2);
-            expect(mixpanelTrackMock.mock.calls[0][0]).toBe("FIMS_START");
-            expect(mixpanelTrackMock.mock.calls[0][1]).toEqual({
-              event_category: "UX",
-              event_type: "action",
-              fims_label: referenceCtaLabel,
-              organization_fiscal_code: organizationFiscalCode,
-              organization_name: organizationName,
-              service_id: referenceServiceId,
-              service_name: serviceName,
-              source
-            });
-          })
-        )
+        it(`should match event name, and expected parameters for ${
+          organizationFiscalCode ? "defined   " : "undefined "
+        } organization fiscal code, ${
+          organizationName ? "defined   " : "undefined "
+        } organization name, ${
+          serviceName ? "defined   " : "undefined "
+        } service name`, () => {
+          const source = MESSAGES_ROUTES.MESSAGE_DETAIL;
+          const mixpanelTrackMock = generateMixpanelTrackMock();
+          void trackAuthenticationStart(
+            referenceServiceId,
+            serviceName,
+            organizationName,
+            organizationFiscalCode,
+            referenceCtaLabel,
+            source
+          );
+          expect(mixpanelTrackMock.mock.calls.length).toBe(1);
+          expect(mixpanelTrackMock.mock.calls[0].length).toBe(2);
+          expect(mixpanelTrackMock.mock.calls[0][0]).toBe("FIMS_START");
+          expect(mixpanelTrackMock.mock.calls[0][1]).toEqual({
+            event_category: "UX",
+            event_type: "action",
+            fims_label: referenceCtaLabel,
+            organization_fiscal_code: organizationFiscalCode,
+            organization_name: organizationName,
+            service_id: referenceServiceId,
+            service_name: serviceName,
+            source
+          });
+        })
       )
     )
   );

ts/features/fims/common/analytics/index.ts

@@ -12,7 +12,7 @@ export const trackAuthenticationStart = (
   organizationName: string | undefined,
   organizationFiscalCode: string | undefined,
   ctaLabel: string,
-  source: "message_detail" | "service_detail" | "credential_detail"
+  source: string
 ) => {
   const eventName = `FIMS_START`;
   const props = buildEventProperties("UX", "action", {