From 864160280872fdb400c2e0e61aaaa1b858fba3f8 Mon Sep 17 00:00:00 2001 From: Torsten Grote Date: Tue, 24 Mar 2020 09:08:00 -0300 Subject: Allow fractional withdrawals with cashier app --- .../src/main/java/net/taler/common/Amount.kt | 27 ++++++++++++++++++---- .../src/test/java/net/taler/common/AmountTest.kt | 21 +++++++++++++++++ 2 files changed, 43 insertions(+), 5 deletions(-) (limited to 'taler-kotlin-common') diff --git a/taler-kotlin-common/src/main/java/net/taler/common/Amount.kt b/taler-kotlin-common/src/main/java/net/taler/common/Amount.kt index 48bd643..49b699f 100644 --- a/taler-kotlin-common/src/main/java/net/taler/common/Amount.kt +++ b/taler-kotlin-common/src/main/java/net/taler/common/Amount.kt @@ -64,7 +64,7 @@ data class Amount( * of 50_000_000 would correspond to 50 cents. */ val fraction: Int -) { +) : Comparable { companion object { @@ -88,10 +88,14 @@ data class Amount( fun fromJSONString(str: String): Amount { val split = str.split(":") if (split.size != 2) throw AmountParserException("Invalid Amount Format") - // currency - val currency = checkCurrency(split[0]) + return fromString(split[0], split[1]) + } + + @Throws(AmountParserException::class) + @SuppressLint("CheckedExceptions") + fun fromString(currency: String, str: String): Amount { // value - val valueSplit = split[1].split(".") + val valueSplit = str.split(".") val value = checkValue(valueSplit[0].toLongOrNull()) // fraction val fraction: Int = if (valueSplit.size > 1) { @@ -103,7 +107,7 @@ data class Amount( ?.roundToInt() checkFraction(fraction) } else 0 - return Amount(currency, value, fraction) + return Amount(checkCurrency(currency), value, fraction) } @Throws(AmountParserException::class) @@ -197,4 +201,17 @@ data class Amount( return "$amountStr $currency" } + override fun compareTo(other: Amount): Int { + check(currency == other.currency) { "Can only compare amounts with the same currency" } + when { + value == other.value -> { + if (fraction < other.fraction) return -1 + if (fraction > other.fraction) return 1 + return 0 + } + value < other.value -> return -1 + else -> return 1 + } + } + } diff --git a/taler-kotlin-common/src/test/java/net/taler/common/AmountTest.kt b/taler-kotlin-common/src/test/java/net/taler/common/AmountTest.kt index c09da3c..97d9667 100644 --- a/taler-kotlin-common/src/test/java/net/taler/common/AmountTest.kt +++ b/taler-kotlin-common/src/test/java/net/taler/common/AmountTest.kt @@ -275,6 +275,27 @@ class AmountTest { assertFalse(Amount.fromJSONString("EUR:0001.0").isZero()) } + @Test + fun `test comparision`() { + assertTrue(Amount.fromJSONString("EUR:0") <= Amount.fromJSONString("EUR:0")) + assertTrue(Amount.fromJSONString("EUR:0") <= Amount.fromJSONString("EUR:0.00000001")) + assertTrue(Amount.fromJSONString("EUR:0") < Amount.fromJSONString("EUR:0.00000001")) + assertTrue(Amount.fromJSONString("EUR:0") < Amount.fromJSONString("EUR:1")) + assertTrue(Amount.fromJSONString("EUR:0") == Amount.fromJSONString("EUR:0")) + assertTrue(Amount.fromJSONString("EUR:42") == Amount.fromJSONString("EUR:42")) + assertTrue(Amount.fromJSONString("EUR:42.00000001") == Amount.fromJSONString("EUR:42.00000001")) + assertTrue(Amount.fromJSONString("EUR:42.00000001") >= Amount.fromJSONString("EUR:42.00000001")) + assertTrue(Amount.fromJSONString("EUR:42.00000002") >= Amount.fromJSONString("EUR:42.00000001")) + assertTrue(Amount.fromJSONString("EUR:42.00000002") > Amount.fromJSONString("EUR:42.00000001")) + assertTrue(Amount.fromJSONString("EUR:0.00000002") > Amount.fromJSONString("EUR:0.00000001")) + assertTrue(Amount.fromJSONString("EUR:0.00000001") > Amount.fromJSONString("EUR:0")) + assertTrue(Amount.fromJSONString("EUR:2") > Amount.fromJSONString("EUR:1")) + + assertThrows("could compare amounts with different currencies") { + Amount.fromJSONString("EUR:0.5") < Amount.fromJSONString("USD:0.50000001") + } + } + private inline fun assertThrows( msg: String? = null, function: () -> Any -- cgit v1.2.3