Skip to content

T4: Finalized API testing and CI, Fixed checkstyle and minor bugs, Updated documentation#69

Merged
ggracelii merged 4 commits intomainfrom
api-testing-bug-fixing-documentation
Dec 5, 2025
Merged

T4: Finalized API testing and CI, Fixed checkstyle and minor bugs, Updated documentation#69
ggracelii merged 4 commits intomainfrom
api-testing-bug-fixing-documentation

Conversation

@jag2430
Copy link
Collaborator

@jag2430 jag2430 commented Dec 4, 2025

Finalized automated API testing, CI pipeline, and project documentation.

Changes

API Testing & CI

  • Expanded Postman/Newman API tests to cover all endpoints with equivalence partitions
  • Fixed variable persistence issue (pm.collectionVariablespm.environment) for Newman compatibility
  • Verified CI pipeline runs all unit, integration, and API tests automatically

Bug Fixes

  • Updated deleteTransactionViaGet to return proper HTTP status codes (404/500) instead of 200 with error text
  • Fixed zero budget update logic in updateUserJson
  • Resolved checkstyle violations

Documentation

  • Updated api-testing.md with equivalence partitions documentation
  • Expanded README with CI instructions, third-party client guide, and multiple client handling
  • Added Javadoc comments where needed

Unit Tests

  • Updated RouteControllerTests for new deleteTransactionViaGet behavior
  • Expanded test coverage ti ensure coverage for both valid and invalid equivalence partitions for each input

@jag2430 jag2430 requested review from chrvn, ggracelii and hc8756 December 5, 2025 00:47
@jag2430 jag2430 changed the title Fixed minor logic error in RouteController, improved javadoc comments… T4: Finalized API testing and CI, Fixed checkstyle and minor bugs, Updated documentation Dec 5, 2025
@ggracelii ggracelii merged commit 22d5bef into main Dec 5, 2025
6 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