Skip to content

[MOSIP-45706] Let localhost admin UI tests sign in with the admin token - #868

Open
jayesh12234 wants to merge 6 commits into
mosip:developfrom
jayesh12234:develop
Open

jayesh12234 wants to merge 6 commits into
mosip:developfrom
jayesh12234:develop

Conversation

@jayesh12234

Copy link
Copy Markdown
Contributor

Summary

  • Localhost UI automation now signs in with the admin token from the internal auth API, because Keycloak rejects the localhost redirect (KER-ACP-009).
  • Audit log failures no longer open the blocking Technical Error dialog.
  • Test dates and dummy text are generated per run, IDE resource paths resolve without path.config, and device_master cleanup runs for the test user. Kernel.properties and config.json target dev1.

Jira: https://mosip.atlassian.net/browse/MOSIP-45706

Test plan

  • Run the UI suite with adminPortalPath=http://localhost:4200/ and ng serve running. The portal should open without the Keycloak error page and without the Technical Error dialog.
  • Run the same suite against the deployed admin portal and confirm the Keycloak login path is unchanged.

Made with Cursor

Keycloak rejects the localhost redirect, and failed audit calls were opening a blocking dialog. Tests now attach the admin session cookie on localhost and ignore audit log errors.

Signed-off-by: jayesh12234 <111957576+jayesh12234@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI; Enforced overrides: Organization
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: dc92a75f-1e01-4c1e-b8d7-c5ac0c32f466
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

kharodejayesh and others added 5 commits October 22, 2026 18:19
CodeQL treated ThreadLocalRandom values stored in the shared properties map as the source for the S3 account and IAM password getters.

Co-authored-by: Cursor <cursoragent@cursor.com>
Client secrets, passwords, and S3 credentials are left blank so they are supplied from the environment instead of the repository.

Co-authored-by: Cursor <cursoragent@cursor.com>
These files are not part of the localhost test sign-in change.

Co-authored-by: Cursor <cursoragent@cursor.com>
Chrome blocks the cross-origin call from port 4200, and the session cookie must be stored on the masterdata host the UI actually calls.

Signed-off-by: Jayesh Kharode <jayesh.kharode@technoforte.co.in>
Co-authored-by: Cursor <cursoragent@cursor.com>
Local runs were clicking menus, dates, and save before the page finished rendering.

Signed-off-by: Jayesh Kharode <jayesh.kharode@technoforte.co.in>
Co-authored-by: Cursor <cursoragent@cursor.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.

2 participants