loading by default

Author: wildjames

packages/cdk/resources/Cognito.ts

@@ -187,8 +187,10 @@ export class Cognito extends Construct {
 
     const callbackUrls = [
       `https://${props.fullCloudfrontDomain}/site/`,
+      // FIXME: This is temporary, until we get routing fixed
       `https://${props.fullCloudfrontDomain}/site/auth_demo.html`,
-      `https://${props.fullCloudfrontDomain}/auth_demo/`
+      `https://${props.fullCloudfrontDomain}/auth_demo/`,
+      `https://${props.fullCloudfrontDomain}/oauth2/idpresponse`
     ]
 
     const logoutUrls = [

packages/cpt-ui/__tests__/SelectYourRolePage.test.tsx

@@ -33,22 +33,6 @@ describe("SelectYourRolePage", () => {
     jest.clearAllMocks();
   });
 
-  it("renders a heading by default (not signed in) but eventually shows an error", async () => {
-    // Not signed in
-    renderWithAuth({
-      isSignedIn: false,
-      idToken: "",
-    });
-
-    // Wait for error summary to appear
-    await waitFor(() => {
-      const errorHeading = screen.getByRole("heading", { name: /Error during role selection/i })
-      expect(errorHeading).toBeInTheDocument();
-      const errorText = screen.getByText("No login session found");
-      expect(errorText).toBeInTheDocument();
-    });
-  });
-
   it("renders loading state when signed in but fetch hasn't resolved yet", async () => {
     // Mock fetch to never resolve
     mockFetch.mockImplementation(() => new Promise(() => {}));
@@ -161,7 +145,7 @@ describe("SelectYourRolePage", () => {
     expect(confirmButton).toBeInTheDocument();
 
     // Roles Without Access details expander
-    const expander = screen.getByText("Roles without access");
+    const expander = screen.getByText("View your roles without access to the clinical prescription tracking service.");
     expect(expander).toBeInTheDocument();
 
     // Expand details if needed or just confirm the table is present

packages/cpt-ui/constants/ui-strings/CardStrings.ts

@@ -38,5 +38,6 @@ export const SELECT_ROLE_PAGE_TEXT = {
   },
   alternativeMessage: "Alternatively, you can choose a new role below.",
   organisation: "Organisation",
-  role: "Role"
+  role: "Role",
+  roles_without_access_table_title: "View your roles without access to the clinical prescription tracking service."
 }