Merge pull request #5626 from masslight/fix-setup

skip telemed prebook when config says so

Author: imbenham

apps/intake/tests/specs/0_paperworkSetup/setup.spec.ts

@@ -36,7 +36,7 @@ test.afterAll(async () => {
 import { CancelPage } from 'tests/utils/CancelPage';
 import { BaseInPersonFlow } from 'tests/utils/in-person/BaseInPersonFlow';
 import { ResourceHandler } from 'tests/utils/resource-handler';
-import { chooseJson, CreateAppointmentResponse, GetSlotDetailsResponse, PROJECT_NAME } from 'utils';
+import { BOOKING_CONFIG, chooseJson, CreateAppointmentResponse, GetSlotDetailsResponse, PROJECT_NAME } from 'utils';
 import { FillingInfo as InPersonFillingInfo } from '../../utils/in-person/FillingInfo';
 import { PrebookInPersonFlow } from '../../utils/in-person/PrebookInPersonFlow';
 import { Locators } from '../../utils/locators';
@@ -362,60 +362,64 @@ test.describe.parallel('In-Person: Create test patients and appointments', () =>
 });
 
 test.describe.parallel('Telemed: Create test patients and appointments', () => {
-  test('Create patient with responsible party with insurance payment prebook appointment', async ({ page }) => {
-    const { prebookFlowClass, paperwork } = await test.step('Set up playwright', async () => {
-      addAppointmentToIdsAndAddMetaTag(page, processId);
-      const prebookFlowClass = new PrebookTelemedFlow(page);
-      const paperwork = new Paperwork(page);
-      return { prebookFlowClass, paperwork };
-    });
+  const TMPrebookDependentTest = BOOKING_CONFIG.homepageOptions.includes('schedule-virtual-visit') ? test : test.skip;
+  TMPrebookDependentTest(
+    'Create patient with responsible party with insurance payment prebook appointment',
+    async ({ page }) => {
+      const { prebookFlowClass, paperwork } = await test.step('Set up playwright', async () => {
+        addAppointmentToIdsAndAddMetaTag(page, processId);
+        const prebookFlowClass = new PrebookTelemedFlow(page);
+        const paperwork = new Paperwork(page);
+        return { prebookFlowClass, paperwork };
+      });
 
-    const { bookingData, filledPaperwork } = await test.step('Book first appointment', async () => {
-      await prebookFlowClass.selectVisitAndContinue();
-      const bookingData = await prebookFlowClass.startVisitWithoutPaperwork();
-      await paperwork.clickProceedToPaperwork();
-      const filledPaperwork = await prebookFlowClass.fillPaperwork({
-        payment: 'insurance',
-        responsibleParty: 'not-self',
-        patientBasicInfo: bookingData.patientBasicInfo,
+      const { bookingData, filledPaperwork } = await test.step('Book first appointment', async () => {
+        await prebookFlowClass.selectVisitAndContinue();
+        const bookingData = await prebookFlowClass.startVisitWithoutPaperwork();
+        await paperwork.clickProceedToPaperwork();
+        const filledPaperwork = await prebookFlowClass.fillPaperwork({
+          payment: 'insurance',
+          responsibleParty: 'not-self',
+          patientBasicInfo: bookingData.patientBasicInfo,
+        });
+        await prebookFlowClass.completeBooking();
+        await prebookFlowClass.cancelAppointment();
+        return { bookingData, filledPaperwork };
       });
-      await prebookFlowClass.completeBooking();
-      await prebookFlowClass.cancelAppointment();
-      return { bookingData, filledPaperwork };
-    });
 
-    await test.step('Book second appointment without filling paperwork', async () => {
-      await prebookFlowClass.selectVisitAndContinue();
-      await prebookFlowClass.startVisitWithoutPaperwork(bookingData.patientBasicInfo);
-    });
+      await test.step('Book second appointment without filling paperwork', async () => {
+        await prebookFlowClass.selectVisitAndContinue();
+        await prebookFlowClass.startVisitWithoutPaperwork(bookingData.patientBasicInfo);
+      });
 
-    await test.step('Write test data to file', async () => {
-      const prebookTelemedPatient: TelemedPrebookPatientTestData = {
-        firstName: bookingData.patientBasicInfo.firstName,
-        lastName: bookingData.patientBasicInfo.lastName,
-        email: bookingData.patientBasicInfo.email,
-        birthSex: bookingData.patientBasicInfo.birthSex,
-        dateOfBirth: bookingData.patientBasicInfo.dob,
-        appointmentId: getLastAppointmentId(page),
-        state: filledPaperwork.stateValue,
-        // todo because i'm not great at type conditional types apparently
-        patientDetailsData: filledPaperwork.patientDetailsData as PatientDetailsData,
-        // todo because i'm not great at type conditional types apparently
-        pcpData: filledPaperwork.pcpData as PrimaryCarePhysicianData,
-        insuranceData: filledPaperwork.insuranceData,
-        secondaryInsuranceData: filledPaperwork.secondaryInsuranceData,
-        responsiblePartyData: filledPaperwork.responsiblePartyData,
-        medicationData: filledPaperwork.medicationData,
-        allergiesData: filledPaperwork.allergiesData,
-        medicalHistoryData: filledPaperwork.medicalHistoryData,
-        surgicalHistoryData: filledPaperwork.surgicalHistoryData,
-        flags: filledPaperwork.flags!,
-        uploadedPhotoCondition: filledPaperwork.uploadedPhotoCondition,
-      };
-      console.log('prebookTelemedPatient', JSON.stringify(prebookTelemedPatient));
-      writeTestData('prebookTelemedPatient.json', prebookTelemedPatient);
-    });
-  });
+      await test.step('Write test data to file', async () => {
+        const prebookTelemedPatient: TelemedPrebookPatientTestData = {
+          firstName: bookingData.patientBasicInfo.firstName,
+          lastName: bookingData.patientBasicInfo.lastName,
+          email: bookingData.patientBasicInfo.email,
+          birthSex: bookingData.patientBasicInfo.birthSex,
+          dateOfBirth: bookingData.patientBasicInfo.dob,
+          appointmentId: getLastAppointmentId(page),
+          state: filledPaperwork.stateValue,
+          // todo because i'm not great at type conditional types apparently
+          patientDetailsData: filledPaperwork.patientDetailsData as PatientDetailsData,
+          // todo because i'm not great at type conditional types apparently
+          pcpData: filledPaperwork.pcpData as PrimaryCarePhysicianData,
+          insuranceData: filledPaperwork.insuranceData,
+          secondaryInsuranceData: filledPaperwork.secondaryInsuranceData,
+          responsiblePartyData: filledPaperwork.responsiblePartyData,
+          medicationData: filledPaperwork.medicationData,
+          allergiesData: filledPaperwork.allergiesData,
+          medicalHistoryData: filledPaperwork.medicalHistoryData,
+          surgicalHistoryData: filledPaperwork.surgicalHistoryData,
+          flags: filledPaperwork.flags!,
+          uploadedPhotoCondition: filledPaperwork.uploadedPhotoCondition,
+        };
+        console.log('prebookTelemedPatient', JSON.stringify(prebookTelemedPatient));
+        writeTestData('prebookTelemedPatient.json', prebookTelemedPatient);
+      });
+    }
+  );
 
   test('Create patient without responsible party and card payment walk-in appointment', async ({ page }) => {
     const walkInFlowClass = await test.step('Set up playwright', async () => {

apps/intake/tests/specs/telemed/PaperworkTelemedExistingPatient.spec.ts

@@ -2,6 +2,7 @@
 import { BrowserContext, expect, Page, test } from '@playwright/test';
 import * as fs from 'fs';
 import * as path from 'path';
+import { BOOKING_CONFIG } from 'utils';
 import { Locators } from '../../utils/locators';
 import { Paperwork, PATIENT_ADDRESS, PATIENT_ADDRESS_LINE_2, PATIENT_CITY, PATIENT_ZIP } from '../../utils/Paperwork';
 import { TelemedPrebookPatientTestData } from '../0_paperworkSetup/types';
@@ -26,7 +27,11 @@ test.afterAll(async () => {
   await context.close();
 });
 
-test.describe.parallel('Telemed - Prefilled Paperwork, Responsible Party: not self, Payment: Insurance', () => {
+const VirtualPrebookDependentTests = BOOKING_CONFIG.homepageOptions?.includes('schedule-virtual-visit')
+  ? test.describe.parallel
+  : test.describe.skip;
+
+VirtualPrebookDependentTests('Telemed - Prefilled Paperwork, Responsible Party: not self, Payment: Insurance', () => {
   test('VVPP-1. Contact information', async () => {
     await test.step('VVPP-1.1. Open Contact information page directly', async () => {
       await page.goto(`paperwork/${patient.appointmentId}/contact-information`);

apps/intake/tests/specs/telemed/homepage.spec.ts

@@ -33,7 +33,11 @@ test.describe.parallel('THP. Telemed Homepage', () => {
       }
     }
 
-    await expect(page.getByRole('heading', { name: 'Book a visit', level: 2 })).toBeVisible({
+    await expect(
+      page.getByRole('tablist', {
+        name: 'Appointment tabs for switching between appointments slots for today and tomorrow',
+      })
+    ).toBeVisible({
       timeout: 15000,
     });
   });