Skip to content

Add unit tests for API validation (#42)#80

Open
Nakshatra480 wants to merge 1 commit intohyphae:add-license-1from
Nakshatra480:fix/issue-42-api-tests
Open

Add unit tests for API validation (#42)#80
Nakshatra480 wants to merge 1 commit intohyphae:add-license-1from
Nakshatra480:fix/issue-42-api-tests

Conversation

@Nakshatra480
Copy link

Hey! This adds some basic tests to cover the core API functionality - mainly checking that all the service address routes are correct, deal operations work as expected, and error handling is solid.

Changes

  • Added ApiValidationTest.java with 9 tests covering:
    • Route address validation (ServiceAddress methods)
    • Controller/User service routes
    • Deal ID generation and attributes
    • Error enum parsing and object creation

Testing

All tests pass locally

Fixes hyphae#42

Signed-off-by: Nakshatra Sharma <nakshatra.sharma3012@gmail.com>
@Nakshatra480 Nakshatra480 force-pushed the fix/issue-42-api-tests branch from b135897 to 88bf22a Compare January 26, 2026 10:20
@Nakshatra480
Copy link
Author

@axmsoftware @AkarshSahlot could you please review this PR when you have time?

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