Production deploy

.github/workflows/cron-deploy-to-production.yml

@@ -17,7 +17,7 @@ jobs:
         # Requires a Personal Access Token generated by an OSL GitHub org admin
         # Allows this action to trigger the push-production.yml action
         # PAT requires the "Read and write" permissions on repository "Contents"
-        # XXX: Expires 23/11/24, see docs here on generating a new one - https://bit.ly/3YNr5sM
+        # Note: Jess generated latest token, expires 23/11/25, see docs here on generating a new one - https://bit.ly/3YNr5sM
         PERSONAL_ACCESS_TOKEN: ${{ secrets.PROD_DEPLOY_PAT }}
       run: |
         set -xe

api.planx.uk/.env.test.example

@@ -45,3 +45,7 @@ ORDNANCE_SURVEY_API_KEY=👻
 
 IDOX_NEXUS_TOKEN_URL=👻
 IDOX_NEXUS_SUBMISSION_URL=👻
+
+# Analytics
+METABASE_API_KEY=👻
+METABASE_URL_EXT=metabase.example.com

docker-compose.yml

@@ -145,6 +145,8 @@ services:
       IDOX_NEXUS_TOKEN_URL: ${IDOX_NEXUS_TOKEN_URL}
       JWT_SECRET: ${JWT_SECRET}
       MAPBOX_ACCESS_TOKEN: ${MAPBOX_ACCESS_TOKEN}
+      METABASE_API_KEY: ${METABASE_API_KEY}
+      METABASE_URL_EXT: ${METABASE_URL_EXT}
       MINIO_PORT: ${MINIO_PORT}
       ORDNANCE_SURVEY_API_KEY: ${ORDNANCE_SURVEY_API_KEY}
       PORT: ${API_PORT}

e2e/tests/ui-driven/package.json

@@ -21,7 +21,7 @@
   },
   "packageManager": "[email protected]",
   "devDependencies": {
-    "@playwright/test": "^1.40.1",
+    "@playwright/test": "^1.49.0",
     "@types/node": "18.16.1",
     "eslint-plugin-playwright": "^0.20.0"
   }

editor.planx.uk/src/@planx/components/PlanningConstraints/Public.test.tsx

@@ -4,6 +4,7 @@ import React from "react";
 import { act } from "react-dom/test-utils";
 import { ErrorBoundary } from "react-error-boundary";
 import swr from "swr";
+import useSWR from "swr";
 import { setup } from "testUtils";
 import { vi } from "vitest";
 import { axe } from "vitest-axe";
@@ -21,15 +22,13 @@ const { setState } = useStore;
 
 beforeEach(() => vi.clearAllMocks());
 
