Apache
/
jclouds
mirror of https://github.com/apache/jclouds.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use assertj fluent assertions where appropriate 

This commit demonstrates a few instances where assertj yields more
informative error messages than testng assertEquals and assertTrue.
Note that we could replace all testng asserts with assertj.
This commit is contained in:
Andrew Gaul 2014-06-29 12:33:40 -07:00
parent 70fb8cdea0
commit e104ef0a4c
3 changed files with 19 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
apis/glacier/pom.xml
View File

@ -89,6 +89,12 @@
<artifactId>mockwebserver</artifactId>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.assertj</groupId>
<artifactId>assertj-core</artifactId>
<version>1.6.1</version>
<scope>test</scope>
</dependency>
</dependencies>
<profiles>

18
apis/glacier/src/test/java/org/jclouds/glacier/GlacierClientLiveTest.java
View File

@ -16,9 +16,9 @@
*/
package org.jclouds.glacier;
import static org.assertj.core.api.Assertions.assertThat;
import static org.jclouds.glacier.util.TestUtils.MiB;
import static org.jclouds.glacier.util.TestUtils.buildPayload;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertNotNull;
import static org.testng.Assert.assertNull;
import static org.testng.Assert.assertTrue;
@ -65,21 +65,23 @@ public class GlacierClientLiveTest extends BaseApiLiveTest<GlacierClient>{
String path = api.createVault(VAULT_NAME1).toString();
api.createVault(VAULT_NAME2);
api.createVault(VAULT_NAME3);
assertTrue(path.contains("https://glacier.us-east-1.amazonaws.com/"));
assertTrue(path.contains("/vaults/" + VAULT_NAME1));
assertThat(path)
.contains("https://glacier.us-east-1.amazonaws.com/")
.contains("/vaults/" + VAULT_NAME1);
}
@Test(groups = { "integration", "live" }, dependsOnMethods = { "testCreateVault" })
public void testListAndDescribeVaults() throws Exception {
PaginatedVaultCollection vaults = api.listVaults();
assertTrue(vaults.contains(api.describeVault(VAULT_NAME1)));
assertTrue(vaults.contains(api.describeVault(VAULT_NAME2)));
assertTrue(vaults.contains(api.describeVault(VAULT_NAME3)));
assertThat(vaults).containsAll(ImmutableList.of(
api.describeVault(VAULT_NAME1),
api.describeVault(VAULT_NAME2),
api.describeVault(VAULT_NAME3)));
}
@Test(groups = { "integration", "live" }, dependsOnMethods = { "testCreateVault" })
public void testListMultipartUploadsWithEmptyList() throws Exception {
assertEquals(api.listMultipartUploads(VAULT_NAME1).size(), 0);
assertThat(api.listMultipartUploads(VAULT_NAME1)).hasSize(0);
}
@Test(groups = { "integration", "live" }, dependsOnMethods = { "testListMultipartUploadsWithEmptyList" })
@ -104,7 +106,7 @@ public class GlacierClientLiveTest extends BaseApiLiveTest<GlacierClient>{
for (MultipartUploadMetadata upload : uploads) {
list.add(upload.getMultipartUploadId());
}
assertTrue(list.build().contains(uploadId));
assertThat(list.build()).contains(uploadId);
assertTrue(api.abortMultipartUpload(VAULT_NAME1, uploadId));
} finally {
api.abortMultipartUpload(VAULT_NAME1, uploadId);

5
apis/glacier/src/test/java/org/jclouds/glacier/GlacierClientMockTest.java
View File

@ -18,6 +18,7 @@ package org.jclouds.glacier;
import static com.google.common.base.Charsets.UTF_8;
import static com.google.common.util.concurrent.MoreExecutors.sameThreadExecutor;
import static org.assertj.core.api.Assertions.assertThat;
import static org.jclouds.Constants.PROPERTY_MAX_RETRIES;
import static org.jclouds.Constants.PROPERTY_SO_TIMEOUT;
import static org.jclouds.glacier.util.TestUtils.MiB;
@ -198,7 +199,7 @@ public class GlacierClientMockTest {
mr.addHeader(HttpHeaders.CONTENT_LENGTH, mr.getBody().length);
server.enqueue(mr);
assertTrue(client.listVaults().isEmpty());
assertThat(client.listVaults()).isEmpty();
}
@Test
@ -382,7 +383,7 @@ public class GlacierClientMockTest {
mr.addHeader(HttpHeaders.CONTENT_LENGTH, mr.getBody().length);
server.enqueue(mr);
assertEquals(client.listMultipartUploads(VAULT_NAME, PaginationOptions.Builder.limit(1).marker(MARKER)).size(), 0);
assertThat(client.listMultipartUploads(VAULT_NAME, PaginationOptions.Builder.limit(1).marker(MARKER))).isEmpty();
assertEquals(server.takeRequest().getRequestLine(),
"GET /-/vaults/examplevault/multipart-uploads?limit=1&marker=" + MARKER + " " + HTTP);
}