refactor: rename golden tests → fixtures (consistent positive framing) #267

Merged
clawdie merged 1 commit from rename/golden-to-fixtures into main 2026-06-28 10:22:23 +02:00
10 changed files with 11 additions and 11 deletions

View file

@ -75,7 +75,7 @@ static linking on FreeBSD (no `openssl-sys` dependency).
| Crate | Role |
| ----------------------- | ----------------------------------------------------------------------- |
| `colibri-contracts` | Manifest/capability/event schema (golden tests) |
| `colibri-contracts` | Manifest/capability/event schema (fixture tests) |
| `colibri-deepseek` | DeepSeek cache-hit probe, prefix metering |
| `colibri-runtime` | Host status ingestion, runtime inventory |
| `colibri-glasspane` | Agent state machine + event ingestion |
@ -213,7 +213,7 @@ cargo test --workspace
cargo clippy --workspace --all-targets -- -D warnings
```
Golden tests in `crates/colibri-contracts/tests/golden.rs` validate manifest
Fixture tests in `crates/colibri-contracts/tests/fixtures.rs` validate manifest
output against reference fixtures. These fixtures are cross-platform — if a
Linux-produced manifest differs from FreeBSD 15, investigate and document
the cross-platform difference before proceeding.

View file

@ -21,7 +21,7 @@ Optional Headroom compression sidecar: `docs/wiki/headroom-sidecar.md`.
| ----------------------- | ----------------------------------------------------------------------- |
| `colibri` (root) | Workspace root + probe binaries (colibri-probe, runtime-inventory) |
| `colibri-mcp` | MCP bridge for editor integration (Zed, Claude Code) via stdio JSON-RPC |
| `colibri-contracts` | JSON schema contracts (golden tests) |
| `colibri-contracts` | JSON schema contracts (fixture tests) |
| `colibri-deepseek` | DeepSeek cache-hit probe, prefix metering |
| `colibri-runtime` | Host status ingestion, runtime inventory |
| `colibri-glasspane` | Agent 5-state machine (zot/pi JSONL events → state) |

View file

@ -1,4 +1,4 @@
//! Golden tests: the contract structs must accept the real committed manifests
//! Fixture tests: the contract structs must accept the real committed manifests
//! (produced by different agents/hosts) and round-trip without data loss.
use colibri_contracts::*;

View file

@ -609,7 +609,7 @@ impl Session {
}
// ---------------------------------------------------------------------------
// T1.4 golden tests — PromptAssembly + CacheMetrics
// T1.4 fixture tests — PromptAssembly + CacheMetrics
// ---------------------------------------------------------------------------
#[cfg(test)]

View file

@ -36,7 +36,7 @@ colibri-daemon — Unix-socket server (the always-on supervisor)
| `colibri-skills` | Read-only skills catalog |
| `colibri-mcp` | MCP bridge — editor integration + external MCP host |
| `colibri-deepseek` | Cache-hit probe + prefix metering for DeepSeek |
| `colibri-contracts` | Manifest/capability/event schemas (golden tests) |
| `colibri-contracts` | Manifest/capability/event schemas (fixture tests) |
| `colibri-runtime` | Host status ingestion, runtime inventory |
| `clawdie` | Host installer/deployer (ZFS layout, rc.d service) |

View file

@ -36,7 +36,7 @@ Schema constants and structs live in `crates/colibri-contracts/src/lib.rs`.
## Golden tests
`crates/colibri-contracts/tests/golden.rs` parses every committed manifest in
`crates/colibri-contracts/tests/fixtures.rs` parses every committed manifest in
`manifests/` and asserts round-trip equality. The fixtures are intended to be
**cross-platform** — if a manifest produced on Linux differs from one produced
on FreeBSD 15, the difference must be understood and documented before it is

View file

@ -63,7 +63,7 @@ warning.
| [layered-soul](./layered-soul.md) | How Colibri consumes the layered-soul reviewed-context repo today vs planned |
| [task-board](./task-board.md) | Capability match scoring, cron scheduling, intake drain, SQLite backing |
| [quality-gates](./quality-gates.md) | `ci-checks.sh` as the pre-merge gate; why drift reached `main` before |
| [contracts](./contracts.md) | Stable JSON schemas (run-manifest, runtime-inventory, provider-test), golden tests |
| [contracts](./contracts.md) | Stable JSON schemas (run-manifest, runtime-inventory, provider-test), fixture tests |
| [store-schema](./store-schema.md) | SQLite coordination schema and migration discipline |
| [external-mcp](./external-mcp.md) | MCP bridge for editors + external stdio MCP host; read/write/external-call gates |
| [operator-cli](./operator-cli.md) | The `colibri` CLI as a thin typed Unix-socket client over the daemon API |

View file

@ -77,7 +77,7 @@ socket location without recompiling:
## Golden fixtures
`crates/colibri-contracts/tests/golden.rs` parses committed inventory and
`crates/colibri-contracts/tests/fixtures.rs` parses committed inventory and
host-status manifests in `manifests/` and round-trips them through the Rust
structs. Those fixtures come from real hosts (`osa`, `domedog`, `debby`, the
operator USB) and are treated as cross-platform source material.

View file

@ -39,7 +39,7 @@ Konstante shem in strukture živijo v `crates/colibri-contracts/src/lib.rs`.
## Zlati testi
`crates/colibri-contracts/tests/golden.rs` razčleni vsak potrjen manifest v
`crates/colibri-contracts/tests/fixtures.rs` razčleni vsak potrjen manifest v
`manifests/` in preveri enakost povratnega zapisa. Primerki so namenjeni
**medplatformni** rabi — če se manifest, ustvarjen na Linuxu, razlikuje od
tistega na FreeBSD 15, je treba razliko razumeti in dokumentirati, preden se

View file

@ -73,7 +73,7 @@ cargo clippy --workspace --all-targets -- -D warnings # ✅ zero warnings
cargo test --workspace # ✅ 228 passed, 0 failed
```
228 tests cover all 13 crates including unit, integration, golden, and
228 tests cover all 13 crates including unit, integration, fixture, and
live-socket tests. This confirms the `0.11.0` tree compiles and passes on
Linux. FreeBSD runtime validation (`cargo test` on FreeBSD 15) is the
remaining step before the first poudriere build.