-const swrMock = (swr as jest.Mock).mock;
-
 vi.mock("swr", () => ({
   default: vi.fn((url: () => string) => {
     const isGISRequest = url()?.startsWith(
-      `${import.meta.env.VITE_APP_API_URL}/gis`
+      `${import.meta.env.VITE_APP_API_URL}/gis`,
     );
     const isRoadsRequest = url()?.startsWith(
-      `${import.meta.env.VITE_APP_API_URL}/roads`
+      `${import.meta.env.VITE_APP_API_URL}/roads`,
     );
 
     if (isGISRequest) return { data: digitalLandResponseMock };
@@ -50,7 +49,7 @@ describe("error state", () => {
           disclaimer="This page does not include information about historic planning conditions that may apply to this property."
           handleSubmit={vi.fn()}
         />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByTestId("error-summary-invalid-graph")).toBeInTheDocument();
@@ -66,7 +65,7 @@ describe("error state", () => {
           fn="property.constraints.planning"
           disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
     const results = await axe(container);
     expect(results).toHaveNoViolations();
@@ -82,7 +81,7 @@ describe("following a FindProperty component", () => {
         teamIntegrations: {
           hasPlanningData: true,
         },
-      })
+      }),
     );
   });
 
@@ -96,11 +95,11 @@ describe("following a FindProperty component", () => {
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         handleSubmit={handleSubmit}
-      />
+      />,
     );
 
     expect(
-      getByRole("heading", { name: "Planning constraints" })
+      getByRole("heading", { name: "Planning constraints" }),
     ).toBeInTheDocument();
 
     await user.click(getByTestId("continue-button"));
@@ -115,7 +114,7 @@ describe("following a FindProperty component", () => {
         description="Things that might affect your project"
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
-      />
+      />,
     );
     const results = await axe(container);
     expect(results).toHaveNoViolations();
@@ -129,14 +128,14 @@ describe("following a FindProperty component", () => {
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         handleSubmit={vi.fn()}
-      />
+      />,
     );
 
     expect(swr).toHaveBeenCalled();
 
     // Planning data is called first
-    const swrURL = swrMock.calls[0][0]();
-    const swrResponse = swrMock.results[0].value;
+    const swrURL = (vi.mocked(useSWR).mock.calls[0][0] as () => {})();
+    const swrResponse = vi.mocked(useSWR).mock.results[0].value;
 
     expect(swrURL).toContain("/gis");
     expect(swrResponse).toEqual({ data: digitalLandResponseMock });
@@ -150,14 +149,14 @@ describe("following a FindProperty component", () => {
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         handleSubmit={vi.fn()}
-      />
+      />,
     );
 
     expect(swr).toHaveBeenCalled();
 
     // Classified roads are called second
-    const swrURL = swrMock.calls[1][0]();
-    const swrResponse = swrMock.results[1].value;
+    const swrURL = (vi.mocked(useSWR).mock.calls[1][0] as () => {})();
+    const swrResponse = vi.mocked(useSWR).mock.results[1].value;
 
     expect(swrURL).toContain("/roads");
     expect(swrResponse).toEqual({ data: classifiedRoadsResponseMock });
@@ -171,7 +170,7 @@ describe("following a FindProperty component", () => {
         teamIntegrations: {
           hasPlanningData: true,
         },
-      })
+      }),
     );
 
     setup(
@@ -181,23 +180,25 @@ describe("following a FindProperty component", () => {
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         handleSubmit={vi.fn()}
-      />
+      />,
     );
 
     expect(swr).toHaveBeenCalled();
 
     // Planning constraints API still called
-    const planingConstraintsURL = swrMock.calls[0][0]();
-    const planingConstraintsResponse = swrMock.results[0].value;
+    const planingConstraintsURL = (
+      vi.mocked(useSWR).mock.calls[0][0] as () => {}
+    )();
+    const planingConstraintsResponse = vi.mocked(useSWR).mock.results[0].value;
 
     expect(planingConstraintsURL).toContain("/gis");
     expect(planingConstraintsResponse).toEqual({
       data: digitalLandResponseMock,
     });
 
     // Classified roads API not called due to missing USRN
-    const swrURL = swrMock.calls[1][0]();
-    const swrResponse = swrMock.results[1].value;
+    const swrURL = (vi.mocked(useSWR).mock.calls[1][0] as () => {})();
+    const swrResponse = vi.mocked(useSWR).mock.results[1].value;
 
     expect(swrURL).toBeNull();
     expect(swrResponse).toEqual({ data: null });
@@ -211,12 +212,12 @@ describe("following a FindProperty component", () => {
         fn="property.constraints.planning"
         disclaimer="This page does not include information about historic planning conditions that may apply to this property."
         handleSubmit={vi.fn()}
-      />
+      />,
     );
 
     // Positive constraints visible by default
     expect(
-      getByRole("heading", { name: /These are the planning constraints/ })
+      getByRole("heading", { name: /These are the planning constraints/ }),
     ).toBeVisible();
     expect(getByRole("button", { name: /Parks and gardens/ })).toBeVisible();
 
@@ -227,7 +228,7 @@ describe("following a FindProperty component", () => {
     expect(showNegativeConstraintsButton).toBeVisible();
 
     const negativeConstraintsContainer = getByTestId(
-      "negative-constraints-list"
+      "negative-constraints-list",
     );
     expect(negativeConstraintsContainer).not.toBeVisible();
 
@@ -248,12 +249,12 @@ describe("following a FindProperty component", () => {
         description="Things that might affect your project"
         fn="property.constraints.planning"
         handleSubmit={vi.fn()}
-      />
+      />,
     );
     expect(
       queryByText(
-        "This page does not include information about historic planning conditions that may apply to this property."
-      )
+        "This page does not include information about historic planning conditions that may apply to this property.",
+      ),
     ).toBeVisible();
   });
 });
@@ -268,7 +269,7 @@ describe("demo state", () => {
           hasPlanningData: false,
         },
         teamSlug: "demo",
-      })
+      }),
     );
   });
   it("should render an error when teamSlug is demo", async () => {
@@ -282,21 +283,21 @@ describe("demo state", () => {
           disclaimer="This page does not include information about historic planning conditions that may apply to this property."
           handleSubmit={handleSubmit}
         />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     const errorMessage = queryByText(
-      "Planning Constraints are not enabled for demo users"
+      "Planning Constraints are not enabled for demo users",
     );
     expect(errorMessage).toBeVisible();
 
     // Check planning constraints has not rendered
     // reused positive constraints from basic layout test
     expect(
-      queryByRole("heading", { name: /These are the planning constraints/ })
+      queryByRole("heading", { name: /These are the planning constraints/ }),
     ).not.toBeInTheDocument();
     expect(
-      queryByRole("button", { name: /Parks and gardens/ })
+      queryByRole("button", { name: /Parks and gardens/ }),
     ).not.toBeInTheDocument();
 
     // Ensure a demo user can continue on in the application

editor.planx.uk/src/@planx/components/Send/Public.test.tsx

@@ -5,7 +5,7 @@ import { FullStore, useStore } from "pages/FlowEditor/lib/store";
 import React from "react";
 import { act } from "react-dom/test-utils";
 import { setup } from "testUtils";
-import { vi } from "vitest";
+import { it,vi } from "vitest";
 import { axe } from "vitest-axe";
 
 import hasuraEventsResponseMock from "./mocks/hasuraEventsResponseMock";
@@ -179,29 +179,34 @@ describe("Uniform overrides for Buckinghamshire", () => {
   });
 });
 
-it("generates a valid breadcrumb", async () => {
-  const handleSubmit = vi.fn();
+it(
+  "generates a valid breadcrumb",
+  async () => {
+    const handleSubmit = vi.fn();
 
-  setup(
-    <SendComponent
-      title="Send"
-      destinations={["bops", "uniform"]}
-      handleSubmit={handleSubmit}
-    />,
-  );
+    setup(
+      <SendComponent
+        title="Send"
+        destinations={["bops", "uniform"]}
+        handleSubmit={handleSubmit}
+      />,
+    );
 
-  await waitFor(() => expect(mockAxios.post).toHaveBeenCalledTimes(1));
-  expect(handleSubmit).toHaveBeenCalledTimes(1);
+    await waitFor(() => expect(mockAxios.post).toHaveBeenCalledTimes(1));
+    expect(handleSubmit).toHaveBeenCalledTimes(1);
 
-  const breadcrumb = handleSubmit.mock.calls[0][0];
+    const breadcrumb = handleSubmit.mock.calls[0][0];
 
-  expect(breadcrumb.data).toEqual(
-    expect.objectContaining({
-      bopsSendEventId: hasuraEventsResponseMock.bops.event_id,
-      uniformSendEventId: hasuraEventsResponseMock.uniform.event_id,
-    }),
-  );
-});
+    expect(breadcrumb.data).toEqual(
+      expect.objectContaining({
+        bopsSendEventId: hasuraEventsResponseMock.bops.event_id,
+        uniformSendEventId: hasuraEventsResponseMock.uniform.event_id,
+      }),
+    );
+    // Flaky test in CI
+  },
+  { retry: 1 },
+);
 
 it("should not have any accessibility violations", async () => {
   const { container } = setup(