test: improve test coverage

app/javascript/__tests__/api/apiService.test.ts

@@ -101,5 +101,20 @@ describe("apiService", () => {
         expect(deleteSpy).toHaveBeenCalled()
       })
     })
+    it("fails if the token is present but invalid", async () => {
+      jest.spyOn(console, "error").mockImplementation(() => {})
+      Storage.prototype.getItem = (_key: string) => {
+        return JSON.stringify({ "access-token": "test-token", expiry: "0" })
+      }
+      try {
+        await authenticatedPost(url, data, config)
+        console.error("This console error should not be called")
+      } catch (error) {
+        // eslint-disable-next-line jest/no-conditional-expect, jest/no-try-expect
+        expect(error).toBeInstanceOf(Error)
+        // eslint-disable-next-line jest/no-conditional-expect, jest/no-try-expect
+        expect(error.message).toBe("Token expired")
+      }
+    })
   })
 })

app/javascript/__tests__/authentication/context/UserProvider.test.tsx

@@ -99,7 +99,7 @@ describe("UserProvider", () => {
     await waitFor(() => expect(screen.getByText("Sign In")).not.toBeNull())
   })
 
-  it("should handle token invalidation on initial load", async () => {
+  it("should handle token invalidation", async () => {
     ;(isTokenValid as jest.Mock).mockReturnValue(false)
     ;(getProfile as jest.Mock).mockRejectedValueOnce(new Error("Token expired"))
 

app/javascript/__tests__/hooks/useGTMDataLayer.test.tsx

@@ -45,4 +45,15 @@ describe("useGTMDataLayer", () => {
 
     expect(consoleSpy).toHaveBeenCalled()
   })
+
+  it("errors out when no event is provided in the data object", () => {
+    const event = "testEvent"
+    const data = { test: "data", event: "testEvent" }
+
+    const { result } = renderHook(() => useGTMDataLayer())
+
+    result.current.pushToDataLayer(event, data)
+
+    expect(consoleSpy).toHaveBeenCalled()
+  })
 })