feat(#1349729): add tags to the test as role function and add makefile e2e tag filter

Author: kerwan

Makefile

@@ -144,9 +144,10 @@ phpunit: ## Run php unit tests, pass the parameter "p=" to launch tests on a spe
 jest: ## Run jest unit tests
 	@$(DOCKER_COMP) exec pwa yarn test
 
-e2e: ## Run e2e tests or pass the parameter "f=" to filters tests, example: make e2e f=login
+e2e: ## Run e2e tests or pass the parameter "f=" to filters tests and "t=" to run only certain tags, example: make e2e f=login t=standard
 	@$(eval f ?=)
-	@$(DOCKER_COMP) exec e2e yarn test $(f)
+	@$(eval t ?=)
+	@$(DOCKER_COMP) exec e2e yarn test $(f) $(if $(t),--grep $(t))
 
 jest_update: ## Run jest unit tests
 	@$(DOCKER_COMP) exec pwa yarn test:update

front/e2e/src/helper/auth.ts

@@ -1,4 +1,4 @@
-import {Page, expect, test} from '@playwright/test'
+import { Page, expect, test, TestDetails } from '@playwright/test'
 import {TestId, generateTestId} from "./testIds";
 
 const testIds = {
@@ -31,13 +31,22 @@ export enum UserRole {
   CONTRIBUTOR = 'contributor',
 }
 
+type TestDetailsWithRequiredTag = TestDetails & Required<Pick<TestDetails, 'tag'>>
+
 export function runTestsAsRoles(
   roles: UserRole[],
+  details: TestDetailsWithRequiredTag,
   callback: (page: Page, role?: UserRole) => Promise<void>
 ): void {
+  // Adds the @multirole to the existing tag, these tests can be slower than the others
+  // So it might be useful to have an easy way to include/exclude them from a run
+  const detailsWithMultiroleTag: TestDetailsWithRequiredTag  = {
+    ...details,
+    tag: [...new Set([...details.tag, '@multiroles'])]
+  }
   for (const role of roles) {
     // Instantiate an isolated test context for each role
-    test(`Test as ${role} role`, async ({ page }) => {
+    test(`Test as ${role} role`, detailsWithMultiroleTag, async ({ page }) => {
       await login(page, role)
       // Run callback with a page authenticated with the specified role
       await callback(page, role)

front/e2e/src/tests/pages/admin/settings/scope.test.ts

@@ -185,7 +185,7 @@ test.beforeEach(async ({ context }) => {
 });
 
 test.describe('Parameters Page', () => {
-  runTestsAsRoles([UserRole.ADMIN, UserRole.CONTRIBUTOR], async (page: Page) => {
+  runTestsAsRoles([UserRole.ADMIN, UserRole.CONTRIBUTOR], { tag: ['@premium', '@standard'] }, async (page: Page) => {
     await testParametersPage(page)
   })
 })