Skip to content

Commit a463dc1

Browse files
committed
Adjusted clear not to collide when parallel test are running.
1 parent 8684506 commit a463dc1

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

gcloud-java-dns/src/test/java/com/google/gcloud/dns/it/ITDnsTest.java

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -101,17 +101,21 @@ public class ITDnsTest {
101101
.delete(A_RECORD_ZONE1)
102102
.delete(AAAA_RECORD_ZONE1)
103103
.build();
104+
public static final List<String> ZONE_NAMES = ImmutableList.of(ZONE_NAME1,
105+
ZONE_NAME_EMPTY_DESCRIPTION);
104106

105107
public static void clear() {
106-
Page<Zone> zones = DNS.listZones();
107-
Iterator<Zone> zoneIterator = zones.iterateAll();
108-
while (zoneIterator.hasNext()) {
109-
Zone zone = zoneIterator.next();
110-
List<DnsRecord> toDelete = new LinkedList<>();
111-
if (zone.name().startsWith(PREFIX)) {
112-
Iterator<DnsRecord> dnsRecordIterator = zone.listDnsRecords().iterateAll();
113-
while (dnsRecordIterator.hasNext()) {
114-
DnsRecord record = dnsRecordIterator.next();
108+
for (String zoneName : ZONE_NAMES) {
109+
Zone zone = DNS.getZone(zoneName);
110+
if (zone != null) {
111+
Iterator<ChangeRequest> iterator = zone.listChangeRequests().iterateAll();
112+
while (iterator.hasNext()) {
113+
waitUntilComplete(zoneName, iterator.next().id());
114+
}
115+
Iterator<DnsRecord> recordIterator = zone.listDnsRecords().iterateAll();
116+
List<DnsRecord> toDelete = new LinkedList<>();
117+
while (recordIterator.hasNext()) {
118+
DnsRecord record = recordIterator.next();
115119
if (!ImmutableList.of(DnsRecord.Type.NS, DnsRecord.Type.SOA).contains(record.type())) {
116120
toDelete.add(record);
117121
}
@@ -130,7 +134,7 @@ private static List<Zone> filter(Iterator<Zone> iterator) {
130134
List<Zone> result = new LinkedList<>();
131135
while (iterator.hasNext()) {
132136
Zone zone = iterator.next();
133-
if (zone.name().startsWith(PREFIX)) {
137+
if (ZONE_NAMES.contains(zone.name())) {
134138
result.add(zone);
135139
}
136140
}

0 commit comments

Comments
 (0)