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