From ba138712e4c4a067e438a62ad20d54091f5f4446 Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Sat, 18 Mar 2023 00:10:29 +0300 Subject: feat(tvix/eval): add Evaluation::strict to toggle top-level deepseq This makes it possible for callers to control whether they can receive partially evaluated values from an evaluation or not. We're actually flipping the default behaviour to non-strict top-level evaluation, which means that callers have to set `strict = true` on the Evaluation to get the previous behaviour. Change-Id: Ic048e9ba09c88866d4c3177d5fa07db11c4eb20e Reviewed-on: https://cl.tvl.fyi/c/depot/+/8325 Autosubmit: tazjin Tested-by: BuildkiteCI Reviewed-by: sterni --- tvix/eval/src/tests/mod.rs | 2 ++ 1 file changed, 2 insertions(+) (limited to 'tvix/eval/src/tests/mod.rs') diff --git a/tvix/eval/src/tests/mod.rs b/tvix/eval/src/tests/mod.rs index b998600cda43..f800baf05018 100644 --- a/tvix/eval/src/tests/mod.rs +++ b/tvix/eval/src/tests/mod.rs @@ -52,6 +52,7 @@ fn eval_test(code_path: &str, expect_success: bool) { } let mut eval = crate::Evaluation::new_impure(&code, Some(code_path.into())); + eval.strict = true; eval.builtins.extend(mock_builtins::builtins()); let result = eval.evaluate(); @@ -100,6 +101,7 @@ fn identity(code_path: &str) { let code = std::fs::read_to_string(code_path).expect("should be able to read test code"); let mut eval = crate::Evaluation::new(&code, None); + eval.strict = true; eval.io_handle = Box::new(crate::StdIO); let result = eval.evaluate(); -- cgit 1.4.1