Skip to content

Commit d842b4e

Browse files
andricDuandricDu
authoredAug 7, 2019
Merge pull request #377 from overture-stack/feature/assertjReplace
Feature/assertj replace
·
UNKNOWN2.2.3
2 parents d49247e + 034ba5b commit d842b4e

File tree

7 files changed

+123
-111
lines changed

7 files changed

+123
-111
lines changed
 

‎src/test/java/bio/overture/ego/controller/TokenControllerTest.java

+30-31
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
import static java.util.Arrays.asList;
77
import static net.javacrumbs.jsonunit.core.Option.IGNORING_ARRAY_ORDER;
88
import static net.javacrumbs.jsonunit.fluent.JsonFluentAssert.assertThatJson;
9-
import static org.assertj.core.api.Assertions.assertThat;
9+
import static org.junit.Assert.*;
1010

1111
import bio.overture.ego.AuthorizationServiceMain;
1212
import bio.overture.ego.model.dto.PermissionRequest;
@@ -98,9 +98,9 @@ public void issueTokenShouldRevokeRedundantTokens() {
9898
"",
9999
entityGenerator.getScopes("collab.READ"));
100100

101-
assertThat(tokenService.getById(tokenRevoke.getId()).isRevoked()).isFalse();
102-
assertThat(tokenService.getById(otherToken.getId()).isRevoked()).isFalse();
103-
assertThat(tokenService.getById(otherToken2.getId()).isRevoked()).isFalse();
101+
assertFalse(tokenService.getById(tokenRevoke.getId()).isRevoked());
102+
assertFalse(tokenService.getById(otherToken.getId()).isRevoked());
103+
assertFalse(tokenService.getById(otherToken2.getId()).isRevoked());
104104

105105
val scopes = "collab.READ,aws.READ";
106106
val params = new LinkedMultiValueMap<String, Object>();
@@ -112,10 +112,10 @@ public void issueTokenShouldRevokeRedundantTokens() {
112112
val response = initStringRequest().endpoint("o/token").body(params).post();
113113
val responseStatus = response.getStatusCode();
114114

115-
assertThat(responseStatus).isEqualTo(HttpStatus.OK);
116-
assertThat(tokenService.getById(tokenRevoke.getId()).isRevoked()).isTrue();
117-
assertThat(tokenService.getById(otherToken.getId()).isRevoked()).isFalse();
118-
assertThat(tokenService.getById(otherToken2.getId()).isRevoked()).isFalse();
115+
assertEquals(responseStatus, HttpStatus.OK);
116+
assertTrue(tokenService.getById(tokenRevoke.getId()).isRevoked());
117+
assertFalse(tokenService.getById(otherToken.getId()).isRevoked());
118+
assertFalse(tokenService.getById(otherToken2.getId()).isRevoked());
119119
}
120120

121121
@SneakyThrows
@@ -151,7 +151,7 @@ public void issueTokenExactScope() {
151151
val response = initStringRequest().endpoint("o/token").body(params).post();
152152
val statusCode = response.getStatusCode();
153153

154-
assertThat(statusCode).isEqualTo(HttpStatus.OK);
154+
assertEquals(statusCode, HttpStatus.OK);
155155
assertThatJson(response.getBody())
156156
.when(IGNORING_ARRAY_ORDER)
157157
.node("scope")
@@ -187,11 +187,11 @@ public void issueTokenWithExcessiveScope() {
187187

188188
val response = initStringRequest().endpoint("o/token").body(params).post();
189189
val statusCode = response.getStatusCode();
190-
assertThat(statusCode).isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR);
190+
assertEquals(statusCode, HttpStatus.INTERNAL_SERVER_ERROR);
191191

192192
val jsonResponse = MAPPER.readTree(response.getBody());
193-
assertThat(jsonResponse.get("error").asText())
194-
.isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase());
193+
assertEquals(
194+
jsonResponse.get("error").asText(), HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase());
195195
}
196196

