Skip to content

Commit f1db5eb

Browse files
g2vinayJonathanGiles
authored andcommitted
Test run time optimizations (Azure#4384)
1 parent 728cc1c commit f1db5eb

File tree

3 files changed

+15
-15
lines changed

3 files changed

+15
-15
lines changed

sdk/keyvault/azure-keyvault-keys/src/test/java/com/azure/security/keyvault/keys/KeyAsyncClientTest.java

+6-6
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@ public void getDeletedKey() {
337337
assertNotNull(deletedKey);
338338
}).verifyComplete();
339339
pollOnKeyDeletion(keyToDeleteAndGet.name());
340-
sleep(30000);
340+
sleepInRecordMode(30000);
341341

342342
StepVerifier.create(client.getDeletedKey(keyToDeleteAndGet.name()))
343343
.assertNext(deletedKeyResponse -> {
@@ -353,7 +353,7 @@ public void getDeletedKey() {
353353
assertEquals(HttpResponseStatus.NO_CONTENT.code(), voidResponse.statusCode());
354354
}).verifyComplete();
355355
pollOnKeyPurge(keyToDeleteAndGet.name());
356-
sleep(15000);
356+
sleepInRecordMode(15000);
357357
});
358358
}
359359
//
@@ -421,9 +421,9 @@ public void listKeyVersions() {
421421
client.createKey(key).subscribe(keyResponse -> assertKeyEquals(key, keyResponse.value()));
422422
sleepInRecordMode(1000);
423423
}
424-
sleep(30000);
424+
sleepInRecordMode(30000);
425425
client.listKeyVersions(keyName).subscribe(output::add);
426-
sleep(30000);
426+
sleepInRecordMode(30000);
427427

428428
assertEquals(keyVersions.size(), output.size());
429429

@@ -455,9 +455,9 @@ public void listKeys() {
455455
client.createKey(key).subscribe(keyResponse -> assertKeyEquals(key, keyResponse.value()));
456456
sleepInRecordMode(1000);
457457
}
458-
sleep(30000);
458+
sleepInRecordMode(30000);
459459
client.listKeys().subscribe(output::add);
460-
sleep(30000);
460+
sleepInRecordMode(30000);
461461

462462
for (KeyBase actualKey : output) {
463463
if (keys.containsKey(actualKey.name())) {

sdk/keyvault/azure-keyvault-keys/src/test/java/com/azure/security/keyvault/keys/KeyClientTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ public void restoreKey() {
217217
pollOnKeyDeletion(keyToBackupAndRestore.name());
218218
client.purgeDeletedKey(keyToBackupAndRestore.name());
219219
pollOnKeyPurge(keyToBackupAndRestore.name());
220-
sleep(60000);
220+
sleepInRecordMode(60000);
221221
Key restoredKey = client.restoreKey(backupBytes).value();
222222
assertEquals(keyToBackupAndRestore.name(), restoredKey.name());
223223
assertEquals(keyToBackupAndRestore.expires(), restoredKey.expires());
@@ -240,7 +240,7 @@ public void listKeys() {
240240
HashMap<String, KeyCreateOptions> keysToList = keys;
241241
for (KeyCreateOptions key : keysToList.values()) {
242242
assertKeyEquals(key, client.createKey(key));
243-
sleep(5000);
243+
sleepInRecordMode(5000);
244244
}
245245

246246
for (KeyBase actualKey : client.listKeys()) {

sdk/keyvault/azure-keyvault-secrets/src/test/java/com/azure/security/keyvault/secrets/SecretAsyncClientTest.java

+7-7
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ public void getDeletedSecret() {
217217
assertNotNull(deletedSecret);
218218
}).verifyComplete();
219219
pollOnSecretDeletion(secretToDeleteAndGet.name());
220-
sleep(30000);
220+
sleepInRecordMode(30000);
221221

222222
StepVerifier.create(client.getDeletedSecret(secretToDeleteAndGet.name()))
223223
.assertNext(deletedSecretResponse -> {
@@ -233,7 +233,7 @@ public void getDeletedSecret() {
233233
assertEquals(HttpResponseStatus.NO_CONTENT.code(), voidResponse.statusCode());
234234
}).verifyComplete();
235235
pollOnSecretPurge(secretToDeleteAndGet.name());
236-
sleep(10000);
236+
sleepInRecordMode(10000);
237237
});
238238
}
239239

@@ -331,7 +331,7 @@ public void restoreSecret() {
331331
}).verifyComplete();
332332
pollOnSecretPurge(secretToBackupAndRestore.name());
333333

334-
sleep(60000);
334+
sleepInRecordMode(60000);
335335

336336
StepVerifier.create(client.restoreSecret(backup))
337337
.assertNext(response -> {
@@ -416,9 +416,9 @@ public void listSecretVersions() {
416416
client.setSecret(secret).subscribe(secretResponse -> assertSecretEquals(secret, secretResponse.value()));
417417
sleepInRecordMode(1000);
418418
}
419-
sleep(30000);
419+
sleepInRecordMode(30000);
420420
client.listSecretVersions(secretName).subscribe(output::add);
421-
sleep(30000);
421+
sleepInRecordMode(30000);
422422

423423
assertEquals(secretVersions.size(), output.size());
424424

@@ -450,9 +450,9 @@ public void listSecrets() {
450450
client.setSecret(secret).subscribe(secretResponse -> assertSecretEquals(secret, secretResponse.value()));
451451
sleepInRecordMode(1000);
452452
}
453-
sleep(30000);
453+
sleepInRecordMode(30000);
454454
client.listSecrets().subscribe(output::add);
455-
sleep(30000);
455+
sleepInRecordMode(30000);
456456

457457
for (SecretBase actualSecret : output) {
458458
if (secretsToList.containsKey(actualSecret.name())) {

0 commit comments

Comments
 (0)