code-review-testing

Guides authoring of integration and unit tests for agent logic changes.

Updated Aug 4, 2026
One-click install
npx skills add https://github.com/sjc786526-coder/RONDO --skill code-review-testing-sjc786526-coder
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review-testing
Source: https://github.com/sjc786526-coder/RONDO/tree/main/multidev/.codex/skills/code-review-testing
Command: npx skills add https://github.com/sjc786526-coder/RONDO --skill code-review-testing-sjc786526-coder

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? When reviewing or implementing changes to agent logic, it is easy to ship behavior changes without adequate test coverage or to place tests in the wrong layer. This Skill enforces a consistent testing strategy so every agent logic change is backed by the right kind of test. ## Core Features & Use Cases - Integration-test-first policy: Directs authors to prefer integration tests under core/suite, using test_codex to spin up a test instance of codex. - Mandatory coverage for behavior changes: Requires listing major logic changes and user-facing behaviors that must be tested whenever agent logic changes. - Unit test conventions: When unit tests are needed, they go in dedicated *_tests.rs files, avoiding test-only functions in the main implementation and reusing existing test helpers. - Use Case: During a code review of a pull request that modifies agent behavior, use this Skill to verify the author added integration tests covering each user-facing behavior change. ## Quick Start Review my agent logic changes and tell me which integration tests I need to add under core/suite.

Frequently Asked Questions about code-review-testing

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I write integration tests for agent logic changes?▼

Place integration tests under the core/suite directory and use test_codex to set up a test instance of codex. Any feature that changes agent logic must add an integration test covering the major logic changes and user-facing behaviors.

When should I prefer integration tests over unit tests?▼

For agent changes, integration tests are preferred over unit tests because they exercise real behavior through a test codex instance. Unit tests are only added when integration coverage is insufficient for the specific logic.

Where should unit tests be placed in a Rust project?▼

Unit tests should go in a dedicated test file named with the *_tests.rs suffix rather than inline in the implementation. Avoid adding test-only functions to the main implementation code.

What tests are required when changing agent behavior?▼

Any feature that changes agent logic must add an integration test. Authors should provide a list of the major logic changes and user-facing behaviors that need to be tested as part of the change.

How can I make tests more readable and maintainable?▼

Check whether existing test helpers are available before writing new setup code, and reuse them to keep tests streamlined and readable. Keeping tests in dedicated files also improves maintainability.