197197
@SneakyThrows
@@ -230,7 +230,7 @@ public void issueTokenForLimitedScopes() {
230230
val response = initStringRequest().endpoint("o/token").body(params).post();
231231
val statusCode = response.getStatusCode();
232232

233-
assertThat(statusCode).isEqualTo(HttpStatus.OK);
233+
assertEquals(statusCode, HttpStatus.OK);
234234
assertThatJson(response.getBody())
235235
.when(IGNORING_ARRAY_ORDER)
236236
.node("scope")
@@ -275,10 +275,9 @@ public void issueTokenForInvalidScope() {
275275
val response = initStringRequest().endpoint("o/token").body(params).post();
276276

277277
val statusCode = response.getStatusCode();
278-
assertThat(statusCode).isEqualTo(HttpStatus.NOT_FOUND);
278+
assertEquals(statusCode, HttpStatus.NOT_FOUND);
279279
val jsonResponse = MAPPER.readTree(response.getBody());
280-
assertThat(jsonResponse.get("error").asText())
281-
.isEqualTo(HttpStatus.NOT_FOUND.getReasonPhrase());
280+
assertEquals(jsonResponse.get("error").asText(), HttpStatus.NOT_FOUND.getReasonPhrase());
282281
}
283282

284283
@SneakyThrows
@@ -296,11 +295,11 @@ public void issueTokenForInvalidUser() {
296295
val response = initStringRequest().endpoint("o/token").body(params).post();
297296

298297
val statusCode = response.getStatusCode();
299-
assertThat(statusCode).isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR);
298+
assertEquals(statusCode, HttpStatus.INTERNAL_SERVER_ERROR);
300299

301300
val jsonResponse = MAPPER.readTree(response.getBody());
302-
assertThat(jsonResponse.get("error").asText())
303-
.isEqualTo(HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase());
301+
assertEquals(
302+
jsonResponse.get("error").asText(), HttpStatus.INTERNAL_SERVER_ERROR.getReasonPhrase());
304303
}
305304

306305
@SneakyThrows
@@ -319,7 +318,7 @@ public void checkRevokedToken() {
319318
val response = initStringRequest().endpoint("o/check_token").body(params).post();
320319

321320
val statusCode = response.getStatusCode();
322-
assertThat(statusCode).isEqualTo(HttpStatus.UNAUTHORIZED);
321+
assertEquals(statusCode, HttpStatus.UNAUTHORIZED);
323322
}
324323

325324
@SneakyThrows
@@ -338,7 +337,7 @@ public void checkValidToken() {
338337
val response = initStringRequest().endpoint("o/check_token").body(params).post();
339338

340339
val statusCode = response.getStatusCode();
341-
assertThat(statusCode).isEqualTo(HttpStatus.MULTI_STATUS);
340+
assertEquals(statusCode, HttpStatus.MULTI_STATUS);
342341
}
343342

344343
@SneakyThrows
@@ -354,7 +353,7 @@ public void checkInvalidToken() {
354353
val response = initStringRequest().endpoint("o/check_token").body(params).post();
355354

356355
val statusCode = response.getStatusCode();
357-
assertThat(statusCode).isEqualTo(HttpStatus.UNAUTHORIZED);
356+
assertEquals(statusCode, HttpStatus.UNAUTHORIZED);
358357
}
359358

