try to restore original code

Author: dkuku

native/explorer/src/datatypes.rs

@@ -635,7 +635,7 @@ impl ExDecimal {
             base.checked_mul(10_i128.pow(self.exp as u32))
                 .ok_or_else(|| {
                     ExplorerError::Other(
-                        "cannot decode a valid decimal from term; check that `coef` fits into an `i128`. error: throw(<term>)".to_string()
+                        "decimal coefficient overflow: value exceeds i128 limits".to_string(),
                     )
                 })
         } else {
@@ -659,7 +659,7 @@ impl ExDecimal {
 
 impl Literal for ExDecimal {
     fn lit(self) -> Expr {
-        let coef = self.signed_coef().unwrap();
+        let coef = self.signed_coef().expect("decimal coefficient overflow");
         let scale = self.scale();
 
         Expr::Literal(LiteralValue::Scalar(Scalar::new(
@@ -727,7 +727,6 @@ impl<'a> rustler::Decoder<'a> for ExValidValue<'a> {
                 } else if let Ok(duration) = term.decode::<ExDuration>() {
                     Ok(ExValidValue::Duration(duration))
                 } else if let Ok(decimal) = term.decode::<ExDecimal>() {
-                    decimal.signed_coef().map_err(|_| rustler::Error::BadArg)?;
                     Ok(ExValidValue::Decimal(decimal))
                 } else {
                     Err(rustler::Error::BadArg)

native/explorer/src/series/from_list.rs

@@ -240,16 +240,17 @@ pub fn s_from_list_decimal(
                     })
             }
 
-            TermType::Map => item
-                .decode::<ExDecimal>()
-                .map_err(|error| {
-                    ExplorerError::Other(format!(
-                        "cannot decode a valid decimal from term; check that `coef` fits into an `i128`. error: {error:?}"
-                    ))
-                })
-                .and_then(|ex_decimal| {
-                    ex_decimal.signed_coef().map(|coef| AnyValue::Decimal(coef, ex_decimal.scale()))
-                }),
+            TermType::Map => {
+                let ex_decimal = item
+                    .decode::<ExDecimal>()
+                    .map_err(|error| {
+                        ExplorerError::Other(format!(
+                            "cannot decode a valid decimal from term; check that `coef` fits into an `i128`. error: {error:?}"
+                        ))
+                    })?;
+                let coef = ex_decimal.signed_coef()?;
+                Ok(AnyValue::Decimal(coef, ex_decimal.scale()))
+            },
 
             TermType::Atom => Ok(AnyValue::Null),
 

test/explorer/series_test.exs

@@ -2026,8 +2026,7 @@ defmodule Explorer.SeriesTest do
 
     test "overflow with values exceeding i128 limits" do
       assert_raise RuntimeError,
-                   "Generic Error: cannot decode a valid decimal from term;" <>
-                     " check that `coef` fits into an `i128`. error: throw(<term>)",
+                   "Generic Error: decimal coefficient overflow: value exceeds i128 limits",
                    fn -> Series.from_list([Decimal.new("3.4e38")]) end
     end
   end