From f600aa5322f6628e1af63e9dd4c6ad073020e152 Mon Sep 17 00:00:00 2001 From: Griffin Smith Date: Tue, 20 Sep 2022 17:21:58 -0400 Subject: test(tvix/eval): Make proptests a bit smaller Generate smaller recursive values for generated Values, and run fewer cases for the attrs proptests which are particularly egregious. Change-Id: Ia35c7c120270feaf045be1deb440c87ebb185c27 Reviewed-on: https://cl.tvl.fyi/c/depot/+/6716 Autosubmit: grfn Reviewed-by: tazjin Reviewed-by: sterni Tested-by: BuildkiteCI --- tvix/eval/src/value/arbitrary.rs | 2 +- tvix/eval/src/value/attrs/tests.rs | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'tvix') diff --git a/tvix/eval/src/value/arbitrary.rs b/tvix/eval/src/value/arbitrary.rs index 56ccd3b28a..20be3732e0 100644 --- a/tvix/eval/src/value/arbitrary.rs +++ b/tvix/eval/src/value/arbitrary.rs @@ -63,7 +63,7 @@ fn leaf_value() -> impl Strategy { } fn non_internal_value() -> impl Strategy { - leaf_value().prop_recursive(10, 256, 10, |inner| { + leaf_value().prop_recursive(3, 5, 5, |inner| { prop_oneof![ any_with::(( Default::default(), diff --git a/tvix/eval/src/value/attrs/tests.rs b/tvix/eval/src/value/attrs/tests.rs index 539a07e00f..4556d26dd0 100644 --- a/tvix/eval/src/value/attrs/tests.rs +++ b/tvix/eval/src/value/attrs/tests.rs @@ -7,7 +7,7 @@ mod nix_eq { use proptest::prelude::ProptestConfig; use test_strategy::proptest; - #[proptest(ProptestConfig { cases: 5, ..Default::default() })] + #[proptest(ProptestConfig { cases: 2, ..Default::default() })] fn reflexive(x: NixAttrs) { let mut observer = NoOpObserver {}; let mut vm = VM::new(&mut observer); @@ -15,7 +15,7 @@ mod nix_eq { assert!(x.nix_eq(&x, &mut vm).unwrap()) } - #[proptest(ProptestConfig { cases: 5, ..Default::default() })] + #[proptest(ProptestConfig { cases: 2, ..Default::default() })] fn symmetric(x: NixAttrs, y: NixAttrs) { let mut observer = NoOpObserver {}; let mut vm = VM::new(&mut observer); @@ -26,7 +26,7 @@ mod nix_eq { ) } - #[proptest(ProptestConfig { cases: 5, ..Default::default() })] + #[proptest(ProptestConfig { cases: 2, ..Default::default() })] fn transitive(x: NixAttrs, y: NixAttrs, z: NixAttrs) { let mut observer = NoOpObserver {}; let mut vm = VM::new(&mut observer); -- cgit 1.4.1