Fix test

Signed-off-by: Moritz Hoffmann <mh@materialize.com>

Author: antiguru

src/compute/src/render/reduce.rs

@@ -2038,7 +2038,7 @@ fn finalize_accum<'a>(aggr_func: &'a AggregateFunc, accum: &'a Accum, total: Dif
     }
 }
 
-/// The type for accumulator counting. Set to [`Overflowing<u128>`]
+/// The type for accumulator counting. Set to [`Overflowing<u128>`](mz_ore::Overflowing).
 type AccumCount = mz_ore::Overflowing<i128>;
 
 /// Accumulates values for the various types of accumulable aggregations.

src/ore/src/overflowing.rs

@@ -19,7 +19,6 @@
 #[cfg(feature = "proptest")]
 use proptest_derive::Arbitrary;
 use serde::{Deserialize, Serialize};
-#[cfg(feature = "proptest")]
 use std::ops::{Add, AddAssign, Div, Mul, Neg, Rem, Sub, SubAssign};
 
 /// Overflowing number. Operations panic on overflow, even in release mode.
@@ -667,9 +666,10 @@ mod overflowing_support {
         match mode {
             #[cfg(not(target_arch = "wasm32"))]
             MODE_SOFT_PANIC => crate::soft_panic_or_log!("Overflow: {description}"),
-            // We cannot use the logging `soft_panic_or_log` in wasm, so we skip logging instead.
+            // We cannot use the logging `soft_panic_or_log` in wasm, so we panic instead (soft
+            // assertions are always enabled in wasm).
             #[cfg(target_arch = "wasm32")]
-            MODE_SOFT_PANIC => crate::soft_panic_no_log!("Overflow: {description}"),
+            MODE_SOFT_PANIC => panic!("Overflow: {description}"),
             MODE_PANIC => panic!("Overflow: {description}"),
             // MODE_IGNORE and all other (impossible) values
             _ => {}