gaspersic/java-design-patterns
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #1463 from charlesfinley/master

Browse Source 
Fixes Issue #1462 - Unit Test assertEquals parameters are reversed
This commit is contained in:
Ilkka Seppälä 2020-08-01 10:39:20 +03:00 committed by GitHub
commit 43569ecd66
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
arrange-act-assert/src/test/java/com/iluwatar/arrangeactassert/CashAAATest.java
View File

@ -60,7 +60,7 @@ public class CashAAATest {
//Act //Act
cash.plus(4); cash.plus(4);
//Assert //Assert
assertEquals(cash.count(), 7); assertEquals(7, cash.count());
} }
@Test @Test
@ -71,7 +71,7 @@ public class CashAAATest {
var result = cash.minus(5); var result = cash.minus(5);
//Assert //Assert
assertTrue(result); assertTrue(result);
assertEquals(cash.count(), 3); assertEquals(3, cash.count());
} }
@Test @Test
@ -82,7 +82,7 @@ public class CashAAATest {
var result = cash.minus(6); var result = cash.minus(6);
//Assert //Assert
assertFalse(result); assertFalse(result);
assertEquals(cash.count(), 1); assertEquals(1, cash.count());
} }
@Test @Test
@ -94,6 +94,6 @@ public class CashAAATest {
var result = cash.minus(3); var result = cash.minus(3);
//Assert //Assert
assertTrue(result); assertTrue(result);
assertEquals(cash.count(), 8); assertEquals(8, cash.count());
} }
} }

8
arrange-act-assert/src/test/java/com/iluwatar/arrangeactassert/CashAntiAAATest.java
View File

@ -44,16 +44,16 @@ public class CashAntiAAATest {
var cash = new Cash(3); var cash = new Cash(3);
//test plus //test plus
cash.plus(4); cash.plus(4);
assertEquals(cash.count(), 7); assertEquals(7, cash.count());
//test minus //test minus
cash = new Cash(8); cash = new Cash(8);
assertTrue(cash.minus(5)); assertTrue(cash.minus(5));
assertEquals(cash.count(), 3); assertEquals(3, cash.count());
assertFalse(cash.minus(6)); assertFalse(cash.minus(6));
assertEquals(cash.count(), 3); assertEquals(3, cash.count());
//test update //test update
cash.plus(5); cash.plus(5);
assertTrue(cash.minus(5)); assertTrue(cash.minus(5));
assertEquals(cash.count(), 3); assertEquals(3, cash.count());
} }
} }