add tests, cleanup

src/connect/component.jsx

@@ -4,7 +4,6 @@ import {
   getClientID,
   getClientMetadataID,
   getUserIDToken,
-  getSessionID,
 } from "@paypal/sdk-client/src";
 
 // TODO: What's the expected structure/approach for this interface. It's not a zoid
@@ -22,6 +21,7 @@ export const getConnectComponent = async (merchantProps) => {
       sdkVersion: "", // PPCP proposal: string instead of function
       minified: false,
     });
+
     // FPTI: sdkversion, fraudnet info
     return await window.braintree.connect.create({
       ...loadResult.metadata, // returns a localeURL for assets

src/connect/component.test.js

@@ -0,0 +1,74 @@
+/* @flow */
+
+import {
+  getClientID,
+  getClientMetadataID,
+  getUserIDToken,
+} from "@paypal/sdk-client/src";
+import { loadAxo } from "@paypal/connect-loader-component";
+import { describe, expect, test, vi } from "vitest";
+
+import { getConnectComponent } from "./component";
+
+describe("getConnectComponent: returns ConnectComponent", () => {
+  const mockAxoMetadata = { someData: "data" };
+  const mockProps = { someProp: "value" };
+  beforeEach(() => {
+    window.braintree = {
+      connect: {
+        create: vi.fn(),
+      },
+    };
+
+    vi.mock("@paypal/sdk-client/src", () => {
+      return {
+        getClientID: vi.fn(() => "mock-client-id"),
+        getClientMetadataID: vi.fn(() => "mock-cmid"),
+        getUserIDToken: vi.fn(() => "mock-uid"),
+      };
+    });
+
+    vi.mock("@paypal/connect-loader-component", () => {
+      return {
+        loadAxo: vi.fn(),
+      };
+    });
+
+    loadAxo.mockResolvedValue({ metadata: mockAxoMetadata });
+
+    // getClientID.mockReturnValue("mock-client-id");
+    // getClientMetadataID.mockReturnValue("mock-cmid");
+    // getUserIDToken.mockReturnValue("mock-uid");
+  });
+
+  test("loadAxo and window.braintree.connect.create are called with proper data", async () => {
+    await getConnectComponent(mockProps);
+
+    expect(getClientID).toHaveBeenCalled();
+    expect(getClientMetadataID).toHaveBeenCalled();
+    expect(getUserIDToken).toHaveBeenCalled();
+    expect(loadAxo).toHaveBeenCalled();
+
+    expect(window.braintree.connect.create).toHaveBeenCalledWith({
+      ...mockAxoMetadata,
+      ...mockProps,
+      ppcp: {
+        clientID: "mock-client-id",
+        clientMetadataID: "mock-cmid",
+        userIdToken: "mock-uid",
+        fraudnet: expect.any(Function),
+      },
+    });
+  });
+
+  test("loadAxo failure is handled", async () => {
+    const errorMessage = "Something went wrong";
+    // eslint-disable-next-line no-import-assign
+    loadAxo = vi.fn().mockRejectedValue(errorMessage);
+
+    const error = await getConnectComponent(mockProps);
+
+    expect(error).toBeInstanceOf(Error);
+    expect(error.message).toEqual(errorMessage);
+  });
+});

src/connect/interface.js

@@ -6,5 +6,5 @@ import type { LazyExport } from "../types";
 import { getConnectComponent, type ConnectComponent } from "./component";
 
 export const Connect: LazyExport<ConnectComponent> = async (merchantProps) => {
-  return await getConnectComponent(merchantProps);
+  return getConnectComponent(merchantProps);
 };

src/connect/interface.test.js

@@ -0,0 +1,19 @@
+/* @flow */
+
+import { describe, expect, vi } from "vitest";
+
+import { getConnectComponent } from "./component";
+import { Connect } from "./interface";
+
+describe("interface.js", () => {
+  vi.mock("./component", () => {
+    return {
+      getConnectComponent: vi.fn(),
+    };
+  });
+  it("should call getConnectComponent with merchant props", async () => {
+    const merchantProps = { props: "someProps" };
+    await Connect(merchantProps);
+    expect(getConnectComponent).toBeCalledWith(merchantProps);
+  });
+});