Skip to content

Commit

Permalink
✨ : delete state related to stack
Browse files Browse the repository at this point in the history
  • Loading branch information
juwit committed Aug 5, 2022
1 parent dc9f931 commit 4f9462e
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
import io.gaia_app.stacks.bo.Stack;
import io.gaia_app.stacks.repository.JobRepository;
import io.gaia_app.stacks.repository.StackRepository;
import io.gaia_app.stacks.repository.TerraformStateRepository;
import io.gaia_app.stacks.service.StackCostCalculator;
import io.gaia_app.organizations.Organization;
import io.gaia_app.organizations.User;
Expand All @@ -31,16 +32,20 @@ public class StackRestController {

private CredentialsRepository credentialsRepository;

private TerraformStateRepository stateRepository;

@Autowired
public StackRestController(
StackRepository stackRepository,
StackCostCalculator stackCostCalculator,
JobRepository jobRepository,
CredentialsRepository credentialsRepository) {
CredentialsRepository credentialsRepository,
TerraformStateRepository stateRepository) {
this.stackRepository = stackRepository;
this.stackCostCalculator = stackCostCalculator;
this.jobRepository = jobRepository;
this.credentialsRepository = credentialsRepository;
this.stateRepository = stateRepository;
}

@GetMapping
Expand Down Expand Up @@ -93,6 +98,7 @@ public void delete(@PathVariable String id, User user){
// delete stack if it was found
stackRepository.delete(stack);
jobRepository.deleteByStackId(id);
stateRepository.deleteById(id);
}

@PostMapping("/{id}/{jobType}")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import io.gaia_app.stacks.bo.*;
import io.gaia_app.stacks.repository.JobRepository;
import io.gaia_app.stacks.repository.StackRepository;
import io.gaia_app.stacks.repository.TerraformStateRepository;
import io.gaia_app.stacks.service.StackCostCalculator;
import io.gaia_app.organizations.Organization;
import io.gaia_app.organizations.User;
Expand Down Expand Up @@ -55,6 +56,9 @@ class StackRestControllerTest {
@Mock
private CredentialsRepository credentialsRepository;

@Mock
private TerraformStateRepository terraformStateRepository;

@BeforeEach
void setUp() {
adminUser.setAdmin(true);
Expand Down Expand Up @@ -183,6 +187,7 @@ void delete_shouldRemoveStack_forAdmin(){
// then
verify(stackRepository).delete(stack);
verify(jobRepository).deleteByStackId("42");
verify(terraformStateRepository).deleteById("42");
}

@Test
Expand All @@ -197,6 +202,7 @@ void delete_shouldRemoveStack_forStandardUser() {
// then
verify(stackRepository).delete(stack);
verify(jobRepository).deleteByStackId("42");
verify(terraformStateRepository).deleteById("42");
}

@Test
Expand All @@ -211,6 +217,7 @@ void delete_shouldRemoveStack_forUserWithoutOrganization() {
// then
verify(stackRepository).delete(stack);
verify(jobRepository).deleteByStackId("42");
verify(terraformStateRepository).deleteById("42");
}

@Test
Expand Down

0 comments on commit 4f9462e

Please sign in to comment.