Add API Endpoint Test Cases and Mock Service Implementation#4
Merged
Nuvindu merged 23 commits intoballerina-platform:mainfrom Feb 22, 2025
Merged
Add API Endpoint Test Cases and Mock Service Implementation#4Nuvindu merged 23 commits intoballerina-platform:mainfrom
Nuvindu merged 23 commits intoballerina-platform:mainfrom
Conversation
Update openapi.json and sanitations.md to fix payload binding errors
Read, Create, Update and Delete endpoints of association definitions are tested.
Read, Create , Update and Delete endpoints for association definition configuration are tested
aaishahhamdha
requested review from
Nuvindu and
ThisaruGuruge
as code owners
Merge pull request ballerina-platform#3 from aaishahhamdha/main
Nuvindu
reviewed
Feb 20, 2025
Nuvindu
reviewed
Feb 20, 2025
Contributor
|
Commit messages are good! |
Used test:assertEquals() function when comparing two values instead of test:assertTrue()
Adds exact return type to the functions in place if 'var' type
Nuvindu
reviewed
Feb 20, 2025
Nuvindu
reviewed
Feb 20, 2025
Refactored function payloads to use specific Ballerina records instead of JSON payloads for better type safety and maintainability
Nuvindu
reviewed
Feb 22, 2025
Nuvindu
reviewed
Feb 22, 2025
Applied JSON syntax highlighting in markdown, and improved descriptions.
Nuvindu
reviewed
Feb 22, 2025
Improve code quality by limiting 120 characters per line
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request adds test scenarios, including negative test cases, for multiple API endpoints. It uses a mock service to simulate responses, ensuring the API works correctly in different situations and improving test coverage.
Examples
Checklist