Added extra awaits

Author: clarissarichard

playwright/pageobjects/web-reader.page.ts

@@ -136,7 +136,6 @@ class HtmlReaderPage extends WebReaderPage {
 
   async scrollDown(): Promise<void> {
     await this.tocButton.click();
-    //await expect(this.tocButton).toBeVisible();
     await this.chapterName.click();
     await expect(await this.tocButton.getAttribute('aria-expanded')).toBe(
       'false'
@@ -183,11 +182,13 @@ class PdfReaderPage extends WebReaderPage {
   }
 
   async scrollDown(): Promise<void> {
+    await expect(this.pageTwo).toBeVisible();
     await this.pageTwo.scrollIntoViewIfNeeded();
   }
 
   async scrollUp(): Promise<void> {
     await this.scrollDown();
+    await expect(this.pageOne).toBeVisible();
     await this.pageOne.scrollIntoViewIfNeeded();
   }
 }

playwright/tests/html-reader-settings.spec.ts

@@ -100,9 +100,9 @@ test.describe('Test HTML pub', () => {
     await htmlReaderPage.chapterName.click();
     await htmlReaderPage.settingsButton.click();
     await htmlReaderPage.scrollingStyle.click();
-    await expect(htmlReaderPage.chapterName).toBeVisible();
+    await expect(htmlReaderPage.chapterHeading).toBeVisible();
     await htmlReaderPage.paginatedStyle.click();
-    await expect(htmlReaderPage.chapterName).toBeVisible();
+    await expect(htmlReaderPage.chapterHeading).toBeVisible();
   });
 
   test('Maintain changed settings when exit and reenter reader', async ({

playwright/tests/pdf-reader-settings.spec.ts

@@ -5,6 +5,7 @@ test.describe('Test PDF pub', () => {
   test('Confirm reader settings are visible', async ({ page }) => {
     const pdfReaderPage = new PdfReaderPage(page);
     await pdfReaderPage.loadPage('/pdf/collection');
+    await expect(pdfReaderPage.settingsButton).toBeVisible();
     await expect(pdfReaderPage.fullScreenButton).toBeVisible();
     await pdfReaderPage.settingsButton.click();
     await expect(pdfReaderPage.zoomInButton).toBeVisible();
@@ -35,8 +36,10 @@ test.describe('Test PDF pub', () => {
     await pdfReaderPage.settingsButton.click();
     await pdfReaderPage.scrollingStyle.click();
     await expect(pdfReaderPage.scrollingStyle).toBeChecked();
+    await pdfReaderPage.settingsButton.click();
     await pdfReaderPage.scrollDown();
     await pdfReaderPage.scrollUp();
+    await pdfReaderPage.settingsButton.click();
     await pdfReaderPage.paginatedStyle.click();
     await expect(pdfReaderPage.paginatedStyle).toBeChecked();
   });
@@ -56,6 +59,7 @@ test.describe('Test PDF pub', () => {
   test('Open and exit full screen', async ({ page }) => {
     const pdfReaderPage = new PdfReaderPage(page);
     await pdfReaderPage.loadPage('/pdf/collection');
+    await expect(pdfReaderPage.fullScreenButton).toBeVisible();
     await pdfReaderPage.fullScreenButton.click();
     await pdfReaderPage.exitFullScreenButton.click();
     await expect(pdfReaderPage.fullScreenButton).toBeVisible();