GameCult/Idunn
Watch
9
0
Fork
You've already forked Idunn
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Drop the workload-only rows from the abort cleanup test

Browse source 
A transaction holding a workload and no activation is refused by the
fixture before the cleanup rule is reached, so those rows could not
speak for the rule. The method-level owed test keeps the workload-only
case.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This commit is contained in:
Meta Vens 2026-09-30 00:36:31 +02:00
commit e7ee053427
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/control_plane.rs
View file

@ -9844,7 +9844,8 @@ mod tests {
#[test]
fn a_stored_abort_whose_candidate_cleanup_disagrees_with_the_owed_rule_is_refused() -> Result<()> {
let world = ContinuityProjection::after_candidate_observed()?;
// (activation issued, workload observed, recorded cleanup, accepted)
// (activation issued, workload observed, recorded cleanup, accepted).
// Workload-only rows are omitted: the fixture refuses that shape for other reasons.
let rows = [
(false, false, CleanupEvidence::Skipped, true),
(false, false, CleanupEvidence::Pending, false),
@ -9852,8 +9853,6 @@ mod tests {
(true, false, CleanupEvidence::Skipped, false),
(true, false, CleanupEvidence::Pending, true),
(true, false, CleanupEvidence::Complete, true),
(false, true, CleanupEvidence::Skipped, false),
(false, true, CleanupEvidence::Pending, true),
(true, true, CleanupEvidence::Skipped, false),
(true, true, CleanupEvidence::Complete, true),
];