Skip to content

Commit ab615f4

Browse files
committed
optimize testing assertions
Signed-off-by: Alex Knop <[email protected]>
1 parent 30d972a commit ab615f4

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

app/src/androidTest/java/com/owncloud/android/ui/activity/ConflictsResolveActivityIT.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ public void cancel() {
167167
ConflictsResolveActivity sut = activityRule.launchActivity(intent);
168168

169169
sut.listener = decision -> {
170-
assertEquals(decision, ConflictsResolveDialog.Decision.CANCEL);
170+
assertEquals(ConflictsResolveDialog.Decision.CANCEL, decision);
171171
returnCode = true;
172172
};
173173

@@ -210,7 +210,7 @@ public void keepExisting() {
210210
ConflictsResolveActivity sut = activityRule.launchActivity(intent);
211211

212212
sut.listener = decision -> {
213-
assertEquals(decision, ConflictsResolveDialog.Decision.KEEP_SERVER);
213+
assertEquals(ConflictsResolveDialog.Decision.KEEP_SERVER, decision);
214214
returnCode = true;
215215
};
216216

@@ -256,7 +256,7 @@ public void keepNew() {
256256
ConflictsResolveActivity sut = activityRule.launchActivity(intent);
257257

258258
sut.listener = decision -> {
259-
assertEquals(decision, ConflictsResolveDialog.Decision.KEEP_LOCAL);
259+
assertEquals(ConflictsResolveDialog.Decision.KEEP_LOCAL, decision);
260260
returnCode = true;
261261
};
262262

@@ -303,7 +303,7 @@ public void keepBoth() {
303303
ConflictsResolveActivity sut = activityRule.launchActivity(intent);
304304

305305
sut.listener = decision -> {
306-
assertEquals(decision, ConflictsResolveDialog.Decision.KEEP_BOTH);
306+
assertEquals(ConflictsResolveDialog.Decision.KEEP_BOTH, decision);
307307
returnCode = true;
308308
};
309309

app/src/androidTest/java/com/owncloud/android/util/EncryptionTestIT.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@
7979
import static org.junit.Assert.assertEquals;
8080
import static org.junit.Assert.assertNotEquals;
8181
import static org.junit.Assert.assertNull;
82+
import static org.junit.Assert.assertArrayEquals;
8283

8384
public class EncryptionTestIT extends AbstractIT {
8485
@Rule public RetryTestRule retryTestRule = new RetryTestRule();
@@ -149,7 +150,7 @@ public void encryptStringAsymmetric() throws Exception {
149150

150151
byte[] key2 = decodeStringToBase64Bytes(decryptedString);
151152

152-
assertTrue(Arrays.equals(key1, key2));
153+
assertArrayEquals(key1, key2);
153154
}
154155

155156
@Test
@@ -164,7 +165,7 @@ public void encryptStringAsymmetricCorrectPublicKey() throws Exception {
164165

165166
byte[] key2 = decodeStringToBase64Bytes(decryptedString);
166167

167-
assertTrue(Arrays.equals(key1, key2));
168+
assertArrayEquals(key1, key2);
168169
}
169170

170171
@Test(expected = BadPaddingException.class)

0 commit comments

Comments
 (0)