diff --git a/tests/change-page-mode.spec.ts b/tests/change-page-mode.spec.ts index b48d19352..02e2e128f 100644 --- a/tests/change-page-mode.spec.ts +++ b/tests/change-page-mode.spec.ts @@ -1,6 +1,6 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { clickPageMoreActions } from './libs/page-logic'; loadPage(); test.describe('Change page mode(Paper or Edgeless)', () => { @@ -35,12 +35,8 @@ test.describe('Change page mode(Paper or Edgeless)', () => { expect(await edgeless.isVisible()).toBe(true); }); - test.skip('Convert to edgeless by editor header items', async ({ page }) => { - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + test('Convert to edgeless by editor header items', async ({ page }) => { + await clickPageMoreActions(page); const menusEdgelessItem = page.getByTestId('editor-option-menu-edgeless'); await menusEdgelessItem.click(); diff --git a/tests/libs/page-logic.ts b/tests/libs/page-logic.ts new file mode 100644 index 000000000..64b9f053c --- /dev/null +++ b/tests/libs/page-logic.ts @@ -0,0 +1,11 @@ +export async function newPage(page) { + return page.getByTestId('sliderBar').getByText('New Page').click(); +} + +export async function clickPageMoreActions(page) { + return page + .getByTestId('editor-header-items') + .getByRole('button') + .nth(1) + .click(); +} diff --git a/tests/local-first-delete-page.spec.ts b/tests/local-first-delete-page.spec.ts index 2e95d8b8d..814ba9548 100644 --- a/tests/local-first-delete-page.spec.ts +++ b/tests/local-first-delete-page.spec.ts @@ -1,16 +1,16 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; +import { newPage } from './libs/page-logic'; loadPage(); test.describe('Local first delete page', () => { - test.skip('New a page , then delete it in all pages, permanently delete it', async ({ + test('New a page , then delete it in all pages, permanently delete it', async ({ page, }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to restore'); - const originPageUrl = page.url(); const newPageId = page.url().split('/').reverse()[0]; await page.getByRole('link', { name: 'All pages' }).click(); const cell = page.getByRole('cell', { diff --git a/tests/local-first-export-page.spec.ts b/tests/local-first-export-page.spec.ts index 71d9fd4bd..90be01ace 100644 --- a/tests/local-first-export-page.spec.ts +++ b/tests/local-first-export-page.spec.ts @@ -1,9 +1,9 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage, clickPageMoreActions } from './libs/page-logic'; loadPage(); -test.describe.skip('Local first export page', () => { +test.describe('Local first export page', () => { test('New a page ,then open it and export html', async ({ page }) => { await page.getByText('New Page').click(); await page.getByPlaceholder('Title').click(); @@ -18,11 +18,7 @@ test.describe.skip('Local first export page', () => { expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const exportParentBtn = page.getByRole('tooltip', { name: 'Add to favourites Convert to Edgeless Export Delete', }); @@ -37,7 +33,7 @@ test.describe.skip('Local first export page', () => { }); test('New a page ,then open it and export markdown', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page .getByPlaceholder('Title') @@ -49,11 +45,7 @@ test.describe.skip('Local first export page', () => { expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const exportParentBtn = page.getByRole('tooltip', { name: 'Add to favourites Convert to Edgeless Export Delete', }); diff --git a/tests/local-first-favorite-page.spec.ts b/tests/local-first-favorite-page.spec.ts index 7a7aec133..75eb312e2 100644 --- a/tests/local-first-favorite-page.spec.ts +++ b/tests/local-first-favorite-page.spec.ts @@ -1,11 +1,11 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage, clickPageMoreActions } from './libs/page-logic'; loadPage(); -test.describe.skip('Local first favorite and cancel favorite page', () => { +test.describe('Local first favorite and cancel favorite page', () => { test('New a page and open it ,then favorite it', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to favorite'); await page.getByRole('link', { name: 'All pages' }).click(); @@ -15,16 +15,12 @@ test.describe.skip('Local first favorite and cancel favorite page', () => { expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const favoriteBtn = page.getByTestId('editor-option-menu-favorite'); await favoriteBtn.click(); }); test('Cancel favorite', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to favorite'); await page.getByRole('link', { name: 'All pages' }).click(); @@ -34,11 +30,7 @@ test.describe.skip('Local first favorite and cancel favorite page', () => { expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const favoriteBtn = page.getByTestId('editor-option-menu-favorite'); await favoriteBtn.click(); diff --git a/tests/local-first-favorites-items.spec.ts b/tests/local-first-favorites-items.spec.ts index 85e3d3c90..bf05995ab 100644 --- a/tests/local-first-favorites-items.spec.ts +++ b/tests/local-first-favorites-items.spec.ts @@ -1,11 +1,11 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage, clickPageMoreActions } from './libs/page-logic'; loadPage(); -test.describe.skip('Local first favorite items ui', () => { +test.describe('Local first favorite items ui', () => { test('Show favorite items in sidebar', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to favorite'); const newPageId = page.url().split('/').reverse()[0]; @@ -15,11 +15,7 @@ test.describe.skip('Local first favorite items ui', () => { }); expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const favoriteBtn = page.getByTestId('editor-option-menu-favorite'); await favoriteBtn.click(); @@ -32,7 +28,7 @@ test.describe.skip('Local first favorite items ui', () => { }); test('Show favorite items in favorite list', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to favorite'); await page.getByRole('link', { name: 'All pages' }).click(); @@ -41,11 +37,7 @@ test.describe.skip('Local first favorite items ui', () => { }); expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const favoriteBtn = page.getByTestId('editor-option-menu-favorite'); await favoriteBtn.click(); diff --git a/tests/local-first-new-page.spec.ts b/tests/local-first-new-page.spec.ts index a6753cbc2..7a4f5bd69 100644 --- a/tests/local-first-new-page.spec.ts +++ b/tests/local-first-new-page.spec.ts @@ -1,18 +1,18 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage } from './libs/page-logic'; loadPage(); -test.describe.skip('local first new page', () => { +test.describe('local first new page', () => { test('click btn new page', async ({ page }) => { const originPageId = page.url().split('/').reverse()[0]; - await page.getByText('New Page').click(); + await newPage(page); const newPageId = page.url().split('/').reverse()[0]; expect(newPageId).not.toBe(originPageId); }); test('click btn bew page and find it in all pages', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page'); await page.getByRole('link', { name: 'All pages' }).click(); diff --git a/tests/local-first-openpage-newtab.spec.ts b/tests/local-first-openpage-newtab.spec.ts index b7e698345..0375bbed8 100644 --- a/tests/local-first-openpage-newtab.spec.ts +++ b/tests/local-first-openpage-newtab.spec.ts @@ -1,11 +1,11 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage } from './libs/page-logic'; loadPage(); -test.describe.skip('local first new page', () => { +test.describe('local first new page', () => { test('click btn bew page and open in tab', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page'); const newPageUrl = page.url(); diff --git a/tests/local-first-restore-page.spec.ts b/tests/local-first-restore-page.spec.ts index 8b3d2ccae..b75093143 100644 --- a/tests/local-first-restore-page.spec.ts +++ b/tests/local-first-restore-page.spec.ts @@ -1,13 +1,13 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage } from './libs/page-logic'; loadPage(); test.describe('Local first delete page', () => { test('New a page , then delete it in all pages, restore it', async ({ page, }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to restore'); const originPageUrl = page.url(); diff --git a/tests/local-first-show-delete-modal.spec.ts b/tests/local-first-show-delete-modal.spec.ts index a2cf040b7..902f89d53 100644 --- a/tests/local-first-show-delete-modal.spec.ts +++ b/tests/local-first-show-delete-modal.spec.ts @@ -1,11 +1,11 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage, clickPageMoreActions } from './libs/page-logic'; loadPage(); -test.describe.skip('Local first delete page', () => { +test.describe('Local first delete page', () => { test('New a page ,then open it and show delete modal', async ({ page }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to delete'); await page.getByRole('link', { name: 'All pages' }).click(); @@ -15,11 +15,7 @@ test.describe.skip('Local first delete page', () => { expect(cell).not.toBeUndefined(); await cell.click(); - await page - .getByTestId('editor-header-items') - .getByRole('button') - .nth(2) - .click(); + await clickPageMoreActions(page); const deleteBtn = page.getByTestId('editor-option-menu-delete'); await deleteBtn.click(); const confirmTip = page.getByText('Delete page?'); @@ -29,7 +25,7 @@ test.describe.skip('Local first delete page', () => { test('New a page ,then go to all pages and show delete modal', async ({ page, }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to delete'); const newPageId = page.url().split('/').reverse()[0]; diff --git a/tests/local-first-trash-page.spec.ts b/tests/local-first-trash-page.spec.ts index a25c4696b..093c51cc8 100644 --- a/tests/local-first-trash-page.spec.ts +++ b/tests/local-first-trash-page.spec.ts @@ -1,13 +1,13 @@ import { test, expect } from '@playwright/test'; import { loadPage } from './libs/load-page'; - +import { newPage } from './libs/page-logic'; loadPage(); -test.describe.skip('Local first trash page', () => { +test.describe('Local first trash page', () => { test('New a page , then delete it in all pages, finally find it in trash', async ({ page, }) => { - await page.getByText('New Page').click(); + await newPage(page); await page.getByPlaceholder('Title').click(); await page.getByPlaceholder('Title').fill('this is a new page to delete'); const newPageId = page.url().split('/').reverse()[0];