Skip to content

test(autoresearch): add runner, routes, and ChromaDB indexing tests (#2637)#3075

Merged
mrveiss merged 1 commit intoDev_new_guifrom
feat/2637-autoresearch-tests
Mar 31, 2026
Merged

test(autoresearch): add runner, routes, and ChromaDB indexing tests (#2637)#3075
mrveiss merged 1 commit intoDev_new_guifrom
feat/2637-autoresearch-tests

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Mar 31, 2026

Summary

  • 75 ExperimentRunner tests: validation, subprocess mocking, evaluation logic, cancel, concurrency lock
  • 20 route integration tests: CRUD, 409 conflict, pagination, validation errors
  • 22 ChromaDB indexing tests: document build, metadata, state transitions, error handling
  • Total: ~117 new test cases for the three untested AutoResearch components

Closes #2637

Test plan

  • Runner: validate_extra_params (14), build_command (4), execute (5), run (7), evaluate (7), cancel (3)
  • Routes: create (6), list (7), get (2), stats (2), baseline (3), status (2), cancel (2)
  • ChromaDB: build_document (8), build_metadata (8), index (5), save_experiment (7)
  • flake8 passes

🤖 Generated with Claude Code

…2637)

75 ExperimentRunner tests (validation, subprocess, evaluation, cancel,
concurrency). 20 route integration tests (CRUD, conflict, pagination).
22 ChromaDB indexing tests (document build, metadata, state transitions).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mrveiss mrveiss merged commit 3d9ab94 into Dev_new_gui Mar 31, 2026
2 of 4 checks passed
@mrveiss mrveiss deleted the feat/2637-autoresearch-tests branch March 31, 2026 17:58
@github-actions
Copy link
Copy Markdown

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant