update and remove tests

Author: jackiequach

playwright/pageobjects/web-reader.page.ts

@@ -44,14 +44,12 @@ class WebReaderPage {
     this.scrollingMode = page.getByText('Scrolling', { exact: true });
 
     // footer
-    this.nextPageButton = this.page
-      .getByRole('contentinfo')
-      .getByRole('button', {
-        name: 'Next Page',
-      });
-    this.previousPageButton = page
-      .getByRole('contentinfo')
-      .getByRole('button', { name: 'Previous Page' });
+    this.nextPageButton = this.page.getByRole('button', {
+      name: 'Next page',
+    });
+    this.previousPageButton = page.getByRole('button', {
+      name: 'Previous page',
+    });
   }
 
   async changeScreenSize(): Promise<void> {
@@ -294,4 +292,4 @@ class PdfReaderPage extends WebReaderPage {
   }
 }
 
-export { WebReaderPage, HtmlReaderPage, PdfReaderPage };
+export { HtmlReaderPage, PdfReaderPage, WebReaderPage };

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

@@ -1,4 +1,4 @@
-import { test, expect } from '@playwright/test';
+import { expect, test } from '@playwright/test';
 import { HtmlReaderPage } from '../pageobjects/web-reader.page.ts';
 
 test.describe('Test navigation in HTML pub', () => {
@@ -14,29 +14,6 @@ test.describe('Test navigation in HTML pub', () => {
     await expect(htmlReaderPage.lastChapter).toBeVisible();
   });
 
-  test('Click next/previous buttons on first page in paginated mode', async ({
-    page,
-  }) => {
-    const htmlReaderPage = new HtmlReaderPage(page);
-    await htmlReaderPage.loadPub('/html/moby-epub3');
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeEnabled();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeDisabled();
-    await htmlReaderPage.nextPageButton.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeEnabled();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeEnabled();
-    await htmlReaderPage.previousPageButton.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeEnabled();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeDisabled();
-  });
-
   test('Click next/previous buttons on first page in scrolling mode', async ({
     page,
   }) => {
@@ -64,34 +41,6 @@ test.describe('Test navigation in HTML pub', () => {
     await expect(htmlReaderPage.previousPageButton).toBeDisabled();
   });
 
-  test('Click next/previous buttons on last page in paginated mode', async ({
-    page,
-  }) => {
-    const htmlReaderPage = new HtmlReaderPage(page);
-    await htmlReaderPage.loadPub('/html/moby-epub3');
-    await expect(htmlReaderPage.tocButton).toBeVisible();
-    await htmlReaderPage.tocButton.click();
-    await expect(htmlReaderPage.lastChapter).toBeVisible();
-    await htmlReaderPage.lastChapter.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeEnabled();
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeDisabled();
-    await htmlReaderPage.previousPageButton.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeEnabled();
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeEnabled();
-    await htmlReaderPage.nextPageButton.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeEnabled();
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeDisabled();
-  });
-
   test('Click next/previous buttons on last page in scrolling mode', async ({
     page,
   }) => {
@@ -173,21 +122,6 @@ test.describe('Test navigation in HTML pub', () => {
     await expect(htmlReaderPage.chapterHeading).toBeVisible();
   });
 
-  test('Click internal link in paginated mode', async ({ page }) => {
-    const htmlReaderPage = new HtmlReaderPage(page);
-    await htmlReaderPage.loadPub('/html/moby-epub3');
-    await expect(htmlReaderPage.nextPageButton).toBeVisible();
-    await expect(htmlReaderPage.nextPageButton).toBeEnabled();
-    await expect(htmlReaderPage.previousPageButton).toBeVisible();
-    await expect(htmlReaderPage.previousPageButton).toBeDisabled();
-    await htmlReaderPage.nextPageButton.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.internalLink).toBeVisible();
-    await htmlReaderPage.internalLink.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.titlePage).toBeVisible();
-  });
-
   test('Click internal link in scrolling mode', async ({ page }) => {
     const htmlReaderPage = new HtmlReaderPage(page);
     await htmlReaderPage.loadPub('/html/moby-epub3');
@@ -207,22 +141,6 @@ test.describe('Test navigation in HTML pub', () => {
     await expect(htmlReaderPage.titlePage).toBeVisible();
   });
 
-  test('Click external link in paginated mode', async ({ page }) => {
-    const htmlReaderPage = new HtmlReaderPage(page);
-    await htmlReaderPage.loadPub('/html/moby-epub3');
-    await expect(htmlReaderPage.tocButton).toBeVisible();
-    await htmlReaderPage.tocButton.click();
-    await expect(htmlReaderPage.lastChapter).toBeVisible();
-    await htmlReaderPage.lastChapter.click();
-    await htmlReaderPage.loadPage();
-    await expect(htmlReaderPage.externalLink).toBeVisible();
-    await htmlReaderPage.externalLink.click();
-    const openLink = page.waitForEvent('popup');
-    const newTab = await openLink;
-    await newTab.waitForLoadState();
-    await expect(newTab).toHaveURL('https://www.gutenberg.org');
-  });
-
   test('Remember last location when exit and reenter reader', async ({
     page,
   }) => {