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 <heinrich.bo...@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