Skip to content

W-20878414 SDK unit tests for auth, clients and operations#40

Merged
charithaT07 merged 3 commits intomainfrom
W-20878414-Sdk-Unit-Tests-auth-clients
Jan 14, 2026
Merged

W-20878414 SDK unit tests for auth, clients and operations#40
charithaT07 merged 3 commits intomainfrom
W-20878414-Sdk-Unit-Tests-auth-clients

Conversation

@charithaT07
Copy link
Copy Markdown
Collaborator

@charithaT07 charithaT07 commented Jan 13, 2026

Summary

Brief description of what this PR does.

This PR contains the units tests for below modules within tooling-sdk:

  • src/auth
  • src/clients
  • src/operations

Testing

How was this tested?


  • Tests pass (pnpm test)
  • Code is formatted (pnpm run format)

@charithaT07 charithaT07 marked this pull request as ready for review January 13, 2026 17:19
@charithaT07 charithaT07 requested a review from clavery as a code owner January 13, 2026 17:19

describe('auth/types', () => {
it('exports ALL_AUTH_METHODS in default priority order', () => {
expect(ALL_AUTH_METHODS).to.deep.equal(['client-credentials', 'implicit', 'basic', 'api-key']);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: this one is a little brittle since it will change

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks . I’ll follow this up in the next PR for expanding unit tests for other CLI components.

Copy link
Copy Markdown
Collaborator

@clavery clavery left a comment

Choose a reason for hiding this comment

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

lgtm

@charithaT07 charithaT07 merged commit 44e88ad into main Jan 14, 2026
2 checks passed
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