if (acmDeployer.primeRapp(rapp) && dmeDeployer.primeRapp(rapp)) {
rapp.setState(RappState.PRIMED);
return ResponseEntity.ok().build();
- } else {
- rapp.setState(RappState.COMMISSIONED);
- return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
}
- } else {
- return ResponseEntity.badRequest()
- .body(String.format(STATE_TRANSITION_NOT_PERMITTED, RappState.PRIMED.name(),
- rapp.getState().name()));
+ rapp.setState(RappState.COMMISSIONED);
+ return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
}
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format(STATE_TRANSITION_NOT_PERMITTED, rapp.getState().name(), RappState.PRIMED.name()));
+
}
public ResponseEntity<String> deprimeRapp(Rapp rapp) {
if (acmDeployer.deprimeRapp(rapp) && dmeDeployer.deprimeRapp(rapp)) {
rapp.setState(RappState.COMMISSIONED);
return ResponseEntity.ok().build();
- } else {
- rapp.setState(RappState.PRIMED);
- return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
}
+ rapp.setState(RappState.PRIMED);
+ return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
+ }
+ if (!rapp.getRappInstances().isEmpty()) {
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ "Unable to deprime as there are active rapp instances.");
} else {
- if (!rapp.getRappInstances().isEmpty()) {
- return ResponseEntity.badRequest().body("Unable to deprime as there are active rapp instances,");
- } else {
- return ResponseEntity.badRequest()
- .body(String.format(STATE_TRANSITION_NOT_PERMITTED, RappState.COMMISSIONED.name(),
- rapp.getState().name()));
- }
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format(STATE_TRANSITION_NOT_PERMITTED, RappState.COMMISSIONED.name(),
+ rapp.getState().name()));
}
}
if (rApp.getRappInstances().isEmpty() && rApp.getState().equals(RappState.COMMISSIONED)) {
rappCacheService.deleteRapp(rApp);
return ResponseEntity.ok().build();
+ }
+ if (!rApp.getRappInstances().isEmpty()) {
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format("Unable to delete %s as there are active rApp instances.", rApp.getName()));
} else {
- if (!rApp.getRappInstances().isEmpty()) {
- return ResponseEntity.badRequest()
- .body("Unable to delete '" + rApp.getName() + "' as there are active rApp instances.");
- } else {
- return ResponseEntity.badRequest().body("Unable to delete '" + rApp.getName()
- + "' as the rApp is not in COMMISSIONED state.");
- }
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format("Unable to delete %s as the rApp is not in COMMISSIONED state.", rApp.getName()));
}
+
}
public ResponseEntity<String> deployRappInstance(Rapp rapp, RappInstance rappInstance) {
return ResponseEntity.accepted().build();
}
return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
- } else {
- return ResponseEntity.badRequest()
- .body(String.format(STATE_TRANSITION_NOT_PERMITTED, rappInstance.getState().name(),
- RappInstanceState.DEPLOYED.name()));
}
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format("Unable to deploy rApp instance %s as it is not in UNDEPLOYED state",
+ rappInstance.getRappInstanceId()));
+
}
public ResponseEntity<String> undeployRappInstance(Rapp rapp, RappInstance rappInstance) {
return ResponseEntity.accepted().build();
}
return ResponseEntity.status(HttpStatus.BAD_GATEWAY).build();
- } else {
- return ResponseEntity.badRequest()
- .body(String.format(STATE_TRANSITION_NOT_PERMITTED, rappInstance.getState().name(),
- RappInstanceState.UNDEPLOYED.name()));
}
+ throw new RappHandlerException(HttpStatus.BAD_REQUEST,
+ String.format("Unable to undeploy rApp instance %s as it is not in DEPLOYED state",
+ rappInstance.getRappInstanceId()));
}
- public ResponseEntity<Object> deleteRappInstance(Rapp rApp, UUID rappInstanceId) {
+ public ResponseEntity<String> deleteRappInstance(Rapp rApp, UUID rappInstanceId) {
if (rApp.getRappInstances().get(rappInstanceId).getState().equals(RappInstanceState.UNDEPLOYED)) {
rappInstanceStateMachine.deleteRappInstance(rApp.getRappInstances().get(rappInstanceId));
rApp.getRappInstances().remove(rappInstanceId);
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.boot.test.mock.mockito.MockBean;
import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
@AutoConfigureMockMvc
private final String validRappFile = "valid-rapp-package.csar";
+ private final String STATE_TRANSITION_NOT_PERMITTED = "State transition from %s to %s is not permitted.";
+
@Test
void testPrimeRapp() {
void testPrimeRappInvalidState() {
Rapp rapp = Rapp.builder().rappId(UUID.randomUUID()).name("").packageName(validRappFile)
.packageLocation(validCsarFileLocation).state(RappState.PRIMING).build();
- assertEquals(HttpStatus.BAD_REQUEST, rappService.primeRapp(rapp).getStatusCode());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.primeRapp(rapp));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format(STATE_TRANSITION_NOT_PERMITTED, RappState.PRIMING, RappState.PRIMED),
+ rappHandlerException.getMessage());
+ assertEquals(RappState.PRIMING, rapp.getState());
}
@Test
void testDeprimeRappInvalidState() {
Rapp rapp = Rapp.builder().rappId(UUID.randomUUID()).name("").packageName(validRappFile)
.packageLocation(validCsarFileLocation).state(RappState.COMMISSIONED).build();
- assertEquals(HttpStatus.BAD_REQUEST, rappService.deprimeRapp(rapp).getStatusCode());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deprimeRapp(rapp));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format(STATE_TRANSITION_NOT_PERMITTED, RappState.COMMISSIONED, RappState.COMMISSIONED),
+ rappHandlerException.getMessage());
assertEquals(RappState.COMMISSIONED, rapp.getState());
}
Rapp rapp = Rapp.builder().rappId(UUID.randomUUID()).name("").packageName(validRappFile)
.packageLocation(validCsarFileLocation).state(RappState.PRIMED)
.rappInstances(Map.of(UUID.randomUUID(), new RappInstance())).build();
- assertEquals(HttpStatus.BAD_REQUEST, rappService.deprimeRapp(rapp).getStatusCode());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deprimeRapp(rapp));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
assertEquals(RappState.PRIMED, rapp.getState());
}
Rapp rapp = Rapp.builder().rappId(UUID.randomUUID()).name("").packageName(validRappFile)
.packageLocation(validCsarFileLocation).state(RappState.PRIMED).build();
RappInstance rappInstance = new RappInstance();
- RappInstanceState rappInstanceState = RappInstanceState.DEPLOYED;
- rappInstance.setState(rappInstanceState);
+ rappInstance.setState(RappInstanceState.DEPLOYED);
rappInstanceStateMachine.onboardRappInstance(rappInstance.getRappInstanceId());
- ResponseEntity<String> responseEntity = rappService.deployRappInstance(rapp, rappInstance);
- assertEquals(HttpStatus.BAD_REQUEST, responseEntity.getStatusCode());
- assertEquals("State transition from " + rappInstanceState + " to DEPLOYED is not permitted.",
- responseEntity.getBody());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deployRappInstance(rapp, rappInstance));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format("Unable to deploy rApp instance %s as it is not in UNDEPLOYED state",
+ rappInstance.getRappInstanceId()), rappHandlerException.getMessage());
+ assertEquals(RappState.PRIMED, rapp.getState());
+
}
@Test
when(acmDeployer.undeployRappInstance(any(), any())).thenReturn(true);
when(smeDeployer.undeployRappInstance(any(), any())).thenReturn(false);
when(dmeDeployer.undeployRappInstance(any(), any())).thenReturn(true);
- assertEquals(HttpStatus.BAD_REQUEST, rappService.undeployRappInstance(rapp, rappInstance).getStatusCode());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.undeployRappInstance(rapp, rappInstance));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
}
@Test
.packageLocation(validCsarFileLocation).state(RappState.PRIMED).build();
RappInstance rappInstance = new RappInstance();
rappInstance.setState(RappInstanceState.DEPLOYED);
+ UUID rappInstanceId = rappInstance.getRappInstanceId();
HashMap<UUID, RappInstance> rAppInstanceMap = new HashMap<>();
- rAppInstanceMap.put(rappInstance.getRappInstanceId(), rappInstance);
+ rAppInstanceMap.put(rappInstanceId, rappInstance);
rapp.setRappInstances(rAppInstanceMap);
rappInstanceStateMachine.onboardRappInstance(rappInstance.getRappInstanceId());
- assertThrows(RappHandlerException.class,
- () -> rappService.deleteRappInstance(rapp, rappInstance.getRappInstanceId()));
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deleteRappInstance(rapp, rappInstanceId));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format("Unable to delete rApp instance %s as it is not in UNDEPLOYED state",
+ rappInstance.getRappInstanceId()), rappHandlerException.getMessage());
+ assertEquals(RappState.PRIMED, rapp.getState());
}
@Test
String rAppName = "rAppInPrimed";
Rapp rApp = Rapp.builder().rappId(UUID.randomUUID()).name(rAppName).packageName(validRappFile)
.packageLocation(validCsarFileLocation).state(RappState.PRIMED).build();
- ResponseEntity<String> responseEntity = rappService.deleteRapp(rApp);
- assertEquals(HttpStatus.BAD_REQUEST, responseEntity.getStatusCode());
- assertEquals("Unable to delete '" + rAppName + "' as the rApp is not in COMMISSIONED state.",
- responseEntity.getBody());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deleteRapp(rApp));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format("Unable to delete %s as the rApp is not in COMMISSIONED state.", rAppName),
+ rappHandlerException.getMessage());
+ assertEquals(RappState.PRIMED, rApp.getState());
}
@Test
rappInstance.setState(RappInstanceState.DEPLOYED);
rappInstanceStateMachine.onboardRappInstance(rappInstance.getRappInstanceId());
rApp.setRappInstances(Map.of(rappInstance.getRappInstanceId(), rappInstance));
- ResponseEntity<String> responseEntity = rappService.deleteRapp(rApp);
- assertEquals(HttpStatus.BAD_REQUEST, responseEntity.getStatusCode());
- assertEquals("Unable to delete '" + rAppName + "' as there are active rApp instances.",
- responseEntity.getBody());
+ RappHandlerException rappHandlerException =
+ assertThrows(RappHandlerException.class, () -> rappService.deleteRapp(rApp));
+ assertEquals(HttpStatus.BAD_REQUEST, rappHandlerException.getStatusCode());
+ assertEquals(String.format("Unable to delete %s as there are active rApp instances.", rAppName),
+ rappHandlerException.getMessage());
+ assertEquals(RappState.PRIMED, rApp.getState());
}
}