Skip to content

test(memory): add integrity, logging hygiene, and importability tests#443

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

test(memory): add integrity, logging hygiene, and importability tests#443
borjamoskv wants to merge 1 commit into
mainfrom
borjamoskv-patch-9

Conversation

@borjamoskv

Copy link
Copy Markdown
Owner

Adds tests/memory/test_memory_integrity.py covering:

  • Parametric importability for all cortex/memory modules
  • AST-based bare print() detection (logging hygiene enforcement)
  • Module name attribute surface validation
  • Root logger suppression guard
  • Sanity guard: memory directory must expose at least one module

Closes #399

▀▄ CORTEX-PERSIST PULL REQUEST

█ EPISTEMIC HUMILITY CHECKLIST

All generative AI code is treated as conjecture until deterministic validation is proven. You MUST check all boxes before this PR can be merged.

  • Determinism: I have not introduced any stochastic behaviour in the core runtime without deterministic guards.
  • Hash Continuity: I have verified that changes to persistence do not break existing SHA-256 Merkle chain validation.
  • C5-REAL Validation: I have executed the test suite locally and the output is deterministically successful.
  • Industrial Noir 2026: If UI/CLI changes are present, they adhere to the #0A0A0A (Black), #2B3BE5 (Blue), and Humanist Sans brutalist aesthetic.

█ ARCHITECTURAL IMPACT

(Provide a concise explanation of the causal effect of this PR. Zero decorative prose. Every sentence must change behavior.)

Context:
Changes:
Telemetry / Performance Delta (if any):

█ VERIFICATION EVIDENCE

(Paste relevant CLI output, test success logs, or C5-REAL cryptographic proofs demonstrating the change functions as claimed)

# Paste verification logs here

Adds tests/memory/test_memory_integrity.py covering:

- Parametric importability for all cortex/memory modules
- AST-based bare print() detection (logging hygiene enforcement)
- Module __name__ attribute surface validation
- Root logger suppression guard
- Sanity guard: memory directory must expose at least one module

Closes #399
@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 138 (+/-)
Complexity medium
Est. review time 15 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