360359
@SneakyThrows
@@ -383,7 +382,7 @@ public void getUserScope() {
383382
val response = initStringRequest().endpoint("o/scopes?userName=%s", userName).get();
384383

385384
val statusCode = response.getStatusCode();
386-
assertThat(statusCode).isEqualTo(HttpStatus.OK);
385+
assertEquals(statusCode, HttpStatus.OK);
387386
assertThatJson(response.getBody())
388387
.when(IGNORING_ARRAY_ORDER)
389388
.node("scopes")
@@ -397,7 +396,7 @@ public void getUserScopeInvalidUserName() {
397396
val response = initStringRequest().endpoint("o/scopes?userName=%s", userName).get();
398397

399398
val statusCode = response.getStatusCode();
400-
assertThat(statusCode).isEqualTo(HttpStatus.NOT_FOUND);
399+
assertEquals(statusCode, HttpStatus.NOT_FOUND);
401400
}
402401

403402
@SneakyThrows
@@ -421,7 +420,7 @@ public void listToken() {
421420
val response = initStringRequest().endpoint("o/token?user_id=%s", userId).get();
422421

423422
val statusCode = response.getStatusCode();
424-
assertThat(statusCode).isEqualTo(HttpStatus.OK);
423+
assertEquals(statusCode, HttpStatus.OK);
425424

426425
// Result should only have unrevoked tokens, ignoring the "exp" field.
427426
val expected =
@@ -443,8 +442,8 @@ public void listTokenEmptyToken() {
443442
val response = initStringRequest().endpoint("o/token?user_id=%s", userId).get();
444443

445444
val statusCode = response.getStatusCode();
446-
assertThat(statusCode).isEqualTo(HttpStatus.OK);
447-
assertThat(response.getBody()).isEqualTo("[]");
445+
assertEquals(statusCode, HttpStatus.OK);
446+
assertEquals(response.getBody(), "[]");
448447
}
449448

450449
@SneakyThrows
@@ -464,17 +463,17 @@ public void tokenShouldHaveNonZeroExpiry() {
464463
val response = initStringRequest().endpoint("o/token").body(params).post();
465464
val responseStatus = response.getStatusCode();
466465

467-
assertThat(responseStatus).isEqualTo(HttpStatus.OK);
466+
assertEquals(responseStatus, HttpStatus.OK);
468467

469468
val listResponse =
470469
initStringRequest().endpoint("o/token?user_id=%s", user.getId().toString()).get();
471470
val listStatusCode = listResponse.getStatusCode();
472-
assertThat(listStatusCode).isEqualTo(HttpStatus.OK);
471+
assertEquals(listStatusCode, HttpStatus.OK);
473472

474473
log.info(listResponse.getBody());
475474
val responseJson = MAPPER.readTree(listResponse.getBody());
476475
val exp = responseJson.get(0).get("exp").asInt();
477-
assertThat(exp).isNotZero();
478-
assertThat(exp).isPositive();
476+
assertTrue(exp != 0);
477+
assertTrue(exp > 0);
479478
}
480479
}

‎src/test/java/bio/overture/ego/controller/TokensOnUserAndPolicyDeletes.java

+10-11
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@
1717

1818
package bio.overture.ego.controller;
1919

20-
import static org.assertj.core.api.Assertions.assertThat;
20+
import static org.junit.Assert.assertEquals;
21+
import static org.junit.Assert.assertTrue;
2122
import static org.springframework.http.HttpHeaders.AUTHORIZATION;
2223
import static org.springframework.http.MediaType.APPLICATION_JSON;
2324

@@ -88,16 +89,15 @@ public void deleteUser_ExistingTokens_TokensDeletedSuccess() {
8889
val deleteUserResponse = initStringRequest().endpoint("/users/%s", userDelete.getId()).delete();
8990

9091
val deleteStatusCode = deleteUserResponse.getStatusCode();
91-
assertThat(deleteStatusCode).isEqualTo(HttpStatus.OK);
92+
assertEquals(deleteStatusCode, HttpStatus.OK);
9293

9394
val checkTokenAfterDeleteResponse = checkToken(tokenToDelete);
9495
// Should be revoked
95-
assertThat(checkTokenAfterDeleteResponse.getStatusCode()).isEqualTo(HttpStatus.UNAUTHORIZED);
96+
assertEquals(checkTokenAfterDeleteResponse.getStatusCode(), HttpStatus.UNAUTHORIZED);
9697

9798
val checkTokenRemainedAfterDeleteResponse = checkToken(tokenToKeep);
9899
// Should be valid
99-
assertThat(checkTokenRemainedAfterDeleteResponse.getStatusCode())
100-
.isEqualTo(HttpStatus.MULTI_STATUS);
100+
assertEquals(checkTokenRemainedAfterDeleteResponse.getStatusCode(), HttpStatus.MULTI_STATUS);
101101
}
102102

103103
/**
@@ -117,16 +117,15 @@ public void deletePolicy_ExistingTokens_TokensDeletedSuccess() {
117117
val deletePolicyResponse =
118118
initStringRequest().endpoint("/policies/%s", policy1.getId()).delete();
119119
val deleteStatusCode = deletePolicyResponse.getStatusCode();
120-
assertThat(deleteStatusCode).isEqualTo(HttpStatus.OK);
120+
assertEquals(deleteStatusCode, HttpStatus.OK);
121121

122122
val checkTokenAfterDeleteResponse = checkToken(tokenToDelete);
123123
// Should be revoked
124-
assertThat(checkTokenAfterDeleteResponse.getStatusCode()).isEqualTo(HttpStatus.UNAUTHORIZED);
124+
assertEquals(checkTokenAfterDeleteResponse.getStatusCode(), HttpStatus.UNAUTHORIZED);
125125

126126
val checkTokenRemainedAfterDeleteResponse = checkToken(tokenToKeep);
127127
// Should be valid
128-
assertThat(checkTokenRemainedAfterDeleteResponse.getStatusCode())
129-
.isEqualTo(HttpStatus.MULTI_STATUS);
128+
assertEquals(checkTokenRemainedAfterDeleteResponse.getStatusCode(), HttpStatus.MULTI_STATUS);
130129
}
131130

132131
/**
@@ -160,8 +159,8 @@ private String setupUserWithToken(User user, Policy policy) {
160159
val checkTokenResponse = checkToken(accessToken);
161160

162161
val checkStatusCode = checkTokenResponse.getStatusCode();
163-
assertThat(checkStatusCode).isEqualTo(HttpStatus.MULTI_STATUS);
164-
assertThat(checkTokenResponse.getBody()).contains(policy.getName() + "." + "WRITE");
162+
assertEquals(checkStatusCode, HttpStatus.MULTI_STATUS);
163+
assertTrue(checkTokenResponse.getBody().contains(policy.getName() + "." + "WRITE"));
165164

166165
return accessToken;
167166
}

‎src/test/java/bio/overture/ego/grpc/service/UserServiceGrpcAuthTest.java

+39-25
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@
22

33
import static bio.overture.ego.utils.EntityGenerator.generateNonExistentId;
44
import static io.grpc.Metadata.ASCII_STRING_MARSHALLER;
5-
import static org.assertj.core.api.Assertions.assertThat;
6-
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
5+
import static org.junit.Assert.*;
76

87
import bio.overture.ego.grpc.GetUserRequest;
98
import bio.overture.ego.grpc.ListUsersRequest;
@@ -133,18 +132,20 @@ public void testDataSetup() {
133132
}
134133
}
135134

136-
@Test
135+
@Test()
137136
public void getUser_noAuth_rejected() {
138137

139138
val noAuthStub = MetadataUtils.attachHeaders(stub, emptyAuthMeta);
140139

141140
// Test that the interceptor rejects this request
142-
assertThatExceptionOfType(StatusRuntimeException.class)
143-
.as("Request should be rejected due to missing JWT")
144-
.isThrownBy(
145-
() ->
146-
noAuthStub.getUser(
147-
GetUserRequest.newBuilder().setId(UUID.randomUUID().toString()).build()));
141+
try {
142+
noAuthStub.getUser(GetUserRequest.newBuilder().setId(UUID.randomUUID().toString()).build());
143+
} catch (Exception e) {
144+
assertEquals(
145+
"Request should be rejected due to missing JWT",
146+
StatusRuntimeException.class,
147+
e.getClass());
148+
}
148149
}
149150

150151
@Test
@@ -155,7 +156,7 @@ public void getUser_userAuth_success() {
155156
// Test that the interceptor rejects this request
156157
val reply =
157158
authStub.getUser(GetUserRequest.newBuilder().setId(testUser.getId().toString()).build());
158-
assertThat(reply.getId().getValue()).isEqualTo(testUser.getId().toString());
159+
assertEquals(reply.getId().getValue(), testUser.getId().toString());
159160
}
160161

161162
@Test
@@ -164,11 +165,14 @@ public void getUser_userAuth_rejectedForWrongUser() {
164165
val authStub = MetadataUtils.attachHeaders(stub, userAuthMeta);
165166
UUID randomId = generateNonExistentId(userService);
166167

167-
// Test that the interceptor rejects this request
168-
assertThatExceptionOfType(StatusRuntimeException.class)
169-
.as("User should not be allowed to access data of a different user.")
170-
.isThrownBy(
171-
() -> authStub.getUser(GetUserRequest.newBuilder().setId(randomId.toString()).build()));
168+
try {
169+
authStub.getUser(GetUserRequest.newBuilder().setId(randomId.toString()).build());
170+
} catch (Exception e) {
171+
assertEquals(
172+
"User should not be allowed to access data of a different user.",
173+
StatusRuntimeException.class,
174+
e.getClass());
175+
}
172176
}
173177

174178
@Test
@@ -178,7 +182,7 @@ public void getUser_adminAuth_success() {
178182
// Test that the interceptor rejects this request
179183
val reply =
180184
authStub.getUser(GetUserRequest.newBuilder().setId(testUser.getId().toString()).build());
181-
assertThat(reply.getId().getValue()).isEqualTo(testUser.getId().toString());
185+
assertEquals(reply.getId().getValue(), testUser.getId().toString());
182186
}
183187

184188
@Test
@@ -188,17 +192,22 @@ public void getUser_appAuth_success() {
188192
// Test that the interceptor rejects this request
189193
val reply =
190194
authStub.getUser(GetUserRequest.newBuilder().setId(testUser.getId().toString()).build());
191-
assertThat(reply.getId().getValue()).isEqualTo(testUser.getId().toString());
195+
assertEquals(reply.getId().getValue(), (testUser.getId().toString()));
192196
}
193197

194198
@Test
195199
public void listUsers_noAuth_rejected() {
196200
val authStub = MetadataUtils.attachHeaders(stub, emptyAuthMeta);
197201

198202
// Test that the interceptor rejects this request
199-
assertThatExceptionOfType(StatusRuntimeException.class)
200-
.as("Request should be rejected due to missing JWT")
201-
.isThrownBy(() -> authStub.listUsers(ListUsersRequest.newBuilder().build()));
203+
try {
204+
authStub.listUsers(ListUsersRequest.newBuilder().build());
205+
} catch (Exception e) {
206+
assertEquals(
207+
"Request should be rejected due to missing JWT",
208+
StatusRuntimeException.class,
209+
e.getClass());
210+
}
202211
}
203212

204213
@Test
@@ -207,9 +216,14 @@ public void listUsers_userAuth_rejected() {
207216
val authStub = MetadataUtils.attachHeaders(stub, userAuthMeta);
208217

209218
// Test that the interceptor rejects this request
210-
assertThatExceptionOfType(StatusRuntimeException.class)
211-
.as("Request should be rejected due to missing JWT")
212-
.isThrownBy(() -> authStub.listUsers(ListUsersRequest.newBuilder().build()));
219+
try {
220+
authStub.listUsers(ListUsersRequest.newBuilder().build());
221+
} catch (Exception e) {
222+
assertEquals(
223+
"Request should be rejected due to missing JWT",
224+
StatusRuntimeException.class,
225+
e.getClass());
226+
}
213227
}
214228

215229
@Test
@@ -218,7 +232,7 @@ public void listUsers_adminAuth_success() {
218232

219233
// Test that the interceptor rejects this request
220234
val reply = authStub.listUsers(ListUsersRequest.newBuilder().build());
221-
assertThat(reply.getUsersCount()).isGreaterThanOrEqualTo(2);
235+
assertTrue(reply.getUsersCount() >= 2);
222236
}
223237

224238
@Test
@@ -227,6 +241,6 @@ public void listUsers_appAuth_success() {
227241

228242
// Test that the interceptor rejects this request
229243
val reply = authStub.listUsers(ListUsersRequest.newBuilder().build());
230-
assertThat(reply.getUsersCount()).isGreaterThanOrEqualTo(2);
244+
assertTrue(reply.getUsersCount() >= 2);
231245
}
232246
}

‎src/test/java/bio/overture/ego/grpc/service/UserServiceGrpcTest.java

+33-33
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import static bio.overture.ego.utils.CollectionUtils.repeatedCallsOf;
44
import static java.util.stream.Collectors.toList;
5-
import static org.assertj.core.api.Assertions.assertThat;
5+
import static org.junit.Assert.*;
66

77
import bio.overture.ego.grpc.GetUserRequest;
88
import bio.overture.ego.grpc.ListUsersRequest;
@@ -125,21 +125,21 @@ public void getUser_success() {
125125
stub.getUser(GetUserRequest.newBuilder().setId(testUser.getId().toString()).build());
126126

127127
// Ensure all fields populated and value matches expected
128-
assertThat(reply.getId().getValue()).isEqualTo(testUser.getId().toString());
129-
assertThat(reply.getFirstName().getValue()).isEqualTo(testUser.getFirstName());
130-
assertThat(reply.getLastName().getValue()).isEqualTo(testUser.getLastName());
131-
assertThat(reply.getEmail().getValue()).isEqualTo(testUser.getEmail());
132-
assertThat(reply.getStatus().getValue()).isEqualTo(testUser.getStatus().toString());
133-
assertThat(reply.getPreferredLanguage().getValue())
134-
.isEqualTo(testUser.getPreferredLanguage().toString());
135-
assertThat(reply.getType().getValue()).isEqualTo(testUser.getType().toString());
136-
137-
assertThat(reply.hasCreatedAt()).isTrue();
138-
assertThat(reply.hasLastLogin()).isTrue();
139-
140-
assertThat(reply.getApplicationsList().size()).isEqualTo(1);
141-
assertThat(reply.getGroupsList().size()).isEqualTo(1);
142-
assertThat(reply.getScopesList().size()).isEqualTo(2);
128+
assertEquals(reply.getId().getValue(), testUser.getId().toString());
129+
assertEquals(reply.getFirstName().getValue(), testUser.getFirstName());
130+
assertEquals(reply.getLastName().getValue(), testUser.getLastName());
131+
assertEquals(reply.getEmail().getValue(), testUser.getEmail());
132+
assertEquals(reply.getStatus().getValue(), testUser.getStatus().toString());
133+
assertEquals(
134+
reply.getPreferredLanguage().getValue(), testUser.getPreferredLanguage().toString());
135+
assertEquals(reply.getType().getValue(), testUser.getType().toString());
136+
137+
assertTrue(reply.hasCreatedAt());
138+
assertTrue(reply.hasLastLogin());
139+
140+
assertEquals(reply.getApplicationsList().size(), 1);
141+
assertEquals(reply.getGroupsList().size(), 1);
142+
assertEquals(reply.getScopesList().size(), 2);
143143
}
144144

145145
@Test
@@ -149,13 +149,13 @@ public void listUser_emptyRequest() {
149149
val reply = stub.listUsers(request);
150150

151151
// Ensure response includes pagination data
152-
assertThat(reply.hasPage()).isTrue();
153-
assertThat(reply.getPage().getMaxResults()).isGreaterThanOrEqualTo(7);
152+
assertTrue(reply.hasPage());
153+
assertTrue(reply.getPage().getMaxResults() >= 7);
154154

155155
// Make sure we got users in the response (quick sanity check, not in depth)
156-
assertThat(reply.getUsersCount()).isGreaterThanOrEqualTo(7);
156+
assertTrue(reply.getUsersCount() >= 7);
157157
val user = reply.getUsers(0);
158-
assertThat(user.hasId()).isTrue();
158+
assertTrue(user.hasId());
159159
}
160160

161161
@Test
@@ -166,13 +166,13 @@ public void listUser_pagedRequests() {
166166
val reply1 = stub.listUsers(request1);
167167

168168
// Correct number of users
169-
assertThat(reply1.getUsersCount()).isEqualTo(2);
169+
assertEquals(reply1.getUsersCount(), 2);
170170

171171
// Correct pagination info
172-
assertThat(reply1.hasPage()).isTrue();
173-
assertThat(reply1.getPage().getMaxResults()).isGreaterThanOrEqualTo(7);
174-
assertThat(reply1.getPage().hasNextPage()).isTrue();
175-
assertThat(reply1.getPage().getNextPage().getValue()).isEqualTo(1);
172+
assertTrue(reply1.hasPage());
173+
assertTrue(reply1.getPage().getMaxResults() >= 7);
174+
assertTrue(reply1.getPage().hasNextPage());
175+
assertEquals(reply1.getPage().getNextPage().getValue(), 1);
176176

177177
val user1 = reply1.getUsers(0);
178178

@@ -185,13 +185,13 @@ public void listUser_pagedRequests() {
185185
val reply2 = stub.listUsers(request2);
186186

187187
// Correct pagination info
188-
assertThat(reply2.hasPage()).isTrue();
189-
assertThat(reply2.getPage().getMaxResults()).isGreaterThanOrEqualTo(7);
190-
assertThat(reply2.getPage().getNextPage().getValue()).isEqualTo(2);
188+
assertTrue(reply2.hasPage());
189+
assertTrue(reply2.getPage().getMaxResults() >= 7);
190+
assertEquals(reply2.getPage().getNextPage().getValue(), 2);
191191

192192
// different user (ensure we're not repeating user blocks)
193193
val user2 = reply2.getUsers(0);
194-
assertThat(user1.getId()).isNotEqualTo(user2.getId());
194+
assertFalse(user1.getId().equals(user2.getId()));
195195
}
196196

197197
@Test
@@ -202,8 +202,8 @@ public void listUser_largePageRequest() {
202202
val reply = stub.listUsers(request);
203203

204204
// Correct number of users
205-
assertThat(reply.getUsersCount()).isGreaterThanOrEqualTo(7);
206-
assertThat(reply.getUsersCount()).isLessThanOrEqualTo(1000);
205+
assertTrue(reply.getUsersCount() >= 7);
206+
assertTrue(reply.getUsersCount() <= 1000);
207207
}
208208

209209
@Test
@@ -214,7 +214,7 @@ public void listUser_nonExistentPageNumberRequest() {
214214
val reply = stub.listUsers(request);
215215

216216
// Correct number of users
217-
assertThat(reply.getUsersCount()).isEqualTo(0);
218-
assertThat(reply.getPage().hasNextPage()).isFalse();
217+
assertEquals(reply.getUsersCount(), 0);
218+
assertFalse(reply.getPage().hasNextPage());
219219
}
220220
}

‎src/test/java/bio/overture/ego/model/enums/AccessLevelTest.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import static bio.overture.ego.model.enums.AccessLevel.DENY;
44
import static bio.overture.ego.model.enums.AccessLevel.READ;
55
import static bio.overture.ego.model.enums.AccessLevel.WRITE;
6-
import static org.assertj.core.api.Assertions.assertThat;
6+
import static org.junit.Assert.assertEquals;
77
import static org.junit.Assert.assertFalse;
88
import static org.junit.Assert.assertTrue;
99

@@ -23,9 +23,9 @@
2323
public class AccessLevelTest {
2424
@Test
2525
public void testFromValue() {
26-
assertThat(AccessLevel.fromValue("read")).isEqualByComparingTo(AccessLevel.READ);
27-
assertThat(AccessLevel.fromValue("write")).isEqualByComparingTo(AccessLevel.WRITE);
28-
assertThat(AccessLevel.fromValue("deny")).isEqualByComparingTo(AccessLevel.DENY);
26+
assertEquals(AccessLevel.fromValue("read"), AccessLevel.READ);
27+
assertEquals(AccessLevel.fromValue("write"), AccessLevel.WRITE);
28+
assertEquals(AccessLevel.fromValue("deny"), AccessLevel.DENY);
2929
}
3030

3131
@Test

‎src/test/java/bio/overture/ego/utils/EntityGenerator.java

+4-3
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@
1313
import static java.util.Arrays.stream;
1414
import static java.util.stream.Collectors.toList;
1515
import static java.util.stream.Collectors.toSet;
16-
import static org.assertj.core.api.Assertions.assertThat;
16+
import static org.junit.Assert.assertEquals;
17+
import static org.junit.Assert.assertTrue;
1718

1819
import bio.overture.ego.model.dto.*;
1920
import bio.overture.ego.model.entity.Application;
@@ -308,7 +309,7 @@ public Policy setupGroupPermission(Group group, Policy policy, AccessLevel level
308309

309310
public Policy setupPolicy(@NonNull String csv) {
310311
val args = newArrayList(COMMA_SPLITTER.split(csv));
311-
assertThat(args).hasSize(2);
312+
assertEquals(args.size(), 2);
312313
val name = args.get(0);
313314

314315
return setupSinglePolicy(name);
@@ -427,7 +428,7 @@ public static int randomBoundedInt(int maxExclusive) {
427428
}
428429

429430
public static int randomBoundedInt(int minInclusive, int maxExclusive) {
430-
assertThat(MAX_VALUE - maxExclusive).isGreaterThan(minInclusive);
431+
assertTrue((MAX_VALUE - maxExclusive) > minInclusive);
431432
return minInclusive + randomBoundedInt(maxExclusive);
432433
}
433434

‎src/test/java/selenium/LoadAdminUITest.java

+3-4
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,12 @@
1919

2020
import static bio.overture.ego.model.enums.ApplicationType.ADMIN;
2121
import static bio.overture.ego.model.enums.StatusType.APPROVED;
22+
import static org.junit.Assert.*;
2223

2324
import bio.overture.ego.model.dto.CreateApplicationRequest;
2425
import bio.overture.ego.service.ApplicationService;
2526
import lombok.SneakyThrows;
2627
import lombok.val;
27-
import org.assertj.core.api.Assertions;
2828
import org.junit.Test;
2929
import org.openqa.selenium.By;
3030
import org.springframework.beans.factory.annotation.Autowired;
@@ -56,7 +56,7 @@ public void loadAdmin_Success() {
5656
driver.get("http://localhost:" + uiPort);
5757
val titleText =
5858
driver.findElement(By.className("Login")).findElement(By.tagName("h1")).getText();
59-
Assertions.assertThat(titleText).isEqualTo("Admin Portal");
59+
assertEquals(titleText, "Admin Portal");
6060

6161
driver.findElement(By.className("fa-facebook")).click();
6262

@@ -76,8 +76,7 @@ public void loadAdmin_Success() {
7676
.findElement(By.tagName("div"))
7777
.findElement(By.tagName("div"))
7878
.getText();
79-
Assertions.assertThat(messageDiv)
80-
.contains("Your account does not have an administrator userType.");
79+
assertTrue(messageDiv.contains("Your account does not have an administrator userType."));
8180

8281
Thread.sleep(1000);
8382
}

0 commit comments

Comments
 (0)
Please sign in to comment.