test: offline unit tests for config, data files and LLM reply parsing #3

Closed

forge-recovery wants to merge 1 commits from ci/offline-unit-tests into develop

pull from: ci/offline-unit-tests

merge into: KofTwentyTwo:develop

KofTwentyTwo:develop

KofTwentyTwo:feature/security-findings-2026-10

Conversation 1 Commits 1 Files Changed 2

+119

forge-recovery commented 2026-11-01 21:05:14 +00:00

Owner

Copy link

Copy source

Adds 14 offline unittest tests under tests/ so the central Python CI profile has real tests to execute. They cover config loading (including the example-config fallback), analysis/collected-data persistence, the LLM JSON reply parser and provider selection. No network, cloud account or LLM credentials are used.

Verified locally with the central python-verify.sh on Python 3.13 as UID 1000 (dependencies=resolve, test-path=tests): ruff clean, 14 tests executed.

Old CI semantics were flake8 plus an import/YAML smoke on Python 3.8 with unpinned requirements and no tests; this PR does not pin requirements. Central policy will set dependencies=resolve and test-path=tests.

🤖 Generated with Claude Code

Adds 14 offline unittest tests under `tests/` so the central Python CI profile has real tests to execute. They cover config loading (including the example-config fallback), analysis/collected-data persistence, the LLM JSON reply parser and provider selection. No network, cloud account or LLM credentials are used. Verified locally with the central `python-verify.sh` on Python 3.13 as UID 1000 (`dependencies=resolve`, `test-path=tests`): ruff clean, 14 tests executed. Old CI semantics were flake8 plus an import/YAML smoke on Python 3.8 with unpinned requirements and no tests; this PR does not pin requirements. Central policy will set `dependencies=resolve` and `test-path=tests`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

forge-recovery added 1 commit 2026-11-01 21:05:14 +00:00

test: offline unit tests for config, data files and LLM reply parsing ... 86ec451f74

Covers load_config, save/load of analysis and collected data, the LLM JSON
reply parser and provider selection. No network, cloud or LLM credentials.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

forge-recovery commented 2026-11-01 21:40:01 +00:00

Author

Owner

Copy link

Copy source

Parked by scope decision: these unit tests go beyond the legacy CI smoke. Closed unmerged; branch ci/offline-unit-tests is preserved.

Parked by scope decision: these unit tests go beyond the legacy CI smoke. Closed unmerged; branch ci/offline-unit-tests is preserved.

forge-recovery closed this pull request 2026-11-01 21:40:01 +00:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

forge-recovery (Forge Recovery) james.maes (James Maes)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KofTwentyTwo/Aestimare#3