@PathParam("dataTypeUid") String dataTypeUid) {
final String url = request.getMethod() + " " + request.getRequestURI();
log.debug("Start handle request of {} - modifier id is {}", url, userId);
- final Optional<DataTypeDataDefinition> dataTypeByUid;
+ final Optional<DataTypeDataDefinition> dataTypeFoundOptional;
try {
- dataTypeByUid = dataTypeOperation.getDataTypeByUid(dataTypeUid);
+ dataTypeFoundOptional = dataTypeOperation.getDataTypeByUid(dataTypeUid);
} catch (final BusinessException e) {
throw e;
} catch (final Exception ex) {
log.error(EcompLoggerErrorCode.UNKNOWN_ERROR, this.getClass().getName(), errorMsg, ex);
return buildErrorResponse(getComponentsUtils().getResponseFormat(ActionStatus.GENERAL_ERROR));
}
- if (dataTypeByUid.isEmpty()) {
- throw new OperationException(ActionStatus.DATA_TYPE_NOT_FOUND, dataTypeUid);
- }
- return buildOkResponse(getComponentsUtils().getResponseFormat(ActionStatus.OK), dataTypeByUid);
+ var dataType = dataTypeFoundOptional.orElseThrow(() -> new OperationException(ActionStatus.DATA_TYPE_NOT_FOUND, dataTypeUid));
+ return buildOkResponse(getComponentsUtils().getResponseFormat(ActionStatus.OK), dataType);
}
}
@Test
void fetchDataTypeTest_Success() {
+ final DataTypeDataDefinition expectedDataType = new DataTypeDataDefinition();
+ expectedDataType.setUniqueId(DATA_TYPE_UID);
when(componentsUtils.getResponseFormat(ActionStatus.OK)).thenReturn(new ResponseFormat(HttpStatus.SC_OK));
- when(dataTypeOperation.getDataTypeByUid(DATA_TYPE_UID)).thenReturn(Optional.of(new DataTypeDataDefinition()));
+ when(dataTypeOperation.getDataTypeByUid(DATA_TYPE_UID)).thenReturn(Optional.of(expectedDataType));
final Response response = target()
.path(PATH)
.get(Response.class);
assertNotNull(response);
assertEquals(HttpStatus.SC_OK, response.getStatus());
+ final DataTypeDataDefinition actualDataType = response.readEntity(DataTypeDataDefinition.class);
+ assertEquals(expectedDataType.getUniqueId(), actualDataType.getUniqueId());
}
@Test