-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[judge-d-24] renamed endpoint /contracts to /contracts/services #71
Conversation
(cherry picked from commit ae2eb5a)
@@ -56,6 +56,16 @@ public ServiceContractsDto create(@PathVariable(name = "provider") String provid | |||
return serviceContractsRepository.getServiceNames(); | |||
} | |||
|
|||
@GetMapping(value = "/contracts", produces = MediaType.APPLICATION_JSON_VALUE) | |||
@ApiOperation(value = "Get contracts registered", nickname = "get names of services") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get registered contracts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation fixed
def 'should calling /contracts should redirect to /contracts/services to improve api discovery'() { | ||
given: | ||
when: | ||
def response = mockMvc.perform( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indents here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code indented
...rver/src/test/groovy/dev/hltech/dredd/interfaces/rest/contracts/ContractsControllerIT.groovy
Outdated
Show resolved
Hide resolved
DONE: * improved endpoint documentation * test code indented and given section removed from ContractsControllerIT
Merged |
In order to increase user experience when working with API I have changed its structure. It is first part of changes towards /contracts/services/{service-name}/versions/{version}