You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by er...@apache.org on 2019/06/13 20:51:52 UTC

[commons-numbers] 03/06: NUMBERS-100: Swap parameters in Assert.assertEquals(long, long) invocation

This is an automated email from the ASF dual-hosted git repository.

erans pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-numbers.git

commit c23a3c5b54d23c911563e8e5e0d068d6bc1953af
Author: Schamschi <he...@gmx.at>
AuthorDate: Sun Jun 9 21:23:41 2019 +0200

    NUMBERS-100: Swap parameters in Assert.assertEquals(long, long) invocation
    
    Swap parameters in an invocation of Assert.assertEquals(long, long) from the method org.apache.commons.numbers.fraction.FractionTest.testGetReducedFraction(). The first parameter should be the expected value, the second parameter the actual value.
---
 .../org/apache/commons/numbers/fraction/FractionTest.java    | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/FractionTest.java b/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/FractionTest.java
index e41ab5d..e4ce493 100644
--- a/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/FractionTest.java
+++ b/commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/FractionTest.java
@@ -595,10 +595,14 @@ public class FractionTest {
         } catch (ArithmeticException ignored) {
             // expected
         }
-        Assert.assertEquals(Fraction.getReducedFraction
-                (2, Integer.MIN_VALUE).getNumerator(),-1);
-        Assert.assertEquals(Fraction.getReducedFraction
-                (1, -1).getNumerator(), -1);
+        Assert.assertEquals(
+                -1,
+                Fraction.getReducedFraction(2, Integer.MIN_VALUE).getNumerator()
+        );
+        Assert.assertEquals(
+                -1,
+                Fraction.getReducedFraction(1, -1).getNumerator()
+        );
     }
 
     @Test