test: increased test coverage for API Endpoints#802
Closed
IshwaryaSriram wants to merge 3 commits intofinos:mainfrom
Closed
test: increased test coverage for API Endpoints#802IshwaryaSriram wants to merge 3 commits intofinos:mainfrom
IshwaryaSriram wants to merge 3 commits intofinos:mainfrom
Conversation
|
✅ Deploy Preview for endearing-brigadeiros-63f9d0 canceled.
|
Contributor
|
Hi @IshwaryaSriram thank you for the contribution! Please remember to sign the CLA so this PR can be eventually merged. |
Contributor
|
Hmm... Two of the tests are failing in CI. Can you review the logs and resolve the failures? https://github.com/finos/git-proxy/actions/runs/13342403098/job/37268710992?pr=802 |
Contributor
|
@IshwaryaSriram Thanks for the contribution! This PR has been superseded by various PRs that increased project coverage, such as #1038. Some extra coverage for the API is attained in #1356. Closing this to clear up the backlog - feel free to open a new PR to improve on top of #1356. |
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
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.
This PR addresses #240 and adds unit test cases, ensuring at least 80% test coverage for src/service/routes/*
Note:
Please review the PR and let me know if there are any updates or changes needed.