Skip to content

ci: [OCISDEV-734] run playwright E2E tests#13654

Merged
LukasHirt merged 2 commits intomasterfrom
ci/e2e-tests
Apr 21, 2026
Merged

ci: [OCISDEV-734] run playwright E2E tests#13654
LukasHirt merged 2 commits intomasterfrom
ci/e2e-tests

Conversation

@LukasHirt
Copy link
Copy Markdown
Collaborator

@LukasHirt LukasHirt commented Mar 27, 2026

Description

Run playwright E2E tests in the GH actions.

Motivation and Context

Drone is gone -> GH actions are the way.

How Has This Been Tested?

  • test environment: 🤖

@LukasHirt LukasHirt self-assigned this Mar 27, 2026
@update-docs

This comment was marked as off-topic.

@LukasHirt LukasHirt changed the title Ci/e2e tests ci: run playwright E2E tests Mar 27, 2026
@LukasHirt LukasHirt changed the title ci: run playwright E2E tests ci: [OCISDEV-734] run playwright E2E tests Mar 31, 2026
@LukasHirt
Copy link
Copy Markdown
Collaborator Author

LukasHirt commented Apr 13, 2026

Strange behaviour with the OCM test suite. The test works fine until it reaches the role selection where it clicks on the role dropdown but nothing seems to happen:

image

There are also strange errors in the console:

[ERR] Failed to get user from created list. Error: user with key 'brian' not found
    at UsersEnvironment.getCreatedUser (file:///home/runner/work/web/web/tests/e2e/support/environment/userManagement.ts:51:13)
    at getValue (file:///home/runner/work/web/web/tests/e2e/support/utils/substitute.ts:18:35)
    at substitute (file:///home/runner/work/web/web/tests/e2e/support/utils/substitute.ts:39:21)
    at Module.userShouldSeeTheFederatedConnections (file:///home/runner/work/web/web/tests/e2e-playwright/steps/ui/federation.ts:46:17)
    at file:///home/runner/work/web/web/tests/e2e-playwright/specs/ocm/ocm.spec.ts:107:14
    at /home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/workerMain.js:310:9
    at /home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/testInfo.js:305:11
    at TimeoutManager.withRunnable (/home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/timeoutManager.js:67:14)
    at TestInfoImpl._runWithTimeout (/home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/testInfo.js:303:7)
    at /home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/workerMain.js:308:7
    at WorkerMain._runTest (/home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/workerMain.js:283:5)
    at WorkerMain.runTestGroup (/home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/worker/workerMain.js:198:11)
    at process.<anonymous> (/home/runner/work/web/web/node_modules/.pnpm/playwright@1.56.1/node_modules/playwright/lib/common/process.js:65:22)

@LukasHirt
Copy link
Copy Markdown
Collaborator Author

The OCM test still fails. I run the test locally and it passes so it is definitely due to the new CI. Also, the user errors are actually expected. I found a comment there mentioning so.

@LukasHirt LukasHirt force-pushed the ci/e2e-tests branch 2 times, most recently from 32a43a8 to 55f9067 Compare April 16, 2026 08:44
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 17, 2026

CLA assistant check
All committers have signed the CLA.

Comment thread tests/drone/providers.json
@LukasHirt LukasHirt marked this pull request as ready for review April 20, 2026 09:39
@LukasHirt
Copy link
Copy Markdown
Collaborator Author

@nirajacharya2 @saw-jan could you please review the test changes? They are on purpose contained within own commit (the second one) so that you can easily check that.

@sonarqubecloud
Copy link
Copy Markdown

@LukasHirt LukasHirt merged commit f1bb395 into master Apr 21, 2026
15 checks passed
@LukasHirt LukasHirt deleted the ci/e2e-tests branch April 21, 2026 07:41
nirajacharya2 pushed a commit that referenced this pull request Apr 21, 2026
---------

Co-authored-by: Michal Klos <michal.klos@kiteworks.com>
LukasHirt added a commit that referenced this pull request Apr 22, 2026
---------

Co-authored-by: Michal Klos <michal.klos@kiteworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants