Skip to content

feat: add comprehensive API test coverage#188

Merged
Kaushl2208 merged 1 commit intofossology:mainfrom
krishi-agrawal:test-coverage
Feb 18, 2026
Merged

feat: add comprehensive API test coverage#188
Kaushl2208 merged 1 commit intofossology:mainfrom
krishi-agrawal:test-coverage

Conversation

@krishi-agrawal
Copy link
Copy Markdown
Contributor

Changes

  • Add tests for all license endpoints (filter, export, import, similarity, search)
  • Add tests for all user endpoints (CRUD, profile, refresh token)
  • Add tests for all obligation endpoints (CRUD, audits, export, import, similarity)
  • Add tests for obligation maps, dashboard, OIDC clients, and health endpoints
  • Fix refresh token sub claim UUID formatting
  • Fix OIDC client deletion nil pointer bug
  • Fixes Increase test coverage #180

Submitter Checklist

  • Includes tests (if there is a feature changed/added)
  • Includes docs ( if changes are user facing)
  • I have tested my changes locally.

References

@krishi-agrawal
Copy link
Copy Markdown
Contributor Author

i made the required changes @ChayanDass ! PTAL.

@krishi-agrawal
Copy link
Copy Markdown
Contributor Author

I made the required changes. PTAL, @deo002 .

Copy link
Copy Markdown
Collaborator

@deo002 deo002 left a comment

Choose a reason for hiding this comment

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

Changes look good! Working as expected!

@deo002
Copy link
Copy Markdown
Collaborator

deo002 commented Feb 17, 2026

Just squash your commits into a single commit and follow the conventional commit guidlines and I'll merge it.

@krishi-agrawal krishi-agrawal force-pushed the test-coverage branch 2 times, most recently from 2e6e386 to d5061e3 Compare February 17, 2026 17:06
Signed-off-by: Krishi Agrawal <krishi.agrawal26@gmail.com>
@krishi-agrawal
Copy link
Copy Markdown
Contributor Author

done!

@deo002
Copy link
Copy Markdown
Collaborator

deo002 commented Feb 18, 2026

Perfect! Thanks for your contributions!

Copy link
Copy Markdown
Member

@Kaushl2208 Kaushl2208 left a comment

Choose a reason for hiding this comment

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

LGTM

@Kaushl2208 Kaushl2208 merged commit 97ec5a7 into fossology:main Feb 18, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increase test coverage

4 participants