Browse Source

Fix expected/actual Assertions order in SimpleCalculatorTest

Harry de Boer 5 years ago
parent
commit
3445fa1a1d

+ 6 - 6
backend/src/test/java/nl/quintor/hdboer/aegoncalc/services/SimpleCalculatorTest.java

@@ -12,8 +12,8 @@ class SimpleCalculatorTest {
     @DisplayName("add: test integers do not over- or underflow")
     void add() {
         Assertions.assertAll(
-                () -> Assertions.assertEquals(calculator.add(1, Integer.MAX_VALUE), 2147483648.0),
-                () -> Assertions.assertEquals(calculator.add(-1, Integer.MIN_VALUE), -2147483649.0)
+                () -> Assertions.assertEquals(2147483648.0, calculator.add(1, Integer.MAX_VALUE)),
+                () -> Assertions.assertEquals(-2147483649.0, calculator.add(-1, Integer.MIN_VALUE))
         );
     }
 
@@ -21,21 +21,21 @@ class SimpleCalculatorTest {
     @DisplayName("subtract: test integers do not over- or underflow")
     void subtract() {
         Assertions.assertAll(
-                () -> Assertions.assertEquals(calculator.subtract(-1, Integer.MAX_VALUE), -2147483648.0),
-                () -> Assertions.assertEquals(calculator.subtract(1, Integer.MIN_VALUE), 2147483649.0)
+                () -> Assertions.assertEquals(-2147483648.0, calculator.subtract(-1, Integer.MAX_VALUE)),
+                () -> Assertions.assertEquals(2147483649.0, calculator.subtract(1, Integer.MIN_VALUE))
         );
     }
 
     @Test
     @DisplayName("multiply: test integers do not overflow")
     void multiply() {
-        Assertions.assertEquals(calculator.multiply(2, Integer.MAX_VALUE), 4294967294.0);
+        Assertions.assertEquals(4294967294.0, calculator.multiply(2, Integer.MAX_VALUE));
     }
 
     @Test
     @DisplayName("divide: test non-integer fraction is retained when dividing")
     void divide() {
-        Assertions.assertEquals(calculator.divide(10, 8), 1.25);
+        Assertions.assertEquals(1.25, calculator.divide(10, 8));
     }
 
     @Test