chore: e2e: improve

Author: coderaiser

test-e2e/client/fixture/a.txt

@@ -6,11 +6,9 @@ test('Insert selects the current file', async ({page}) => {
     await page.goto('/');
 
     const rows = getRows(page);
-    const dotdot = rows.nth(0);
-    
-    await expect(dotdot).toHaveClass(/current-file/);
+    const first = rows.nth(0);
 
-    const first = rows.nth(1);
+    await expect(first).toHaveClass(/current-file/);
 
     await first.click();
     await page.keyboard.press('Insert');
@@ -33,7 +31,7 @@ test('Insert moves the cursor to the next file', async ({page}) => {
     await expect(second).toHaveClass(/current-file/);
 });
 
-test('Insert does not leave the cursor on the last file', async ({page}) => {
+test('Insert does not leave the cursor on the first file', async ({page}) => {
     await page.goto('/');
 
     const rows = getRows(page);
@@ -51,12 +49,14 @@ test('pressing Insert twice selects two consecutive files: first selected', asyn
     await page.goto('/');
 
     const rows = getRows(page);
-    const first = rows.nth(1);
-    const second = rows.nth(2);
+    const first = rows.nth(0);
+    const second = rows.nth(1);
 
     await first.click();
     await page.keyboard.press('Insert');
+    
     await expect(second).toHaveClass(/current-file/);
+    
     await page.keyboard.press('Insert');
 
     await expect(first).toHaveClass(/selected-file/);
@@ -71,23 +71,28 @@ test('pressing Insert twice selects two consecutive files: second selected', asy
 
     await first.click();
     await page.keyboard.press('Insert');
+    
+    await expect(second).toHaveClass(/current-file/);
+    
     await page.keyboard.press('Insert');
 
     await expect(second).toHaveClass(/selected-file/);
 });
 
-test('pressing Insert twice leaves cursor on the second file', async ({page}) => {
+test('pressing Insert twice leaves cursor on the third file', async ({page}) => {
     await page.goto('/');
 
     const rows = getRows(page);
+    const first = rows.nth(0);
     const second = rows.nth(1);
+    const third = rows.nth(2);
 
-    await rows
-        .nth(1)
-        .click();
-    
-    await page.keyboard.press('Insert');
+    await first.click();
     await page.keyboard.press('Insert');
 
     await expect(second).toHaveClass(/current-file/);
+    
+    await page.keyboard.press('Insert');
+    
+    await expect(third).toHaveClass(/current-file/);
 });

test-e2e/client/keys/insert.js

@@ -20,9 +20,9 @@ test('F3 on a .png file does not open the text viewer', async ({page}) => {
     await page.keyboard.press('F3');
 
     const modal = getModal(page);
-    const modalText = await modal.textContent();
+    await expect(modal).toBeVisible();
 
-    await expect(modalText).not.toContain('PNG');
+    await expect(modal).not.toContainText('PNG');
 });
 
 test('double-click on a .png file opens the image viewer', async ({page}) => {
@@ -38,9 +38,9 @@ test('double-click on a .png file does not open the text viewer', async ({page})
     await getFile(page, 'view.png').dblclick();
 
     const modal = getModal(page);
-    const modalText = await modal.textContent();
+    await expect(modal).toBeVisible();
 
-    await expect(modalText).not.toContain('PNG');
+    await expect(modal).not.toContainText('PNG');
 });
 
 test('F3 on a .txt file opens the text viewer', async ({page}) => {
@@ -59,5 +59,6 @@ test('F3 on a .txt file shows file content', async ({page}) => {
 
     const modal = getModal(page);
     await expect(modal).toBeVisible();
+    
     await expect(modal).not.toHaveText('');
 });