Code Review
/
dmaap
/
dbcapi.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Tests for DR_NodeResource & bugfixes
[dmaap/dbcapi.git]
/
src
/
main
/
java
/
org
/
onap
/
dmaap
/
dbcapi
/
resources
/
DR_NodeResource.java
diff --git
a/src/main/java/org/onap/dmaap/dbcapi/resources/DR_NodeResource.java
b/src/main/java/org/onap/dmaap/dbcapi/resources/DR_NodeResource.java
index
49dc69a
..
d29f353
100644
(file)
--- a/
src/main/java/org/onap/dmaap/dbcapi/resources/DR_NodeResource.java
+++ b/
src/main/java/org/onap/dmaap/dbcapi/resources/DR_NodeResource.java
@@
-118,10
+118,11
@@
public class DR_NodeResource extends BaseLoggingClass {
ApiService resp = new ApiService();
try {
ApiService resp = new ApiService();
try {
- checker.required( "dcaeLocation", n
ame
);
+ checker.required( "dcaeLocation", n
ode.getDcaeLocationName()
);
checker.required( "fqdn", node.getFqdn());
} catch ( RequiredFieldException rfe ) {
checker.required( "fqdn", node.getFqdn());
} catch ( RequiredFieldException rfe ) {
- return responseBuilder.error(rfe.getApiError());
+ return responseBuilder.error(new ApiError(BAD_REQUEST.getStatusCode(),
+ "missing required field", "dcaeLocation, fqdn"));
}
node.setFqdn(name);
DR_Node nNode = dr_nodeService.updateDr_Node(node, resp.getErr());
}
node.setFqdn(name);
DR_Node nNode = dr_nodeService.updateDr_Node(node, resp.getErr());
@@
-143,15
+144,8
@@
public class DR_NodeResource extends BaseLoggingClass {
public Response deleteDr_Node(
@PathParam("fqdn") String name
){
public Response deleteDr_Node(
@PathParam("fqdn") String name
){
-
ApiService resp = new ApiService();
ApiService resp = new ApiService();
- try {
- checker.required( "fqdn", name);
- } catch ( RequiredFieldException rfe ) {
- logger.debug( rfe.getApiError().toString() );
- return responseBuilder.error(rfe.getApiError());
- }
dr_nodeService.removeDr_Node(name, resp.getErr());
if ( resp.getErr().is2xx() ) {
return responseBuilder.success(NO_CONTENT.getStatusCode(), null);
dr_nodeService.removeDr_Node(name, resp.getErr());
if ( resp.getErr().is2xx() ) {
return responseBuilder.success(NO_CONTENT.getStatusCode(), null);