Skip to content

Add wso2.com_ed option to SAML metadata test#26879

Open
udeepa15 wants to merge 1 commit intowso2:masterfrom
udeepa15:oid4vp-testing
Open

Add wso2.com_ed option to SAML metadata test#26879
udeepa15 wants to merge 1 commit intowso2:masterfrom
udeepa15:oid4vp-testing

Conversation

@udeepa15
Copy link
Copy Markdown
Contributor

@udeepa15 udeepa15 commented Mar 3, 2026

Include the "wso2.com_ed" certificate alias in the certificateAlias.options array of the tenant SAML metadata test resource. This updates the test JSON (modules/integration/tests-integration/.../saml-metadata-tenant.json) to include an additional certificate alias option used by integration tests.

Summary by CodeRabbit

Tests

  • Updated test resources to include additional certificate alias configuration options for improved test coverage.

Include the "wso2.com_ed" certificate alias in the certificateAlias.options array of the tenant SAML metadata test resource. This updates the test JSON (modules/integration/tests-integration/.../saml-metadata-tenant.json) to include an additional certificate alias option used by integration tests.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a7a2bcf and a6d270d.

📒 Files selected for processing (1)
  • modules/integration/tests-integration/tests-backend/src/test/resources/org/wso2/identity/integration/test/rest/api/server/application/management/v1/saml-metadata-tenant.json

Walkthrough

A new certificate alias option ("wso2.com_ed") is added to the certificateAlias.options array in a SAML metadata tenant test resource file, expanding available configuration choices for testing.

Changes

Cohort / File(s) Summary
SAML Metadata Test Resources
modules/integration/tests-integration/tests-backend/src/test/resources/org/wso2/identity/integration/test/rest/api/server/application/management/v1/saml-metadata-tenant.json
Added "wso2.com_ed" to the certificateAlias.options array, providing an additional test certificate alias option alongside the existing "wso2.com" value.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisker twitch, a tail held high,
New aliases bloom, oh what a sigh!
"wso2.com_ed" joins the test array's dance,
One line added, a certificate's glance! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding a new certificate alias option to a SAML metadata test resource file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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 and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 3, 2026

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.

1 participant