Merge pull request #1087 from CosmWasm/Uint128-fraction

Update Fraction implementation in Decimal to Uint128

CHANGELOG.md

@@ -21,6 +21,8 @@ and this project adheres to
   `do_instantiate`, `do_execute`, `do_migrate`, `do_sudo`, `do_reply`,
   `do_query`, `ibc_channel_open`, `ibc_channel_connect`, `ibc_channel_close`,
   `ibc_packet_receive`, `ibc_packet_ack` and `ibc_packet_timeout`.
+- cosmwasm-std: In `Decimal` change `Fraction<u128>` to `Fraction<Uint128>`,
+  such that `Decimal::numerator` and `::denominator` now return `Uint128`.
 
 ### Removed
 

packages/std/src/math/decimal.rs

@@ -95,15 +95,15 @@ impl Decimal {
     }
 }
 
-impl Fraction<u128> for Decimal {
+impl Fraction<Uint128> for Decimal {
     #[inline]
-    fn numerator(&self) -> u128 {
-        self.0.u128()
+    fn numerator(&self) -> Uint128 {
+        self.0
     }
 
     #[inline]
-    fn denominator(&self) -> u128 {
-        Self::DECIMAL_FRACTIONAL.u128()
+    fn denominator(&self) -> Uint128 {
+        Self::DECIMAL_FRACTIONAL
     }
 
     /// Returns the multiplicative inverse `1/d` for decimal `d`.
@@ -360,8 +360,14 @@ mod tests {
     #[test]
     fn decimal_implements_fraction() {
         let fraction = Decimal::from_str("1234.567").unwrap();
-        assert_eq!(fraction.numerator(), 1_234_567_000_000_000_000_000u128);
-        assert_eq!(fraction.denominator(), 1_000_000_000_000_000_000u128);
+        assert_eq!(
+            fraction.numerator(),
+            Uint128::from(1_234_567_000_000_000_000_000u128)
+        );
+        assert_eq!(
+            fraction.denominator(),
+            Uint128::from(1_000_000_000_000_000_000u128)
+        );
     }
 
     #[test]