Skip to content

Conversation

akafazov
Copy link
Contributor

@akafazov akafazov commented Oct 8, 2025

Add email user activation in mailpit between user registration and portal login steps.

refers to #168

- Implemented `activateUserEmail` function to automate email activation for newly registered users.
- Updated registration test to use dynamic user parameters for email and password.
- Added steps to log in to the portal after email activation and verify successful login.

On-behalf-of: @SAP [email protected]
Signed-off-by: Angel Kafazov <[email protected]>
@akafazov akafazov requested review from Copilot and nexus49 October 8, 2025 10:10
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the e2e Playwright tests for local setup by adding email user activation in Keycloak between user registration and portal login steps. The fix addresses test failures where users couldn't log in because their email wasn't verified in Keycloak.

  • Added activateUserEmail function to programmatically verify user emails in Keycloak admin interface
  • Refactored test to use variables for user data and added complete login flow after registration
  • Added new task in Taskfile.yaml to run portal e2e tests

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

File Description
local-setup/e2e/test-register-and-navigate.test.ts Added email activation function and complete user registration/login flow
Taskfile.yaml Added test:portal-e2e task to run e2e tests

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@akafazov akafazov requested a review from Copilot October 8, 2025 10:21
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

akafazov and others added 15 commits October 9, 2025 17:59
On-behalf-of: @SAP [email protected]
Signed-off-by: Angel Kafazov <[email protected]>
On-behalf-of: @SAP [email protected]
Signed-off-by: Angel Kafazov <[email protected]>
Copy link
Contributor

@nexus49 nexus49 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nexus49 nexus49 merged commit 3c45cd2 into main Oct 10, 2025
15 checks passed
@nexus49 nexus49 deleted the fix/e2e-tests branch October 10, 2025 09:50
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.

2 participants