Skip to content

Conversation

@DimalshaMadushani
Copy link
Contributor

Purpose

Add unit tests for the first set of files in the server/actions folder of @asgardeo/nextjs.

Also noted that in getBrandingPreference.ts, the variable sessionId is defined but not used — this might be an implementation oversight.

Next steps:
I plan to continue adding test coverage for the remaining files in the @asgardeo/nextjs SDK to complete the coverage improvement initiative.


Related Issues

Related PRs

  • N/A

Checklist

  • Followed the CONTRIBUTING guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)

Security checks

@asgardeo-github-bot
Copy link

⚠️ No Changeset found

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go.

If these changes should result in a version bump, you need to add a changeset.

Refer Release Documentation to learn how to add a changeset.

@brionmario brionmario merged commit 4b9ddff into asgardeo:main Oct 24, 2025
3 of 5 checks passed
@brionmario brionmario added the hacktoberfest-accepted Label required by the Hacktoberfest participating PRs to be listed on the user's profile label Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted Label required by the Hacktoberfest participating PRs to be listed on the user's profile

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants