Skip to content

tests(memory): comprehensive test suite for ReconsolidationTracker, C…#437

Open
borjamoskv wants to merge 1 commit into
mainfrom
borjamoskv-patch-3
Open

tests(memory): comprehensive test suite for ReconsolidationTracker, C…#437
borjamoskv wants to merge 1 commit into
mainfrom
borjamoskv-patch-3

Conversation

@borjamoskv

Copy link
Copy Markdown
Owner

Summary

Adds tests/memory/test_reconsolidation.py with 20 deterministic tests covering:

  • ReconsolidationOutcome enum values and str-enum contract
  • LabilizationRecord — is_labile, is_expired, age_seconds, zero-window expiry
  • ConfirmationBiasDetector — bias_score (zero / pure-confirm / mixed), is_biased, biased_engrams, report
  • ReconsolidationTracker — on_access, confirm, contradict, sweep (expired → IGNORED), audit_trail ordering, all_audit_events, dream_sweep hook, confirmation_bias_report, multi-engram isolation

All tests use graceful ImportError skips; no flaky timers (only 10ms sleeps for expiry edge-cases).

Closes #399

…onfirmationBiasDetector, LabilizationRecord

This file contains comprehensive tests for the reconsolidation module, covering various components such as ReconsolidationOutcome, LabilizationRecord, ConfirmationBiasDetector, and ReconsolidationTracker. It includes tests for lifecycle methods, bias detection, and concurrent safety.
@github-actions github-actions Bot added the tests label Jun 6, 2026
@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

∞ MÖBIUS — PR Analysis

Metric Value
Files changed 1
Total changes 286 (+/-)
Complexity high
Est. review time 30 min
Has tests?
Has Rust changes? No

Labels applied: tests


Generated by MÖBIUS (Clojure/Babashka) — where code IS data

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(memory): add tests for reconsolidation and manager modules

1 participant