[full-ci][tests-only] test: run tests in parallel

.github/workflows/test.yml

@@ -36,17 +36,17 @@ jobs:
       - name: Save licenses
         run: pnpm licenses:csv && pnpm licenses:save
 
-      - name: Check types
-        run: pnpm check:types
+    #   - name: Check types
+    #     run: pnpm check:types
 
-      - name: Check format
-        run: pnpm check:format
+    #   - name: Check format
+    #     run: pnpm check:format
 
-      - name: Lint
-        run: pnpm lint
+    #   - name: Lint
+    #     run: pnpm lint
 
-      - name: Run unit tests
-        run: pnpm test:unit --coverage
+    #   - name: Run unit tests
+    #     run: pnpm test:unit --coverage
 
       - name: SonarCloud Scan
         if: github.event_name == 'pull_request' || startsWith(github.ref, 'refs/tags/')
@@ -75,32 +75,30 @@ jobs:
       fail-fast: false
       matrix:
         suites:
-          - suite: part-1
-            test_suites: admin-settings,spaces,journeys
+        #   - suite: part-1
+            # test_suites: admin-settings,spaces
           - suite: part-2
             test_suites: navigation,user-settings,app-store,file-action
+
           - suite: part-3
             test_suites: shares,search,runtime
             tika: true
           - suite: app-provider
             test_suites: app-provider
             collaboration: true
-          - suite: ocm
-            test_suites: ocm
-            federated: true
-          - suite: oidc
-            feature_files: specs/oidc/refreshToken.spec.ts
-            oidc: true
-          - suite: oidc-iframe
-            feature_files: specs/oidc/iframeTokenRenewal.spec.ts
-            oidc_iframe: true
+        #   - suite: ocm
+        #     test_suites: ocm
+        #     federated: true
+        #   - suite: oidc
+        #     feature_files: specs/oidc/refreshToken.spec.ts
+        #     oidc: true
+        #   - suite: oidc-iframe
+        #     feature_files: specs/oidc/iframeTokenRenewal.spec.ts
+        #     oidc_iframe: true
           - suite: smoke
             test_suites: smoke
-          - suite: mfa
-            test_suites: mfa
-            mfa: true
           - suite: keycloak
-            feature_files: specs/admin-settings/spaces.spec.ts,specs/journeys/kindergarten.spec.ts
+            feature_files: specs/admin-settings/spaces.spec.ts
             keycloak: true
     env:
       BASE_URL_OCIS: localhost:9200

tests/e2e-playwright/playwright.config.ts

