test: add randomized `BatchSwapOutputData` in `SwapClaim` proptests #3490

redshiftzero · 2023-12-07T19:59:08Z

cronokirby · 2023-12-07T21:36:13Z

crates/core/component/dex/src/swap_claim/proof.rs

+        let delta_1 = (4001..2000000000u128).prop_map(Amount::from);
+        let delta_2 = (4001..2000000000u128).prop_map(Amount::from);


Do we need to have such a large range?

That's not that big if you account for the denom scaling factors, typically 10^6, 10^9 or 10^18 (!)

test: add randomized BatchSwapOutputData in SwapClaim proptests

acd9952

redshiftzero temporarily deployed to smoke-test December 7, 2023 19:59 — with GitHub Actions Inactive

cronokirby reviewed Dec 7, 2023

View reviewed changes

redshiftzero merged commit dbe8abf into main Dec 11, 2023
7 checks passed

redshiftzero deleted the test-swap-claim branch December 11, 2023 18:04

Provide feedback

		let delta_1 = (4001..2000000000u128).prop_map(Amount::from);
		let delta_2 = (4001..2000000000u128).prop_map(Amount::from);