Skip to content

Commit bde23ab

Browse files
hyperpolymathclaude
andcommitted
test(fixtures): add type_info: None to Hypothesis/Definition/Variable literals
echidna::core::{Hypothesis, Definition, Variable} gained an `Option<TypeInfo>` `type_info` field (likely part of the type-system decoration layer in feat 8c46e2b / 53af277 etc.) but the following fixture call sites were not updated, breaking cargo check --all-targets: - tests/common/generators.rs:71 (Hypothesis) - tests/common/generators.rs:94 (Definition) - tests/common/generators.rs:113 (Variable) - tests/common/mod.rs:44 (Hypothesis) - benches/proof_benchmarks.rs:39 (Hypothesis) - tests/sanity_suite.rs:53 (Variable — make_proof_state Bool) - tests/sanity_suite.rs:76 (Variable — make_proof_state_int Int) - tests/sanity_suite.rs:510 (Variable — adversarial contradiction test) All 8 sites get `type_info: None,` — safe default for property-based / benchmark / sanity fixtures that don't need type-info decoration. After fix: cargo test --test neural_property_tests --test aspect_tests --test e2e_prover_test --test security_aspect_tests --test sanity_suite passes (32 test units across the 5 binaries). Known remaining cargo check --all-targets breakage, NOT fixed here: examples/aspect_tagging_demo.rs:7 — unresolved import of `echidna::aspect::NeuralTagger`. Separate defect, tracked independently. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 99381fe commit bde23ab

4 files changed

Lines changed: 12 additions & 1 deletion

File tree

benches/proof_benchmarks.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ fn bench_proof_state_construction(c: &mut Criterion) {
4040
name: format!("h_{i}"),
4141
ty: Term::Const("Prop".to_string()),
4242
body: None,
43+
type_info: None,
4344
}],
4445
target: Term::App {
4546
func: Box::new(Term::Const("eq".to_string())),

tests/common/generators.rs

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ pub fn arb_hypothesis() -> impl Strategy<Value = Hypothesis> {
7272
name,
7373
ty,
7474
body,
75+
type_info: None,
7576
})
7677
}
7778

@@ -95,6 +96,7 @@ pub fn arb_definition() -> impl Strategy<Value = Definition> {
9596
name,
9697
ty,
9798
body,
99+
type_info: None,
98100
})
99101
}
100102

@@ -110,7 +112,11 @@ pub fn arb_theorem() -> impl Strategy<Value = Theorem> {
110112

111113
/// Strategy for generating a variable
112114
pub fn arb_variable() -> impl Strategy<Value = Variable> {
113-
(var_name(), arb_term()).prop_map(|(name, ty)| Variable { name, ty })
115+
(var_name(), arb_term()).prop_map(|(name, ty)| Variable {
116+
name,
117+
ty,
118+
type_info: None,
119+
})
114120
}
115121

116122
/// Strategy for generating contexts

tests/common/mod.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ pub fn multi_goal_proof_state() -> ProofState {
4545
name: "h".to_string(),
4646
ty: Term::Const("P".to_string()),
4747
body: None,
48+
type_info: None,
4849
}],
4950
target: Term::Const("Q".to_string()),
5051
},

tests/sanity_suite.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ fn make_proof_state(bool_vars: &[&str], axioms: &[&str], goal_term: &str) -> Pro
5353
ctx.variables.push(Variable {
5454
name: v.to_string(),
5555
ty: Term::Const("Bool".to_string()),
56+
type_info: None,
5657
});
5758
}
5859
for ax in axioms {
@@ -76,6 +77,7 @@ fn make_proof_state_int(int_vars: &[&str], axioms: &[&str], goal_term: &str) ->
7677
ctx.variables.push(Variable {
7778
name: v.to_string(),
7879
ty: Term::Const("Int".to_string()),
80+
type_info: None,
7981
});
8082
}
8183
for ax in axioms {
@@ -510,6 +512,7 @@ async fn sanity_goal_set_inconsistency() {
510512
ctx.variables.push(echidna::core::Variable {
511513
name: "P".to_string(),
512514
ty: echidna::core::Term::Const("Bool".to_string()),
515+
type_info: None,
513516
});
514517
let state = ProofState {
515518
goals: vec![

0 commit comments

Comments
 (0)