[OMEGA-239] Register the new memory safety tests in run_mandatory#258
Merged
TossSky merged 1 commit intoJul 16, 2026
Conversation
The two tests added for safe history and prompt file reading are not listed in Autotests/run_mandatory, so CI never runs them and a regression in the feature would go unnoticed. Both are plain mock tests using the llm/comm fixtures and the same -t test container run_mandatory already brings up, so they need nothing extra. run_mandatory goes from 49 to 52 passed.
TossSky
merged commit Jul 16, 2026
29700e3
into
asi-alliance:OMEGA-239-safe-file-processing
1 check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The two tests this branch adds are not listed in
Autotests/run_mandatory, so CInever collects them. The feature ships with tests that never run, and a regression
in the safe history/prompt reading would go unnoticed.
Both are plain
mock/tests using thellm/commfixtures and the same-t testcontainer
run_mandatoryalready brings up, so nothing extra is needed to run themin the existing CI job.
This registers them, in alphabetical order:
Verified on this branch head (e35f694):
"Agent crashed or failed to respond due to missing prompt"), with it they pass.