diff options
| author |   j00302280 <j00101220@huawei.com> | 2016-10-10 17:56:49 +0800 |
|---|---|---|
| committer | j00302280 <j00101220@huawei.com> | 2016-10-10 17:56:49 +0800 |
| commit | d2c51b8d20c56fa9a0ddc40b2a9d49e421362c44 (patch) | |
| tree | 6cd0cefc48361f724ea70de420822d200ed8e3c7 /servicegateway/service/src | |
| parent | 9064db27c583df2bae277c5f297af451917815b0 (diff) | |
Resolve the Bug GSO-9
Change-Id: I1cbbc996ce1a43159073d02c8ebb733cfc7d8d54
Signed-off-by: j00302280 <j00101220@huawei.com>
Diffstat (limited to 'servicegateway/service/src')
3 files changed, 9 insertions, 2 deletions
diff --git a/servicegateway/service/src/main/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImpl.java b/servicegateway/service/src/main/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImpl.java index 113693c8..0a70850c 100644 --- a/servicegateway/service/src/main/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImpl.java +++ b/servicegateway/service/src/main/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImpl.java @@ -64,7 +64,7 @@ public class ServiceGatewayImpl implements IServiceGateway { if(StringUtils.isEmpty(reqContent)) { LOGGER.error("ServiceGatewayImpl createService reqContent is null."); - return null; + throw new ServiceException("ServiceGatewayImpl createService reqContent is null."); } // Parse request @@ -128,6 +128,11 @@ public class ServiceGatewayImpl implements IServiceGateway { */ @Override public void deleteService(String serviceId, HttpServletRequest httpRequest) throws ServiceException { + if(httpRequest == null) + { + LOGGER.error("ServiceGatewayImpl.deleteService httpRequest is null"); + throw new ServiceException("ServiceGatewayImpl.deleteService httpRequest is null"); + } // Parse request String reqContent = RestUtils.getRequestBody(httpRequest); Map<String, Object> requestBody = JsonUtil.unMarshal(reqContent, Map.class); diff --git a/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/roa/impl/ServiceGatewayRoaModuleImplTest.java b/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/roa/impl/ServiceGatewayRoaModuleImplTest.java index adbcd757..4fd620cc 100644 --- a/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/roa/impl/ServiceGatewayRoaModuleImplTest.java +++ b/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/roa/impl/ServiceGatewayRoaModuleImplTest.java @@ -122,6 +122,8 @@ public class ServiceGatewayRoaModuleImplTest { */ @Test public void testDeleteService() throws ServiceException { + mockGetRequestBody(FILE_PATH + "createServiceInstance.json"); + mockPost(responseSuccess); serviceRoa.deleteService("1", httpRequest); } diff --git a/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImplTest.java b/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImplTest.java index fddebe00..d1cb3916 100644 --- a/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImplTest.java +++ b/servicegateway/service/src/test/java/org/openo/gso/gui/servicegateway/service/impl/ServiceGatewayImplTest.java @@ -65,7 +65,7 @@ public class ServiceGatewayImplTest { * @since GSO 0.5 */ @Test(expected = ServiceException.class) - public void testDeleteServiceFail() throws ServiceException { + public void testDeleteServiceFail() throws ServiceException { serviceManager.deleteService(null, httpRequest); } |