Skip to content

Commit 4728a5a

Browse files
zoewanggcenedhryn
authored andcommitted
Use SDK waiter in the cf integ test
1 parent cba776c commit 4728a5a

File tree

1 file changed

+6
-13
lines changed

1 file changed

+6
-13
lines changed

services/cloudformation/src/it/java/software/amazon/awssdk/services/cloudformation/SendEmptyListIntegrationTest.java

+6-13
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import static org.hamcrest.Matchers.not;
2020
import static org.junit.Assert.assertThat;
2121

22-
import java.time.Duration;
2322
import java.util.Collections;
2423
import java.util.List;
2524
import org.junit.After;
@@ -30,10 +29,9 @@
3029
import software.amazon.awssdk.services.cloudformation.model.CreateStackRequest;
3130
import software.amazon.awssdk.services.cloudformation.model.DeleteStackRequest;
3231
import software.amazon.awssdk.services.cloudformation.model.DescribeStacksRequest;
33-
import software.amazon.awssdk.services.cloudformation.model.StackStatus;
3432
import software.amazon.awssdk.services.cloudformation.model.Tag;
3533
import software.amazon.awssdk.services.cloudformation.model.UpdateStackRequest;
36-
import software.amazon.awssdk.testutils.Waiter;
34+
import software.amazon.awssdk.services.cloudformation.waiters.CloudFormationWaiter;
3735
import software.amazon.awssdk.testutils.service.AwsIntegrationTestBase;
3836

3937
/**
@@ -63,6 +61,7 @@ public class SendEmptyListIntegrationTest extends AwsIntegrationTestBase {
6361
"}";
6462

6563
private CloudFormationClient cf;
64+
private CloudFormationWaiter waiter;
6665
private String stackName;
6766

6867
@Before
@@ -78,7 +77,8 @@ public void setup() {
7877
.templateBody(STARTING_TEMPLATE)
7978
.tags(Tag.builder().key("FooKey").value("FooValue").build()).build());
8079

81-
waitUntilComplete(StackStatus.CREATE_COMPLETE);
80+
waiter = cf.waiter();
81+
waiter.waitUntilStackCreateComplete(b -> b.stackName(stackName));
8282
}
8383

8484
@After
@@ -94,7 +94,7 @@ public void explicitlyEmptyTagList_RemovesTagsFromStack() {
9494
.templateBody(STARTING_TEMPLATE)
9595
.tags(Collections.emptyList()).build());
9696

97-
waitUntilComplete(StackStatus.UPDATE_COMPLETE);
97+
waiter.waitUntilStackUpdateComplete(b -> b.stackName(stackName));
9898
assertThat(getTagsForStack(stackName), empty());
9999
}
100100

@@ -105,17 +105,10 @@ public void autoConstructedEmptyTagList_DoesNotRemoveTagsFromStack() {
105105
.stackName(stackName)
106106
.templateBody(UPDATED_TEMPLATE).build());
107107

108-
waitUntilComplete(StackStatus.UPDATE_COMPLETE);
108+
waiter.waitUntilStackUpdateComplete(b -> b.stackName(stackName));
109109
assertThat(getTagsForStack(stackName), not(empty()));
110110
}
111111

112-
private void waitUntilComplete(StackStatus expectedStatus) {
113-
Waiter.run(() -> cf.describeStacks(r -> r.stackName(stackName)))
114-
.until(r -> r.stacks().size() == 1 && r.stacks().get(0).stackStatus() == expectedStatus)
115-
.failOn(r -> r.stacks().size() == 1 && r.stacks().get(0).stackStatus() == StackStatus.ROLLBACK_IN_PROGRESS)
116-
.orFailAfter(Duration.ofMinutes(5));
117-
}
118-
119112
private List<Tag> getTagsForStack(String stackName) {
120113
return cf.describeStacks(
121114
DescribeStacksRequest.builder().stackName(stackName).build())

0 commit comments

Comments
 (0)