From 78cb3e693ccf40a6242ba1723acac88e7d23023d Mon Sep 17 00:00:00 2001
From: Quentin BEY <quentin.bey@mail.numerique.gouv.fr>
Date: Fri, 9 May 2025 17:12:35 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=92=9A(e2e)=20remove=20useless=20test=20o?=
 =?UTF-8?q?n=20"tabs"?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This is supposed to be a validation test on accessibility but in fact
it's just a flaky test which does not provide any information. We need
to replace this with something smarter.
---
 .../app-desk/keyboard-navigation.spec.ts      | 77 -------------------
 1 file changed, 77 deletions(-)
 delete mode 100644 src/frontend/apps/e2e/__tests__/app-desk/keyboard-navigation.spec.ts

diff --git a/src/frontend/apps/e2e/__tests__/app-desk/keyboard-navigation.spec.ts b/src/frontend/apps/e2e/__tests__/app-desk/keyboard-navigation.spec.ts
deleted file mode 100644
index a832076..0000000
--- a/src/frontend/apps/e2e/__tests__/app-desk/keyboard-navigation.spec.ts
+++ /dev/null
@@ -1,77 +0,0 @@
-import { Page, expect, test } from '@playwright/test';
-
-import { keyCloakSignIn } from './common';
-
-const payloadGetTeams = {
-  count: 4,
-  next: null,
-  previous: null,
-  results: [
-    {
-      id: 'b2224958-ac22-4863-9330-4322add64ebe',
-      name: 'Test Group',
-      accesses: ['1a1ba2ba-a58c-4593-8adb-1da6ee9cd3a3'],
-      abilities: {
-        get: true,
-        patch: true,
-        put: true,
-        delete: true,
-        manage_accesses: true,
-      },
-      slug: 'test-group',
-      created_at: '2024-07-17T19:41:11.404763Z',
-      updated_at: '2024-07-17T19:41:11.404763Z',
-    },
-  ],
-};
-
-const mockApiRequests = async (page: Page) => {
-  await page.route('**/teams/?ordering=-created_at', async (route) => {
-    await route.fulfill({
-      json: payloadGetTeams.results,
-    });
-  });
-};
-
-test.describe('Keyboard navigation', () => {
-  test('navigates through all focusable elements from top to bottom on groups index view when one group exists', async ({
-    browser,
-    browserName,
-  }) => {
-    const page = await browser.newPage();
-
-    await mockApiRequests(page);
-
-    await page.goto('/');
-    await keyCloakSignIn(page, browserName, 'team-owner-mail-member');
-
-    await page.waitForURL('http://localhost:3000/teams/');
-
-    const header = page.locator('header');
-
-    // necessary to begin the keyboard navigation directly from first button on the app and only select its elements
-    await header.click();
-
-    const focusableElements = await page
-      .locator(
-        [
-          '.c__app a:not([tabindex="-1"]):visible',
-          '.c__app button:not([tabindex="-1"]):visible',
-          '.c__app input:not([disabled]):not([tabindex="-1"]):visible',
-          '.c__app select:not([disabled]):not([tabindex="-1"]):visible',
-          '.c__app textarea:not([disabled]):not([tabindex="-1"]):visible',
-          '.c__app [tabindex]:not([tabindex="-1"]):visible',
-        ].join(', '),
-      )
-      .all();
-
-    expect(focusableElements.length).toEqual(19);
-
-    for (let i = 0; i < focusableElements.length; i++) {
-      await page.keyboard.press('Tab');
-      // wait for the element to be visible (page scrolls)
-      await focusableElements[i].scrollIntoViewIfNeeded();
-      await expect(focusableElements[i]).toBeFocused();
-    }
-  });
-});