move hovering into save fn

dzonidoo · Aug 1, 2024 · 6e7cbf7 · 6e7cbf7
1 parent 9a8fa14
commit 6e7cbf7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/e2e/client/playwright/multiedit.spec.ts b/e2e/client/playwright/multiedit.spec.ts
@@ -1,13 +1,13 @@
 import {test, expect} from '@playwright/test';
 import {Monitoring} from './page-object-models/monitoring';
 import {Authoring} from './page-object-models/authoring';
-import {Multiedit} from './page-object-models/multiedit';
+import {MultiEdit} from './page-object-models/multiedit';
 import {restoreDatabaseSnapshot, s} from './utils';
 
 test.describe('Multiedit', async () => {
     test('editing articles in multi-edit mode', async ({page}) => {
         const monitoring = new Monitoring(page);
-        const multiedit = new Multiedit(page);
+        const multiedit = new MultiEdit(page);
 
         await restoreDatabaseSnapshot();
         await page.goto('/#/workspace/monitoring');
@@ -23,7 +23,7 @@ test.describe('Multiedit', async () => {
             .locator(s('multiedit-screen', 'multiedit-article=test sports story', 'field--headline'))
             .getByRole('textbox')
             .fill('test sports story 1.1');
-        await page.locator(s('multiedit-screen', 'multiedit-article=test sports story')).hover();
+
         await multiedit.save('test sports story');
 
         await page
@@ -34,7 +34,7 @@ test.describe('Multiedit', async () => {
             .locator(s('multiedit-screen', 'multiedit-article=story 2', 'field--headline'))
             .getByRole('textbox')
             .fill('story 2.1');
-        await page.locator(s('multiedit-screen', 'multiedit-article=story 2')).hover();
+
         await multiedit.save('story 2');
 
         await page.locator(s('multiedit-subnav')).getByRole('button', {name: 'exit'}).click();

diff --git a/e2e/client/playwright/page-object-models/multiedit.ts b/e2e/client/playwright/page-object-models/multiedit.ts
@@ -1,14 +1,16 @@
 import {Page} from '@playwright/test';
 import {s} from '../utils';
 
-export class Multiedit {
+export class MultiEdit {
     private page: Page;
 
     constructor(page: Page) {
         this.page = page;
     }
 
     async save(article: string): Promise<void> {
+        await this.page.locator(s('multiedit-screen', `multiedit-article=${article}`)).hover();
+
         await this.page
             .locator(s('multiedit-screen', `multiedit-article=${article}`))
             .getByRole('button', {name: 'save'})