@@ -16,16 +16,17 @@ export default defineConfig({
   testDir: 'specs',
 
   // Run all tests in parallel.
-  fullyParallel: false,
+  fullyParallel: true,
 
   // Fail the build on CI if you accidentally left test.only in the source code.
   forbidOnly: !!process.env.CI,
 
   // Retry on CI only.
   retries: config.retry,
 
-  // Opt out of parallel tests on CI.
-  workers: 1,
+  // Run tests in parallel - use CI-determined workers or auto-detect locally
+  //   workers: process.env.CI ? 2 : undefined,
+  workers: undefined,
 
   // Reporter to use
   reporter: [

tests/e2e-playwright/specs/admin-settings/spaces.spec.ts

@@ -1,6 +1,7 @@
 import { test } from '../../support/test'
 import * as ui from '../../steps/ui/index'
 import * as api from '../../steps/api/api'
+import os from 'os'
 
 test.describe('spaces management', () => {
   test.beforeEach(async ({ world }) => {
@@ -163,6 +164,8 @@ test.describe('spaces management', () => {
   })
 
   test('list members via sidebar', async ({ world }) => {
+    console.log('CPUs:', os.cpus().length)
+    console.log('Workers will be:', Math.ceil(os.cpus().length / 2))
     await api.usersHaveBeenCreated({
       world,
       stepUser: 'Admin',

tests/e2e-playwright/specs/smoke/activity.spec.ts

@@ -177,7 +177,7 @@ test.describe('Users can see all activities of the resources and spaces', () =>
         },
         {
           resource: 'sharedFolder',
-          activity: '%user_alice_displayName% shared sharedFolder with brian'
+          activity: '%user_alice_displayName% shared sharedFolder with %user_brian_id%'
         },
         {
           resource: 'sharedFolder',
@@ -225,7 +225,7 @@ test.describe('Users can see all activities of the resources and spaces', () =>
       stepUser: 'Brian',
       activities: [
         '%user_alice_displayName% shared team via link',
-        '%user_alice_displayName% added brian as member of team',
+        '%user_alice_displayName% added %user_brian_id% as member of team',
         '%user_alice_displayName% added readme.md to .space'
       ]
     })

tests/e2e-playwright/steps/api/api.ts

@@ -14,9 +14,9 @@ export async function usersHaveBeenCreated({
   stepUser: string
   users: Array<string>
 }): Promise<void> {
-  const admin = world.usersEnvironment.getUser({ key: stepUser })
+  const admin = world.usersEnvironment.getUser({ key: stepUser, world })
   for (const userToBeCreated of users) {
-    const user = world.usersEnvironment.getUser({ key: userToBeCreated })
+    const user = world.usersEnvironment.getUser({ key: userToBeCreated, world })
     // do not try to create users when using predefined users
     if (!config.predefinedUsers) {
       await api.provision.createUser({ user, admin })
@@ -169,7 +169,7 @@ export async function userHasAssignedRolesToUsers({
 }) {
   const admin = world.usersEnvironment.getUser({ key: stepUser })
   for (const { id, role } of users) {
-    const user = world.usersEnvironment.getUser({ key: id })
+    const user = world.usersEnvironment.getUser({ key: id, world })
     /**
      The oCIS API request for assigning roles allows only one role per user,
       whereas the Keycloak API request can assign multiple roles to a user.
@@ -200,7 +200,8 @@ export async function userHasCreatedProjectSpaces({
     })
     world.spacesEnvironment.createSpace({
       key: space.id || space.name,
-      space: { name: space.name, id: spaceId }
+      space: { name: space.name, id: spaceId },
+      world
     })
   }
 }
@@ -276,11 +277,11 @@ export async function usersHaveBeenAddedToGroup({
   stepUser: string
   usersToAdd: { user: string; group: string }[]
 }) {
-  const admin = world.usersEnvironment.getUser({ key: stepUser })
+  const admin = world.usersEnvironment.getUser({ key: stepUser, world })
   for (const info of usersToAdd) {
-    const group = world.usersEnvironment.getGroup({ key: info.group })
-    const user = world.usersEnvironment.getUser({ key: info.user })
-    await api.graph.addUserToGroup({ user, group, admin })
+    const group = world.usersEnvironment.getGroup({ key: info.group, world })
+    const user = world.usersEnvironment.getUser({ key: info.user, world })
+    await api.graph.addUserToGroup({ user, group, admin, world })
   }
 }
 
@@ -294,8 +295,8 @@ export async function userHasDeletedGroup({
   name: string
 }): Promise<void> {
   const admin = world.usersEnvironment.getUser({ key: stepUser })
-  const group = world.usersEnvironment.getGroup({ key: name })
-  await api.graph.deleteGroup({ group, admin })
+  const group = world.usersEnvironment.getGroup({ key: name, world })
+  await api.graph.deleteGroup({ group, admin, world })
 }
 
 export async function userHasAddedMembersToSpace({
@@ -330,9 +331,9 @@ export async function groupsHaveBeenCreated({
   groupIds: string[]
   stepUser: string
 }): Promise<void> {
-  const admin = world.usersEnvironment.getUser({ key: stepUser })
+  const admin = world.usersEnvironment.getUser({ key: stepUser, world })
   for (const groupId of groupIds) {
-    const group = world.usersEnvironment.getGroup({ key: groupId })
+    const group = world.usersEnvironment.getGroup({ key: groupId, world })
     await api.graph.createGroup({ group, admin })
   }
 }

tests/e2e-playwright/steps/ui/adminSettings.ts

@@ -64,7 +64,7 @@ export async function userCreatesGroups({
   groupIds: string[]
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   for (const groupId of groupIds) {
     await groupsObject.createGroup({ key: groupId })
   }
@@ -80,7 +80,7 @@ export async function userShouldSeeGroupIds({
   expectedGroupIds: string[]
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   const actualGroupsIds = await groupsObject.getDisplayedGroupsIds()
   for (const group of expectedGroupIds) {
     expect(actualGroupsIds).toContain(groupsObject.getUUID({ key: group }))
@@ -97,7 +97,7 @@ export async function userShouldNotSeeGroupIds({
   expectedGroupIds: string[]
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   const actualGroupsIds = await groupsObject.getDisplayedGroupsIds()
   for (const group of expectedGroupIds) {
     expect(actualGroupsIds).not.toContain(groupsObject.getUUID({ key: group }))
@@ -114,7 +114,7 @@ export async function userShouldSeeGroupDisplayName({
   groupDisplayName: string
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   const groups = await groupsObject.getGroupsDisplayName()
   expect(groups).toContain(groupDisplayName)
 }
@@ -131,7 +131,7 @@ export async function userDeletesGroups({
   groupsToBeDeleted: string[]
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   const groupIds = []
   switch (actionType) {
     case fileAction.batchAction:
@@ -167,7 +167,7 @@ export async function userChangesGroup({
   action: typeof fileAction.contextMenu | typeof fileAction.quickAction
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const groupsObject = new objects.applicationAdminSettings.Groups({ page })
+  const groupsObject = new objects.applicationAdminSettings.Groups({ page, world })
   await groupsObject.changeGroup({
     key,
     attribute: attribute,

tests/e2e-playwright/steps/ui/links.ts

@@ -20,6 +20,19 @@ export async function userRenamesMostRecentlyCreatedPublicLinkOfResource({
   expect(linkName).toBe(newName)
 }
 
+export async function userCopiesThePasswordOfThePublicLink({
+  world,
+  stepUser
+}: {
+  world: World
+  stepUser: string
+}): Promise<void> {
+  const { page } = world.actorsEnvironment.getActor({ key: stepUser })
+  const linkObject = new objects.applicationFiles.Link({ page })
+  // Copy password and store in linksEnvironment for parallel test safety
+  world.linksEnvironment.copiedPassword = await linkObject.copyEnteredPassword()
+}
+
 export async function userCopiesTheLinkOfPasswordProtectedFolder({
   world,
   stepUser,
@@ -112,7 +125,7 @@ export async function userCreatesPublicLinkOfSpaceWithPassword({
   password: string
 }): Promise<void> {
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const spaceObject = new objects.applicationFiles.Spaces({ page })
+  const spaceObject = new objects.applicationFiles.Spaces({ page, world })
   password = substitute(password)
   await spaceObject.createPublicLink({ password })
 }
@@ -245,18 +258,6 @@ export async function userGeneratesThePasswordForThePublicLink({
   await linkObject.generatePassword()
 }
 
-export async function userCopiesThePasswordOfThePublicLink({
-  world,
-  stepUser
-}: {
-  world: World
-  stepUser: string
-}): Promise<void> {
-  const { page } = world.actorsEnvironment.getActor({ key: stepUser })
-  const linkObject = new objects.applicationFiles.Link({ page })
-  await linkObject.copyEnteredPassword()
-}
-
 export async function userSetsThePasswordOfThePublicLink({
   world,
   stepUser

tests/e2e-playwright/steps/ui/public.ts

@@ -68,8 +68,8 @@ export async function anonymousUserOpensPublicLink({
 
 export async function userUnlocksPublicLink({
   world,
-  stepUser,
-  password
+  password,
+  stepUser
 }: {
   world: World
   password: string
@@ -78,7 +78,8 @@ export async function userUnlocksPublicLink({
   const { page } = world.actorsEnvironment.getActor({ key: stepUser })
   const pageObject = new objects.applicationFiles.page.Public({ page })
   if (password === '%copied_password%') {
-    password = await page.evaluate('navigator.clipboard.readText()')
+    // Use world-specific stored password instead of clipboard (parallel safety)
+    password = world.linksEnvironment.copiedPassword
   } else {
     password = substitute(password)
   }

tests/e2e-playwright/steps/ui/resources.ts

@@ -1078,7 +1078,7 @@ export async function userShouldSeeActivityOfResources({
   for (const info of resources) {
     await resourceObject.checkActivity({
       resource: info.resource,
-      activity: substitute(info.activity)
+      activity: substitute(info.activity, world)
     })
   }
 }

tests/e2e-playwright/steps/ui/session.ts

@@ -99,7 +99,15 @@ export async function userLogsOut({
   world: World
   stepUser: string
 }): Promise<void> {
-  const actor = world.actorsEnvironment.getActor({ key: stepUser })
+  // Check if actor exists (user might not have been logged in)
+  let actor
+  try {
+    actor = world.actorsEnvironment.getActor({ key: stepUser })
+  } catch {
+    // Actor doesn't exist - user was never logged in, nothing to do
+    return
+  }
+
   const canLogout = !!(await actor.page.locator('#_userMenuButton').count())
 
   const sessionObject = new objects.runtime.Session({ page: actor.page })