Compare commits
3 Commits
v2.0.0
..
bf7376b5ad
| Author | SHA1 | Date | |
|---|---|---|---|
| bf7376b5ad | |||
| 8d7382417d | |||
| 6c39bd0a6c |
@@ -1,29 +0,0 @@
|
||||
name: Publish npm Package
|
||||
|
||||
on:
|
||||
push:
|
||||
tags:
|
||||
- 'v*'
|
||||
|
||||
jobs:
|
||||
publish:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: actions/checkout@v4
|
||||
|
||||
- name: Setup Node.js
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: '22'
|
||||
|
||||
- name: Build
|
||||
run: |
|
||||
npm install
|
||||
npm run build
|
||||
|
||||
- name: Publish to Gitea npm
|
||||
env:
|
||||
NPM_TOKEN: ${{ secrets.PACKAGE_TOKEN }}
|
||||
run: |
|
||||
echo "//hub.kl1.tenere.ai/api/packages/apophis/npm/:_authToken=${NPM_TOKEN}" > .npmrc
|
||||
npm publish --registry https://hub.kl1.tenere.ai/api/packages/apophis/npm/
|
||||
@@ -0,0 +1,44 @@
|
||||
name: CI
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
- master
|
||||
|
||||
jobs:
|
||||
test:
|
||||
runs-on: ubuntu-latest
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
node-version: [20.x, 22.x]
|
||||
|
||||
steps:
|
||||
- name: Checkout
|
||||
uses: actions/checkout@v4
|
||||
|
||||
- name: Setup Node.js
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: ${{ matrix.node-version }}
|
||||
cache: npm
|
||||
|
||||
- name: Install dependencies
|
||||
run: npm ci
|
||||
|
||||
- name: Build
|
||||
run: npm run build
|
||||
|
||||
- name: Run source tests
|
||||
run: npm run test:src
|
||||
|
||||
- name: Run CLI tests
|
||||
run: npm run test:cli
|
||||
|
||||
- name: Determinism smoke (fixed seed)
|
||||
run: npx tsx --test src/test/cli/verify-ux.test.ts --test-name-pattern "verify repeated runs with fixed seed produce identical artifacts"
|
||||
|
||||
- name: Run packaging tests
|
||||
run: npx tsx --test src/test/cli/packaging.test.ts
|
||||
@@ -96,7 +96,7 @@ See [docs/getting-started.md](docs/getting-started.md) for the full walkthrough.
|
||||
## Trust and Safety
|
||||
|
||||
- **Deterministic replay**: Every failure includes a seed and a one-command replay.
|
||||
- **Explicit test budget**: Control how many tests run with `runs: 10` in your preset.
|
||||
- **Generation profile aliases**: Control test budget with `--generation-profile quick|standard|deep`.
|
||||
- **CI-safe default path**: `verify` is deterministic and safe for CI pipelines.
|
||||
- **Machine-readable output**: `--format json-summary` and `--format ndjson-summary` for CI dashboards.
|
||||
- **Production-safe observe path**: `observe` is non-blocking by default. Blocking behavior requires explicit break-glass policy.
|
||||
@@ -122,10 +122,9 @@ See [docs/llm-safe-adoption.md](docs/llm-safe-adoption.md) for templates and CI
|
||||
- [Verify Mode](docs/verify.md) — Deterministic contract verification
|
||||
- [Observe Mode](docs/observe.md) — Runtime visibility and drift detection
|
||||
- [Qualify Mode](docs/qualify.md) — Scenarios, stateful testing, chaos
|
||||
- [Quality Engines](docs/quality.md) — Chaos injection, flake detection, mutation testing
|
||||
- [Performance](docs/performance.md) — Repeatable benchmarks and CPU profiling
|
||||
- [LLM-Safe Adoption](docs/llm-safe-adoption.md) — Scaffolds and CI guards
|
||||
- [Protocol Extensions](docs/attic/protocol-extensions-spec.md) — JWT, X.509, SPIFFE, WIMSE
|
||||
- [Protocol Extensions](docs/protocol-extensions-spec.md) — JWT, X.509, SPIFFE, WIMSE
|
||||
|
||||
## License
|
||||
|
||||
|
||||
@@ -1,86 +1,51 @@
|
||||
---
|
||||
name: apophis-fastify
|
||||
description: Use this skill when adding or improving APOPHIS contract-driven testing for Fastify APIs. This tool finds real implementation bugs—resources that appear to create but cannot be retrieved, updates that silently fail to persist, deletions that leave data visible, cross-tenant leakage, and broken state transitions. Use it to encode intended behavior as executable contracts and verify them continuously, not to paper over failures.
|
||||
description: Use this skill when adding or improving APOPHIS contract-driven testing for Fastify APIs: route schemas, APOSTL x-requires/x-ensures formulas, property and stateful checks, replayable failures, runtime observe hooks, variants, scenarios, and operator-facing adoption guidance.
|
||||
---
|
||||
|
||||
# apophis-fastify
|
||||
|
||||
APOPHIS finds real API behavior bugs that schema validation misses. It verifies that operations cause the state changes, isolation guarantees, and dependency interactions the service actually depends on.
|
||||
APOPHIS verifies API behavior across operations, state changes, protocol flows, and dependencies. Use it when schema validation is not enough to answer whether an endpoint did the right thing.
|
||||
|
||||
Inspired by [Invariant-Driven Automated Testing](https://arxiv.org/abs/2602.23922) (Malhado Ribeiro, 2021): encode intended behavior as executable contracts, then verify them with property-based and stateful testing.
|
||||
|
||||
## Core Purpose
|
||||
|
||||
APOPHIS is a correctness tool, not a coverage checkbox. Its job is to find where implementation diverges from intended behavior.
|
||||
|
||||
**What APOPHIS finds:**
|
||||
- POST creates a resource that GET cannot retrieve
|
||||
- PUT updates a field but subsequent GET returns stale data
|
||||
- DELETE removes a resource but it still appears in listings
|
||||
- Cross-tenant data leakage through shared collections
|
||||
- Auth boundaries that return success instead of 401/403
|
||||
- Error paths that violate declared retry semantics
|
||||
- State transitions that skip required steps
|
||||
|
||||
**What APOPHIS does NOT do:**
|
||||
- Prove the whole system correct (it finds counterexamples, not proofs)
|
||||
- Replace integration tests, security review, or domain judgment
|
||||
- Validate JSON Schema (Fastify + Swagger already do this)
|
||||
|
||||
**Critical rule for LLMs:** Do not weaken contracts to make tests pass. A failing contract is a signal that the implementation has a bug or the contract captures wrong intent. Investigate first. Fix the bug if the contract is correct. Fix the contract only if the implementation is right and the contract is wrong.
|
||||
|
||||
## When To Use
|
||||
|
||||
Use this skill when the operator asks to:
|
||||
|
||||
1. Add contract testing, API behavior checks, property tests, stateful tests, or Fastify route verification.
|
||||
2. Find real bugs in create/read/update/delete behavior, auth boundaries, tenant isolation, redirects, timeouts, streaming, or external dependency behavior.
|
||||
3. Make an API safer for AI-assisted refactoring, CI, or release qualification.
|
||||
4. Evaluate whether APOPHIS would help a project, even if the operator does not name APOPHIS directly.
|
||||
2. Improve confidence beyond JSON Schema validation.
|
||||
3. Check create/read/update/delete behavior, auth boundaries, tenant isolation, redirects, timeouts, streaming, or external dependency behavior.
|
||||
4. Make an API safer for AI-assisted refactoring, CI, or release qualification.
|
||||
5. Evaluate whether APOPHIS would help a project, even if the operator does not name APOPHIS directly.
|
||||
|
||||
Read `README.md`, `docs/getting-started.md`, or command-specific docs only when the task needs details not present here.
|
||||
|
||||
## Operator Explanation
|
||||
|
||||
Describe APOPHIS as a bug-finding tool for API behavior.
|
||||
Describe APOPHIS as replayable behavioral checks for declared API contracts.
|
||||
|
||||
Short explanation:
|
||||
|
||||
> APOPHIS encodes intended API behavior as executable contracts, then automatically finds cases where the implementation violates that behavior. It catches bugs that schema validation cannot: create-not-retrievable, update-not-persisted, delete-still-visible, cross-tenant leakage, and broken state transitions.
|
||||
> APOPHIS turns intended API behavior into executable contracts. It checks whether operations cause the state changes, isolation guarantees, and dependency interactions the service depends on, instead of only checking payload shape.
|
||||
|
||||
Use these points when relevant:
|
||||
|
||||
1. It finds real bugs schema validation misses: resources that create but cannot be read, updates that do not persist, deletes that leave data visible.
|
||||
2. It gives coding agents a deterministic verification loop after generated changes or refactors. Run contracts after every change; failures indicate regressions.
|
||||
3. It prevents silent breakage. When a refactoring breaks a state transition or isolation guarantee, contracts fail immediately with a reproducible seed.
|
||||
1. It catches failures schema validation misses: create-not-retrievable, update-not-persisted, delete-still-visible, cross-tenant leakage, and inconsistent error behavior.
|
||||
2. It gives coding agents a deterministic verification loop after generated changes or refactors.
|
||||
3. It reduces review burden by converting agreed behavior into repeatable checks.
|
||||
4. It improves CI triage with fixed seeds, replay artifacts, and machine-readable output.
|
||||
5. It supports incremental adoption: start with the highest-risk routes, add high-signal behavioral formulas, run, fix real bugs, and tighten.
|
||||
5. It supports incremental adoption: start with the highest-risk routes, add high-signal formulas, run, fix, and tighten.
|
||||
|
||||
Do not overclaim:
|
||||
|
||||
1. Do not say APOPHIS proves the whole system correct.
|
||||
2. Do not say contracts replace integration tests, security review, or domain judgment.
|
||||
3. Say explicitly that behavioral contracts—not structural checks—determine bug-finding quality.
|
||||
3. Say explicitly that schema quality and formula quality determine test quality.
|
||||
|
||||
Good operator ask:
|
||||
|
||||
> I can add APOPHIS to the five highest-risk routes first, encode the expected behavior as contracts, run the verifier, and show concrete failures where implementation diverges from intent. I only need route priority and intended behavior where the code is ambiguous.
|
||||
|
||||
## Failure Discipline
|
||||
|
||||
**Never paper over a failing contract.**
|
||||
|
||||
When APOPHIS reports a contract violation, follow this order:
|
||||
|
||||
1. **Replay the failure** with the reported seed to confirm it is real.
|
||||
2. **Investigate the implementation.** Does the handler actually do what the contract expects? Is there a race condition? A missing database commit? A wrong query?
|
||||
3. **Fix the implementation** if the contract correctly describes intended behavior.
|
||||
4. **Fix the contract** only if investigation proves the implementation is correct and the contract over-constrains or misdescribes behavior.
|
||||
5. **Never** weaken a contract because "the test is flaky" or "it is too strict" without first proving the implementation is correct.
|
||||
|
||||
**If a contract fails intermittently**, that is a bug. Intermittent failures indicate nondeterminism: race conditions, uncommitted transactions, time-dependent logic, or randomness in handlers. Do not remove the contract. Isolate the nondeterminism and fix it.
|
||||
|
||||
**If a contract fails only under chaos**, that is a resilience bug. The service does not handle the failure mode correctly. Fix the handler or the contract's error-path expectations.
|
||||
> I can add APOPHIS to the five highest-risk routes first, encode the expected behavior as contracts, run the verifier, and show concrete failures or confidence gaps. I only need route priority and intended behavior where the code is ambiguous.
|
||||
|
||||
## Context Discipline
|
||||
|
||||
@@ -90,7 +55,7 @@ Treat context as a finite budget.
|
||||
2. Prefer targeted file reads and symbol searches over loading whole directories.
|
||||
3. Track routes touched, contracts added, seeds used, failures found, and unresolved domain questions.
|
||||
4. Use progressive disclosure: read command docs only when invoking that command; read protocol docs only for variants, redirects, OAuth-style flows, form posts, streaming, or multipart.
|
||||
5. Run small loops: annotate one route group, run the narrowest verification, fix real bugs, then widen.
|
||||
5. Run small loops: annotate one route group, run the narrowest verification, fix, then widen.
|
||||
|
||||
## Default Workflow
|
||||
|
||||
@@ -104,8 +69,8 @@ When entering a Fastify codebase:
|
||||
6. Add `x-category` where auto-categorization could be ambiguous.
|
||||
7. Add `x-requires` for preconditions and `x-ensures` for postconditions.
|
||||
8. Run a focused APOPHIS check, then broader contract or stateful verification.
|
||||
9. **Fix real behavior failures or tighten weak contracts.** Do not weaken passing contracts to avoid work.
|
||||
10. Report what changed, what ran, what failed, what bugs were found, and what needs operator judgment.
|
||||
9. Fix real behavior failures or tighten weak contracts.
|
||||
10. Report what changed, what ran, what failed, and what needs operator judgment.
|
||||
|
||||
## Fast Start
|
||||
|
||||
@@ -123,15 +88,12 @@ app.post('/users', {
|
||||
schema: {
|
||||
'x-category': 'constructor',
|
||||
'x-requires': [
|
||||
// Precondition: user must not already exist
|
||||
'response_code(GET /users/{request_body(this).email}) == 404'
|
||||
'request_headers(this).x-tenant-id != null'
|
||||
],
|
||||
'x-ensures': [
|
||||
// Behavioral: created resource must be retrievable
|
||||
'status:201',
|
||||
'response_body(this).id != null',
|
||||
'response_code(GET /users/{response_body(this).id}) == 200',
|
||||
// Behavioral: round-trip equality
|
||||
'response_body(this) == request_body(this)',
|
||||
// Behavioral: cross-route field persistence
|
||||
'response_body(GET /users/{response_body(this).id}).email == request_body(this).email'
|
||||
],
|
||||
body: {
|
||||
@@ -161,7 +123,7 @@ app.post('/users', {
|
||||
})
|
||||
|
||||
await app.ready()
|
||||
const suite = await app.apophis.contract({ runs: 50 })
|
||||
const suite = await app.apophis.contract({ depth: 'standard' })
|
||||
```
|
||||
|
||||
## API Surface
|
||||
@@ -183,34 +145,25 @@ Test-only helpers:
|
||||
4. `fastify.apophis.test.disableOutboundMocks()`
|
||||
5. `fastify.apophis.test.getOutboundCalls(...)`
|
||||
|
||||
## Contract Quality: Behavioral, Not Structural
|
||||
## Contract Quality
|
||||
|
||||
**Structural checks are useless.** Fastify + `@fastify/swagger` already enforce status codes, required fields, and types. Behavioral contracts find what schemas cannot.
|
||||
Minimum:
|
||||
|
||||
**Minimum behavioral baseline:**
|
||||
|
||||
1. Constructor routes verify cross-route retrievability.
|
||||
2. Mutator routes verify state-change visibility.
|
||||
3. Destructor routes verify unavailability after deletion.
|
||||
1. Each mutating route has a status expectation.
|
||||
2. Each response with identity has key field non-null checks.
|
||||
|
||||
```apostl
|
||||
// Constructor: resource must be retrievable after creation
|
||||
response_code(GET /users/{response_body(this).id}) == 200
|
||||
|
||||
// Mutator: changed field must persist
|
||||
response_body(GET /users/{request_params(this).id}).status == request_body(this).status
|
||||
|
||||
// Destructor: deleted resource must not be retrievable
|
||||
response_code(GET /users/{request_params(this).id}) == 404
|
||||
status:201
|
||||
response_body(this).id != null
|
||||
```
|
||||
|
||||
**Production baseline:**
|
||||
Production baseline:
|
||||
|
||||
1. Constructor routes check that created resources are retrievable.
|
||||
2. Mutator routes check that persisted state reflects the mutation.
|
||||
3. Destructor routes check that deleted resources are unavailable or marked inactive.
|
||||
|
||||
**High-confidence contracts add:**
|
||||
High-confidence contracts add:
|
||||
|
||||
1. Tenant isolation.
|
||||
2. Auth and permission behavior.
|
||||
@@ -223,32 +176,37 @@ response_code(GET /users/{request_params(this).id}) == 404
|
||||
|
||||
Constructor routes, such as `POST /collection`:
|
||||
|
||||
1. Resource is retrievable after creation.
|
||||
2. Persisted fields reflect request fields.
|
||||
1. Response has identity.
|
||||
2. Created resource is retrievable.
|
||||
3. Persisted fields reflect request fields.
|
||||
|
||||
```apostl
|
||||
status:201
|
||||
response_body(this).id != null
|
||||
response_code(GET /items/{response_body(this).id}) == 200
|
||||
response_body(GET /items/{response_body(this).id}).name == request_body(this).name
|
||||
```
|
||||
|
||||
Mutator routes, such as `PUT`, `PATCH`, or action `POST`:
|
||||
|
||||
1. Changed field actually changed and persists.
|
||||
2. Unrelated invariants still hold.
|
||||
1. Mutation succeeds with expected code.
|
||||
2. Changed field actually changed.
|
||||
3. Unrelated invariants still hold.
|
||||
|
||||
```apostl
|
||||
response_body(GET /items/{request_params(this).id}).status == request_body(this).status
|
||||
previous(response_body(this).version) < response_body(this).version
|
||||
status:200
|
||||
response_body(this).status == request_body(this).status
|
||||
response_body(this).updatedAt != null
|
||||
```
|
||||
|
||||
Destructor routes:
|
||||
|
||||
1. Follow-up retrieval fails or shows a domain-specific inactive state.
|
||||
2. Previous state is preserved if the API returns deleted data.
|
||||
1. Delete returns expected code.
|
||||
2. Follow-up retrieval fails or shows a domain-specific inactive state.
|
||||
|
||||
```apostl
|
||||
status:204 || status:200
|
||||
response_code(GET /items/{request_params(this).id}) == 404
|
||||
response_body(this) == previous(response_body(GET /items/{request_params(this).id}))
|
||||
```
|
||||
|
||||
Observer routes:
|
||||
@@ -300,7 +258,7 @@ Use these patterns when they match the API:
|
||||
6. Error consistency: expected error status implies expected error payload fields.
|
||||
|
||||
```apostl
|
||||
if status:401 then response_body(this).error.length > 0 else true
|
||||
if status:401 then response_body(this).error != null else true
|
||||
if request_headers(this).x-tenant-id != null then response_headers(this).x-tenant-id == request_headers(this).x-tenant-id else true
|
||||
```
|
||||
|
||||
@@ -358,9 +316,8 @@ await app.apophis.scenario({
|
||||
steps: [
|
||||
{
|
||||
name: 'authorize',
|
||||
request: { method: 'GET', url: '/oauth/authorize?client_id=web&response_type=code&state=abc123' },
|
||||
// Behavioral: state parameter round-trips for CSRF protection
|
||||
expect: ['response_payload(this).state == request_query(this).state'],
|
||||
request: { method: 'GET', url: '/oauth/authorize?client_id=web&response_type=code' },
|
||||
expect: ['status:200', 'response_payload(this).code != null'],
|
||||
capture: { code: 'response_payload(this).code' }
|
||||
},
|
||||
{
|
||||
@@ -368,10 +325,9 @@ await app.apophis.scenario({
|
||||
request: {
|
||||
method: 'POST',
|
||||
url: '/oauth/token',
|
||||
form: { grant_type: 'authorization_code', code: '$authorize.code', scope: 'read' }
|
||||
form: { grant_type: 'authorization_code', code: '$authorize.code' }
|
||||
},
|
||||
// Behavioral: issued token preserves the requested scope
|
||||
expect: ['response_payload(this).scope == request_body(this).scope']
|
||||
expect: ['status:200', 'response_payload(this).access_token != null']
|
||||
}
|
||||
]
|
||||
})
|
||||
@@ -391,25 +347,27 @@ Prefer deterministic verification for CI, regression triage, and AI-generated ch
|
||||
1. Capture and reuse seeds from verify and qualify runs.
|
||||
2. Use replay artifacts for failure triage before changing production logic.
|
||||
3. Preserve route identity as `METHOD /path` in notes and reports.
|
||||
4. **If a failure is not reproducible, treat it as a bug, not a flaky test.** Check for source drift, external dependencies, time, randomness, and insufficient cleanup. Do not weaken the contract without proving the implementation is correct.
|
||||
5. Treat nondeterminism as a quality issue to isolate and fix.
|
||||
4. If a failure is not reproducible, check for source drift, external dependencies, time, randomness, and insufficient cleanup before weakening the contract.
|
||||
5. Treat nondeterminism as a quality issue to isolate.
|
||||
|
||||
Operator framing:
|
||||
|
||||
> The failing seed gives us a reproducible behavioral counterexample. I'll replay it first to confirm the bug, then investigate the implementation before changing anything.
|
||||
> The failing seed gives us a reproducible behavioral example. I'll replay it first so we can distinguish a real regression from source drift or nondeterministic app state.
|
||||
|
||||
## Progressive Complexity
|
||||
|
||||
Start with behavioral contracts and add depth only where it pays off:
|
||||
Start simple and add depth only where it pays off:
|
||||
|
||||
**Level 1 — Cross-route behavior**: Every constructor checks retrievability.
|
||||
**Level 1 — Status and shape**: Every route gets an expected status code and key field existence.
|
||||
```apostl
|
||||
response_code(GET /users/{response_body(this).id}) == 200
|
||||
status:201
|
||||
response_body(this).id != null
|
||||
```
|
||||
|
||||
**Level 2 — State persistence**: Mutators check that changes are visible.
|
||||
**Level 2 — Cross-route behavior**: Constructors check retrievability; mutators check persistence.
|
||||
```apostl
|
||||
response_body(GET /users/{request_params(this).id}).email == request_body(this).email
|
||||
response_code(GET /users/{response_body(this).id}) == 200
|
||||
response_body(GET /users/{response_body(this).id}).email == request_body(this).email
|
||||
```
|
||||
|
||||
**Level 3 — Isolation and boundaries**: Tenant, auth, and idempotency checks.
|
||||
@@ -419,21 +377,19 @@ if request_headers(this).x-tenant-id != null then response_headers(this).x-tenan
|
||||
|
||||
**Level 4 — Protocol and dependency flows**: Variants, scenarios, outbound contracts, and chaos.
|
||||
|
||||
Add level 1 before level 4. Do not skip level 1 for resource APIs.
|
||||
Add level 2 before level 4. Do not skip level 2 for resource APIs.
|
||||
|
||||
## Anti-Patterns
|
||||
|
||||
Do not:
|
||||
|
||||
1. Assert only `status:200` everywhere. Schema validation already checks this.
|
||||
2. Check `response_body(this).id != null` when the schema already requires `id`.
|
||||
3. Duplicate JSON Schema checks while ignoring cross-route behavior.
|
||||
4. Encode route internals instead of API-observable outcomes.
|
||||
5. Ignore delete/retrieve or update/retrieve relationships.
|
||||
6. Treat stateful mode as optional for resource APIs.
|
||||
7. **Weaken a contract to make a test pass without proving the implementation is correct.**
|
||||
8. Ask the operator to review every formula before running; run first when intent is clear, then ask about ambiguous domain behavior.
|
||||
9. Load every doc file before making a small change.
|
||||
1. Assert only `status:200` everywhere.
|
||||
2. Duplicate JSON Schema checks while ignoring behavior.
|
||||
3. Encode route internals instead of API-observable outcomes.
|
||||
4. Ignore delete/retrieve or update/retrieve relationships.
|
||||
5. Treat stateful mode as optional for resource APIs.
|
||||
6. Ask the operator to review every formula before running; run first when intent is clear, then ask about ambiguous domain behavior.
|
||||
7. Load every doc file before making a small change.
|
||||
|
||||
## Verification Commands
|
||||
|
||||
@@ -451,7 +407,7 @@ Then execute APOPHIS from the project test harness or CLI as appropriate. For mo
|
||||
1. `README.md` for canonical usage.
|
||||
2. `docs/getting-started.md` for quick setup.
|
||||
3. `docs/cli.md` and command docs for CLI flags and machine output.
|
||||
4. `docs/attic/protocol-extensions-spec.md` for protocol-specific direction.
|
||||
4. `docs/protocol-extensions-spec.md` for protocol-specific direction.
|
||||
|
||||
## Final Check
|
||||
|
||||
@@ -461,6 +417,5 @@ For each route, ask:
|
||||
2. What must be true after this call?
|
||||
3. What related call should now behave differently?
|
||||
4. What isolation, security, dependency, or protocol expectation should not regress?
|
||||
5. If a contract fails, is the implementation wrong or is the contract wrong?
|
||||
|
||||
Write those expectations as behavioral formulas, run them continuously, and treat every failure as a bug to investigate—not an obstacle to remove.
|
||||
Write those expectations as formulas and run them continuously.
|
||||
|
||||
+1
-4
@@ -1,9 +1,6 @@
|
||||
## Outbound Contract-Driven Mocking Spec
|
||||
|
||||
Status: Implemented (Phase 1)
|
||||
|
||||
Phase 1 (implemented): Schema parsing (`x-outbound`), mock runtime, imperative API (`enableOutboundMocks`, `getOutboundCalls`), fetch patching.
|
||||
Phase 2 (pending): APOSTL extensions `outbound_calls(this)` and `outbound_last(this)` for contract assertions.
|
||||
Status: Proposed
|
||||
Date: 2026-04-27
|
||||
|
||||
This document supersedes Arbiter's local draft at `~/Business/workspace/Arbiter/docs/APOPHIS_OUTBOUND_MOCK_PROPOSAL.md` and its interim adapter at `~/Business/workspace/Arbiter/src/server/server/services/StripeFetchAdapter.js`.
|
||||
@@ -1,10 +1,6 @@
|
||||
# APOPHIS Plugin Contract System Specification
|
||||
|
||||
## Status: Partially implemented
|
||||
|
||||
- Registry, types, and registration API: **implemented**
|
||||
- Runner integration (merging plugin contracts into route execution): **pending**
|
||||
- Built-in contracts for `@fastify/auth`, `@fastify/compress`, `@fastify/cors`, `@fastify/rate-limit`: **registered but not yet applied**
|
||||
## Status: Implemented
|
||||
|
||||
**Note**: Plugin contracts are complementary to Protocol Extensions (see `docs/protocol-extensions-spec.md`). Protocol extensions add domain-specific predicates (JWT, X.509, SPIFFE); plugin contracts add hook-phase behavioral contracts for Fastify plugins.
|
||||
|
||||
@@ -135,7 +135,7 @@ fastify.get('/wimse/wit', {
|
||||
})
|
||||
```
|
||||
|
||||
See `docs/attic/protocol-extensions-spec.md` for full JWT extension configuration.
|
||||
See `docs/protocol-extensions-spec.md` for full JWT extension configuration.
|
||||
|
||||
---
|
||||
|
||||
|
||||
+15
-17
@@ -8,12 +8,13 @@ Chaos testing applies the invariant-driven verification approach from [Invariant
|
||||
|
||||
```javascript
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 50,
|
||||
depth: 'standard',
|
||||
chaos: {
|
||||
delay: { probability: 0.1, minMs: 100, maxMs: 500 },
|
||||
error: { probability: 0.1, statusCode: 503 },
|
||||
dropout: { probability: 0.05 },
|
||||
corruption: { probability: 0.1 },
|
||||
probability: 0.1, // 10% of requests get chaos
|
||||
delay: { probability: 1, minMs: 100, maxMs: 500 },
|
||||
error: { probability: 1, statusCode: 503 },
|
||||
dropout: { probability: 1 },
|
||||
corruption: { probability: 1 },
|
||||
},
|
||||
});
|
||||
```
|
||||
@@ -25,18 +26,16 @@ const result = await fastify.apophis.contract({
|
||||
Adds artificial latency. Tests timeout contracts:
|
||||
|
||||
```apostl
|
||||
timeout_occurred(this) == false
|
||||
response_time(this) < 1000
|
||||
```
|
||||
|
||||
**Note**: Delay events are generated by the chaos arbitrary but the inbound delay handler is currently a no-op. Use this for timeout contract documentation; actual delay injection requires the outbound delay strategy or a custom handler.
|
||||
|
||||
### Error
|
||||
|
||||
Forces HTTP status codes. Tests error-handling contracts:
|
||||
|
||||
```apostl
|
||||
// Behavioral: when the service is unavailable, the client receives a valid retry signal
|
||||
if status:503 then response_headers(this).retry-after > 0
|
||||
if status:503 then response_body(this).retry_after != null
|
||||
```
|
||||
|
||||
### Dropout
|
||||
@@ -44,8 +43,7 @@ if status:503 then response_headers(this).retry-after > 0
|
||||
Simulates network failure (status 0). Tests fallback contracts:
|
||||
|
||||
```apostl
|
||||
// Behavioral: partial failure must still return previously cached data
|
||||
if status:0 then response_body(this).cached_data == previous(response_body(GET /cache/{request_params(this).key}))
|
||||
status:200 || status:0
|
||||
```
|
||||
|
||||
### Corruption
|
||||
@@ -53,8 +51,7 @@ if status:0 then response_body(this).cached_data == previous(response_body(GET /
|
||||
Mutates response bodies. Tests parsing robustness:
|
||||
|
||||
```apostl
|
||||
// Behavioral: corrupted requests maintain traceability for debugging
|
||||
if status:400 then response_body(this).request_id == request_headers(this).x-request-id
|
||||
response_body(this).id != null
|
||||
```
|
||||
|
||||
## Corruption Strategies
|
||||
@@ -105,7 +102,7 @@ Failed tests include chaos events in diagnostics:
|
||||
```json
|
||||
{
|
||||
"statusCode": 503,
|
||||
"error": "Contract violation: if status:503 then response_headers(this).retry-after > 0",
|
||||
"error": "Contract violation: status:200",
|
||||
"chaosEvents": [
|
||||
{
|
||||
"type": "error",
|
||||
@@ -123,7 +120,7 @@ Failed tests include chaos events in diagnostics:
|
||||
|
||||
1. **Start small**: `probability: 0.05` (5% of requests)
|
||||
2. **Test one failure mode at a time**: Comment out other chaos types
|
||||
3. **Verify contracts handle chaos**: `if status:503 then response_code(GET /health) == 200`
|
||||
3. **Verify contracts handle chaos**: `if status:503 then response_body(this).error != null`
|
||||
4. **Use seeds for reproducibility**: `seed: 42` makes chaos deterministic
|
||||
|
||||
## Example: Testing Retry Logic
|
||||
@@ -132,7 +129,7 @@ Failed tests include chaos events in diagnostics:
|
||||
fastify.get('/data', {
|
||||
schema: {
|
||||
'x-ensures': [
|
||||
'if status:503 then response_headers(this).retry-after > 0',
|
||||
'if status:503 then response_headers(this).retry-after != null',
|
||||
'redirect_count(this) <= 3',
|
||||
],
|
||||
},
|
||||
@@ -141,7 +138,8 @@ fastify.get('/data', {
|
||||
// Test
|
||||
const result = await fastify.apophis.contract({
|
||||
chaos: {
|
||||
error: { probability: 0.2, statusCode: 503 },
|
||||
probability: 0.2,
|
||||
error: { probability: 1, statusCode: 503 },
|
||||
},
|
||||
});
|
||||
```
|
||||
|
||||
+21
-8
@@ -10,6 +10,7 @@ Every command accepts these flags:
|
||||
|---|---|---|
|
||||
| `--config <path>` | Config file path | Auto-detect |
|
||||
| `--profile <name>` | Profile name from config | First profile |
|
||||
| `--generation-profile <name>` | Generation budget profile (built-in: quick, standard, deep) | Depth-derived |
|
||||
| `--cwd <path>` | Working directory override | `process.cwd()` |
|
||||
| `--format <mode>` | Output format: `human`, `json`, `ndjson`, `json-summary`, `ndjson-summary` | `human` |
|
||||
| `--color <mode>` | Color mode: `auto`, `always`, `never` | `auto` |
|
||||
@@ -61,6 +62,7 @@ apophis verify --profile quick --routes "POST /users"
|
||||
| Flag | Description |
|
||||
|---|---|
|
||||
| `--profile <name>` | Profile name from config |
|
||||
| `--generation-profile <name>` | Override generation budget for this run |
|
||||
| `--routes <filter>` | Route filter pattern (comma-separated, supports wildcards) |
|
||||
| `--seed <number>` | Deterministic seed (generated and printed if omitted) |
|
||||
| `--changed` | Filter to git-modified routes only |
|
||||
@@ -119,6 +121,7 @@ apophis qualify --profile oauth-nightly --seed 42
|
||||
| Flag | Description |
|
||||
|---|---|
|
||||
| `--profile <name>` | Profile name from config |
|
||||
| `--generation-profile <name>` | Override generation budget for this run |
|
||||
| `--seed <number>` | Deterministic seed (generated and printed if omitted) |
|
||||
|
||||
**Examples:**
|
||||
@@ -126,6 +129,18 @@ apophis qualify --profile oauth-nightly --seed 42
|
||||
```bash
|
||||
apophis qualify --profile oauth-nightly --seed 42
|
||||
apophis qualify --profile lifecycle-deep
|
||||
apophis qualify --profile oauth-nightly --generation-profile quick
|
||||
```
|
||||
|
||||
You can define aliases in config:
|
||||
|
||||
```js
|
||||
export default {
|
||||
generationProfiles: {
|
||||
pr: 'quick',
|
||||
nightly: { base: 'thorough' },
|
||||
},
|
||||
}
|
||||
```
|
||||
|
||||
### `apophis replay`
|
||||
@@ -221,7 +236,7 @@ apophis replay --artifact reports/apophis/failure-*.json
|
||||
|
||||
- `--changed` requires a git repository
|
||||
- `migrate` defaults to `--dry-run` (safe by default)
|
||||
- `--workspace` is fully implemented by `verify` and `doctor`. `observe` and `qualify` accept the flag but run in the current package only.
|
||||
- `--workspace` is only supported by `verify` and `doctor` commands
|
||||
- Seeds ensure deterministic generation; handler nondeterminism (e.g., `Date.now()`) can still cause replay divergence
|
||||
|
||||
## Exit Codes
|
||||
@@ -240,12 +255,10 @@ apophis replay --artifact reports/apophis/failure-*.json
|
||||
|---|---|---|---|---|
|
||||
| `verify` | enabled | enabled | optional | optional, usually off |
|
||||
| `observe` | optional | optional | enabled | enabled |
|
||||
| `qualify` | enabled | enabled | optional | disabled by default |
|
||||
| `chaos` | enabled | enabled | optional | disabled by default |
|
||||
| `qualify: scenario` | enabled | enabled | enabled with allowlist | disabled by default |
|
||||
| `qualify: stateful` | enabled | enabled | synthetic-only | disabled by default |
|
||||
| `qualify: chaos` | enabled | enabled | canary-only | disabled by default |
|
||||
| outbound mocks | enabled | enabled | allowlisted only | disabled by default |
|
||||
| runtime throw-on-violation | optional | optional | exceptional | disabled by default |
|
||||
|
||||
Notes:
|
||||
- `qualify` is gated as a whole. The code does not distinguish scenario, stateful, and chaos sub-modes in environment policy.
|
||||
- `chaos` on protected routes requires `allowChaosOnProtected: true`.
|
||||
- `observe` blocking requires `allowBlocking: true`.
|
||||
- Production must never inherit qualify capabilities accidentally from a generic config file.
|
||||
Operational rule: Production must never inherit qualify capabilities accidentally from a generic config file.
|
||||
|
||||
+28
-42
@@ -5,25 +5,21 @@ import crypto from 'crypto'
|
||||
const fastify = Fastify()
|
||||
|
||||
await fastify.register(apophisPlugin, {
|
||||
runtime: 'error',
|
||||
cleanup: true,
|
||||
runtime: 'error', // Validate contracts on every request
|
||||
cleanup: true, // Auto-cleanup resources on exit
|
||||
})
|
||||
|
||||
// In-memory store for demo
|
||||
const users = new Map<string, { id: string; email: string; name: string }>()
|
||||
|
||||
// CREATE — constructor
|
||||
// Behavioral: the created user must be retrievable.
|
||||
// Note: we do not write 'status:201' or 'response_body(this).id != null'.
|
||||
// The schema already validates status codes and required fields.
|
||||
// Contracts should test behavior across operations, not structure.
|
||||
fastify.post('/users', {
|
||||
schema: {
|
||||
'x-category': 'constructor',
|
||||
'x-ensures': [
|
||||
// Round-trip: the server returns exactly what we sent (no mutation, no drops)
|
||||
'response_body(this) == request_body(this)',
|
||||
// Cross-route: the created user must be retrievable
|
||||
'response_code(GET /users/{response_body(this).id}) == 200',
|
||||
'status:201',
|
||||
'response_body(this).id != null',
|
||||
'response_body(this).email == request_body(this).email',
|
||||
],
|
||||
body: {
|
||||
type: 'object',
|
||||
@@ -53,21 +49,19 @@ fastify.post('/users', {
|
||||
})
|
||||
|
||||
// READ — observer
|
||||
// Behavioral: the returned user must match the requested id.
|
||||
fastify.get('/users/:id', {
|
||||
schema: {
|
||||
'x-category': 'observer',
|
||||
'x-requires': [
|
||||
// Precondition: the user must exist for this read to be valid
|
||||
'response_code(GET /users/{request_params(this).id}) == 200'
|
||||
],
|
||||
'x-requires': ['users:id'],
|
||||
'x-ensures': [
|
||||
// The returned id must match the requested id (no mix-up)
|
||||
'status:200',
|
||||
'response_body(this).id == request_params(this).id',
|
||||
],
|
||||
params: {
|
||||
type: 'object',
|
||||
properties: { id: { type: 'string' } },
|
||||
properties: {
|
||||
id: { type: 'string' }
|
||||
},
|
||||
required: ['id']
|
||||
},
|
||||
response: {
|
||||
@@ -90,21 +84,19 @@ fastify.get('/users/:id', {
|
||||
})
|
||||
|
||||
// UPDATE — mutator
|
||||
// Behavioral: after update, the change must be visible on read.
|
||||
fastify.put('/users/:id', {
|
||||
schema: {
|
||||
'x-category': 'mutator',
|
||||
'x-requires': [
|
||||
// The user must exist before updating
|
||||
'response_code(GET /users/{request_params(this).id}) == 200'
|
||||
],
|
||||
'x-requires': ['users:id'],
|
||||
'x-ensures': [
|
||||
// Cross-route: after update, reading the user shows the new data
|
||||
'response_body(GET /users/{request_params(this).id}).email == request_body(this).email',
|
||||
'status:200',
|
||||
'response_body(this).id == request_params(this).id',
|
||||
],
|
||||
params: {
|
||||
type: 'object',
|
||||
properties: { id: { type: 'string' } },
|
||||
properties: {
|
||||
id: { type: 'string' }
|
||||
},
|
||||
required: ['id']
|
||||
},
|
||||
body: {
|
||||
@@ -140,40 +132,34 @@ fastify.put('/users/:id', {
|
||||
})
|
||||
|
||||
// DELETE — destructor
|
||||
// Behavioral: after deletion, the user must no longer exist.
|
||||
fastify.delete('/users/:id', {
|
||||
schema: {
|
||||
'x-category': 'destructor',
|
||||
'x-requires': [
|
||||
// The user must exist before deleting
|
||||
'response_code(GET /users/{request_params(this).id}) == 200'
|
||||
],
|
||||
'x-ensures': [
|
||||
// After deletion, the user is gone
|
||||
'response_code(GET /users/{request_params(this).id}) == 404',
|
||||
// The deleted user data is returned (matches pre-deletion read)
|
||||
'response_body(this) == previous(response_body(GET /users/{request_params(this).id}))',
|
||||
],
|
||||
'x-requires': ['users:id'],
|
||||
'x-ensures': ['status:204'],
|
||||
params: {
|
||||
type: 'object',
|
||||
properties: { id: { type: 'string' } },
|
||||
properties: {
|
||||
id: { type: 'string' }
|
||||
},
|
||||
required: ['id']
|
||||
}
|
||||
}
|
||||
}, async (req, reply) => {
|
||||
const user = users.get(req.params.id)
|
||||
users.delete(req.params.id)
|
||||
reply.status(200)
|
||||
return user
|
||||
reply.status(204)
|
||||
})
|
||||
|
||||
await fastify.ready()
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 50 })
|
||||
// Run contract tests (all non-utility routes, property-based)
|
||||
const result = await fastify.apophis.contract({ depth: 'standard' })
|
||||
console.log('Contract tests:', result.summary)
|
||||
|
||||
const stateful = await fastify.apophis.stateful({ runs: 50, seed: 42 })
|
||||
// Run stateful tests (constructor→mutator→destructor sequences)
|
||||
const stateful = await fastify.apophis.stateful({ depth: 'standard', seed: 42 })
|
||||
console.log('Stateful tests:', stateful.summary)
|
||||
|
||||
// Validate a single route
|
||||
const check = await fastify.apophis.check('POST', '/users')
|
||||
console.log('POST /users check:', check.ok ? 'PASS' : 'FAIL')
|
||||
|
||||
@@ -6,30 +6,21 @@ const fastify = Fastify()
|
||||
// APOPHIS auto-registers @fastify/swagger
|
||||
await fastify.register(apophisPlugin, {})
|
||||
|
||||
// Behavioral contract: what you send is what you get back.
|
||||
// This is not a structural test — the schema already validates shape.
|
||||
// This checks that the server does not mutate or drop fields.
|
||||
fastify.post('/echo', {
|
||||
fastify.get('/health', {
|
||||
schema: {
|
||||
'x-category': 'observer',
|
||||
'x-ensures': [
|
||||
'response_body(this) == request_body(this)'
|
||||
],
|
||||
body: {
|
||||
type: 'object',
|
||||
properties: { message: { type: 'string' } }
|
||||
},
|
||||
'x-ensures': ['status:200'],
|
||||
response: {
|
||||
200: {
|
||||
type: 'object',
|
||||
properties: { message: { type: 'string' } }
|
||||
properties: { status: { type: 'string' } }
|
||||
}
|
||||
}
|
||||
}
|
||||
}, async (req) => req.body)
|
||||
}, async () => ({ status: 'ok' }))
|
||||
|
||||
await fastify.ready()
|
||||
|
||||
// Run contract tests
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
console.log(result.summary)
|
||||
|
||||
@@ -280,8 +280,8 @@ app.get('/users/:id', {
|
||||
type: 'object',
|
||||
properties: { id: { type: 'string' } },
|
||||
'x-ensures': [
|
||||
// Behavioral: returned user must match the requested id
|
||||
'response_body(this).id == request_params(this).id',
|
||||
// Standard APOSTL + extension predicates
|
||||
'status:200',
|
||||
'graph_check(this).user.can_read_user == true',
|
||||
'partial_graph(this).tenant.accessible == true',
|
||||
],
|
||||
|
||||
@@ -284,8 +284,8 @@ fastify.get('/api/resource', {
|
||||
'x-ensures': [
|
||||
'timeout_occurred(this) == false',
|
||||
'redirect_count(this) == 0',
|
||||
// Behavioral: created resource must be retrievable
|
||||
'response_code(GET /api/resource/{response_body(this).id}) == 200',
|
||||
'response_code(this) == 200',
|
||||
'response_body(this).id != null',
|
||||
]
|
||||
}
|
||||
}, handler)
|
||||
|
||||
+1
-164
@@ -50,7 +50,7 @@ app.post('/users', {
|
||||
});
|
||||
```
|
||||
|
||||
> **Warning:** Using `Date.now()` or `Math.random()` in handlers breaks determinism and replay. Use a stable function of the input instead. APOPHIS does not proactively detect nondeterministic handlers; it warns only when a replay diverges from the original run.
|
||||
> **Warning:** Using `Date.now()` or `Math.random()` in handlers breaks determinism and replay. Use a stable function of the input instead.
|
||||
|
||||
## Step 4: Run Verify
|
||||
|
||||
@@ -94,48 +94,6 @@ apophis replay --artifact reports/apophis/failure-2026-04-28T12-30-22Z.json
|
||||
|
||||
Fix the bug in your handler. Re-run verify. The failure should now pass.
|
||||
|
||||
## Behavioral vs Structural Contracts
|
||||
|
||||
APOPHIS contracts should verify **behavior**, not structure. Fastify and `@fastify/swagger` already enforce status codes, required fields, and types. Behavioral contracts catch what schemas cannot:
|
||||
|
||||
| Structural (avoid) | Behavioral (prefer) |
|
||||
|---|---|
|
||||
| `status:200` | `response_body(this) == request_body(this)` |
|
||||
| `response_body(this).id != null` | `response_code(GET /users/{response_body(this).id}) == 200` |
|
||||
| `response_body(this).name != null` | `response_body(GET /users/{id}).name == previous(response_body(this).name)` |
|
||||
|
||||
**Good behavioral patterns (from the paper):**
|
||||
- **Constructor precondition**: Resource must not exist before creation
|
||||
```apostl
|
||||
response_code(GET /users/{request_body(this).email}) == 404
|
||||
```
|
||||
- **Round-trip equality**: POST response matches the request body
|
||||
```apostl
|
||||
response_body(this) == request_body(this)
|
||||
```
|
||||
- **Cross-route retrievability**: Creating a resource makes it readable via GET
|
||||
```apostl
|
||||
response_code(GET /users/{response_body(this).id}) == 200
|
||||
```
|
||||
- **State-change verification**: DELETE causes subsequent GET to return 404
|
||||
```apostl
|
||||
response_code(GET /users/{request_params(this).id}) == 404
|
||||
```
|
||||
- **Previous state preservation**: DELETE returns the last known state
|
||||
```apostl
|
||||
response_body(this) == previous(response_body(GET /users/{request_params(this).id}))
|
||||
```
|
||||
- **Invariant over collections**: All resources satisfy a cross-resource constraint
|
||||
```apostl
|
||||
for t in response_body(GET /tournaments) :-
|
||||
response_body(GET /tournaments/{t.id}/players).length <= t.capacity
|
||||
```
|
||||
|
||||
**Anti-patterns to avoid:**
|
||||
- Checking status codes (handled by schema validation)
|
||||
- Checking field existence (handled by schema validation)
|
||||
- Checking field types (handled by schema validation)
|
||||
|
||||
## Next Steps
|
||||
|
||||
- Add more routes to your profile: `apophis verify --profile quick --routes "POST /users,PUT /users/:id"`
|
||||
@@ -147,127 +105,6 @@ APOPHIS contracts should verify **behavior**, not structure. Fastify and `@fasti
|
||||
- Add observe mode for runtime drift detection: see [observe.md](observe.md)
|
||||
- Add qualify mode for scenario, stateful, and chaos checks: see [qualify.md](qualify.md)
|
||||
|
||||
## Variants
|
||||
|
||||
Test the same route with different headers or content types:
|
||||
|
||||
```javascript
|
||||
await fastify.apophis.contract({
|
||||
variants: [
|
||||
{ name: 'json', headers: { accept: 'application/json' } },
|
||||
{ name: 'xml', headers: { accept: 'application/xml' } }
|
||||
]
|
||||
})
|
||||
```
|
||||
|
||||
Or declare variants in the route schema:
|
||||
|
||||
```javascript
|
||||
app.get('/users', {
|
||||
schema: {
|
||||
'x-variants': [
|
||||
{ name: 'json', headers: { accept: 'application/json' } }
|
||||
]
|
||||
}
|
||||
})
|
||||
```
|
||||
|
||||
## Plugin Options
|
||||
|
||||
When registering the APOPHIS plugin, you can pass these options:
|
||||
|
||||
```javascript
|
||||
await fastify.register(apophis, {
|
||||
// Swagger config passthrough (if @fastify/swagger is not already registered)
|
||||
swagger: { openapi: { info: { title: 'API', version: '1.0.0' } } },
|
||||
|
||||
// Runtime contract validation hooks: 'off', 'warn', or 'error'
|
||||
// Only active in non-production environments
|
||||
runtime: 'warn',
|
||||
|
||||
// Automatically clean up tracked resources after tests
|
||||
cleanup: true,
|
||||
|
||||
// Global timeout in milliseconds for all requests
|
||||
timeout: 5000,
|
||||
|
||||
// Tenant isolation scopes
|
||||
scopes: {
|
||||
tenant1: { headers: { 'x-tenant-id': '1' } },
|
||||
tenant2: { headers: { 'x-tenant-id': '2' } },
|
||||
},
|
||||
|
||||
// Auth and protocol extensions
|
||||
extensions: [jwtAuth, apiKeyAuth],
|
||||
|
||||
// Plugin hook-phase contracts
|
||||
pluginContracts: {
|
||||
'rate-limit': { appliesTo: 'POST /users', ensures: ['status != 429'] },
|
||||
},
|
||||
|
||||
// Outbound dependency contracts
|
||||
outboundContracts: {
|
||||
'payment-api': {
|
||||
target: 'https://payments.example.com',
|
||||
method: 'POST',
|
||||
response: { 200: { type: 'object', properties: { id: { type: 'string' } } } }
|
||||
}
|
||||
}
|
||||
})
|
||||
```
|
||||
|
||||
## Schema Annotations
|
||||
|
||||
APOPHIS reads these OpenAPI schema extensions:
|
||||
|
||||
| Annotation | Location | Description |
|
||||
|---|---|---|
|
||||
| `x-category` | Top-level | Route classification: `constructor`, `mutator`, `observer`, `destructor`, `utility` |
|
||||
| `x-ensures` | Top-level or `response[statusCode]` | Post-condition contracts (APOSTL formulas) |
|
||||
| `x-requires` | Top-level or `response[statusCode]` | Pre-condition contracts (APOSTL formulas) |
|
||||
| `x-variants` | Top-level | Request variants for content-type negotiation or feature flags |
|
||||
| `x-timeout` | Top-level or `response[statusCode]` | Per-route timeout in milliseconds |
|
||||
| `x-outbound` | Top-level | Outbound dependency contracts for this route |
|
||||
| `x-streaming` | Top-level | Mark route as streaming (populates `chunks` and `streamDurationMs` in eval context) |
|
||||
| `x-validate-runtime` | Top-level or `response[statusCode]` | Toggle runtime validation for this route (default: true) |
|
||||
| `x-extension-config` | Top-level | Per-route config for extensions (e.g., `{ jwt: { verify: false } }`) |
|
||||
|
||||
Annotations can be placed on the top-level schema or nested inside `response[statusCode]`. Nested annotations take precedence for that status code.
|
||||
|
||||
## Programmatic API
|
||||
|
||||
After registration, `fastify.apophis` provides:
|
||||
|
||||
```javascript
|
||||
// Run contract tests for all routes
|
||||
const suite = await fastify.apophis.contract({ runs: 50, seed: 42 })
|
||||
|
||||
// Run stateful tests
|
||||
const stateful = await fastify.apophis.stateful({ runs: 50, seed: 42 })
|
||||
|
||||
// Run a single scenario
|
||||
const scenario = await fastify.apophis.scenario({
|
||||
name: 'oauth-basic',
|
||||
steps: [...]
|
||||
})
|
||||
|
||||
// Check a single route
|
||||
const result = await fastify.apophis.check('GET', '/users/:id')
|
||||
|
||||
// Get enriched OpenAPI spec with contract metadata
|
||||
const spec = fastify.apophis.spec()
|
||||
|
||||
// Clean up tracked resources
|
||||
await fastify.apophis.cleanup()
|
||||
|
||||
// Test-only utilities (NODE_ENV=test only)
|
||||
fastify.apophis.test.registerPluginContracts('name', spec)
|
||||
fastify.apophis.test.registerOutboundContracts({ ... })
|
||||
fastify.apophis.test.enableOutboundMocks({ mode: 'example' })
|
||||
fastify.apophis.test.disableOutboundMocks()
|
||||
const calls = fastify.apophis.test.getOutboundCalls('payment-api')
|
||||
```
|
||||
|
||||
## Config Reference
|
||||
|
||||
For the full configuration reference, see [CLI Reference](cli.md).
|
||||
|
||||
@@ -86,6 +86,7 @@ export default {
|
||||
presets: {
|
||||
'llm-safe': {
|
||||
name: 'llm-safe',
|
||||
depth: 'quick',
|
||||
timeout: 3000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -135,6 +135,7 @@ export default {
|
||||
presets: {
|
||||
'platform-observe': {
|
||||
name: 'platform-observe',
|
||||
depth: 'standard',
|
||||
timeout: 10000,
|
||||
parallel: true,
|
||||
chaos: false,
|
||||
|
||||
+5
-5
@@ -23,8 +23,8 @@ BENCH_RUNS=12 BENCH_WARMUP=3 npm run benchmark:cli
|
||||
# Increase inner-loop work for micro-benchmarks
|
||||
BENCH_INNER_ITERS=5000 npm run benchmark:hot
|
||||
|
||||
# Benchmark with varying test counts
|
||||
BENCH_RUNS=10,50,200 npm run benchmark:all
|
||||
# Benchmark generation profile matrix
|
||||
BENCH_GENERATION_PROFILES=quick,standard,thorough npm run benchmark:all
|
||||
```
|
||||
|
||||
## Capture CPU Profile for Qualify
|
||||
@@ -41,10 +41,10 @@ This writes Chrome-compatible CPU profiles to `.profiles/qualify.cpuprofile` and
|
||||
- CLI benchmark uses spawned `node dist/cli/index.js` commands so startup costs are included.
|
||||
- Hot path benchmark runs in-process for lower-noise function-level comparisons.
|
||||
- Use fixed `--seed` for qualify benchmarks to keep runs deterministic.
|
||||
- Schema generation uses fixed defaults (string≤128, array≤10) regardless of run count.
|
||||
- Generation now adapts to depth: `quick` favors bounded payload generation speed, `thorough` keeps broader generation.
|
||||
|
||||
You can override runs per preset:
|
||||
You can override generation per run:
|
||||
|
||||
```bash
|
||||
apophis qualify --profile oauth-nightly --seed 42
|
||||
apophis qualify --profile oauth-nightly --generation-profile quick --seed 42
|
||||
```
|
||||
|
||||
+19
-18
@@ -58,12 +58,11 @@ Scenarios are multi-step flows with capture and rebind:
|
||||
```javascript
|
||||
await fastify.apophis.scenario({
|
||||
name: 'oauth-basic',
|
||||
steps: [
|
||||
steps: [
|
||||
{
|
||||
name: 'authorize',
|
||||
request: { method: 'GET', url: '/oauth/authorize?client_id=web&response_type=code&state=abc123' },
|
||||
// Behavioral: state parameter round-trips for CSRF protection
|
||||
expect: ['response_payload(this).state == request_query(this).state'],
|
||||
request: { method: 'GET', url: '/oauth/authorize?client_id=web&response_type=code' },
|
||||
expect: ['status:200', 'response_payload(this).code != null'],
|
||||
capture: { code: 'response_payload(this).code' }
|
||||
},
|
||||
{
|
||||
@@ -71,10 +70,9 @@ await fastify.apophis.scenario({
|
||||
request: {
|
||||
method: 'POST',
|
||||
url: '/oauth/token',
|
||||
form: { grant_type: 'authorization_code', code: '$authorize.code', scope: 'read' }
|
||||
form: { grant_type: 'authorization_code', code: '$authorize.code' }
|
||||
},
|
||||
// Behavioral: issued token preserves the requested scope
|
||||
expect: ['response_payload(this).scope == request_body(this).scope']
|
||||
expect: ['status:200', 'response_payload(this).access_token != null']
|
||||
}
|
||||
]
|
||||
})
|
||||
@@ -99,7 +97,7 @@ APOPHIS tracks created resources and runs cleanup after test completion.
|
||||
Run stateful tests via the API:
|
||||
|
||||
```javascript
|
||||
const stateful = await fastify.apophis.stateful({ runs: 50, seed: 42 })
|
||||
const stateful = await fastify.apophis.stateful({ depth: 'standard', seed: 42 })
|
||||
console.log('Stateful tests:', stateful.summary)
|
||||
```
|
||||
|
||||
@@ -205,7 +203,7 @@ export default {
|
||||
presets: {
|
||||
'protocol-lab': {
|
||||
name: 'protocol-lab',
|
||||
runs: 200,
|
||||
depth: 'deep',
|
||||
timeout: 15000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
@@ -252,15 +250,18 @@ Human output shows per-gate execution counts (scenario, stateful, chaos, adversi
|
||||
|
||||
Qualify exits with code 1 if zero checks executed. This prevents silent passes when all routes are filtered out or gates are disabled.
|
||||
|
||||
## Test Budget
|
||||
## `--workspace` Flag
|
||||
|
||||
The `runs` field in your preset controls how many property-based tests execute per route. Default is 50. Lower for faster CI feedback, higher for deeper exploration:
|
||||
Run qualify across all packages in a monorepo workspace:
|
||||
|
||||
```javascript
|
||||
presets: {
|
||||
'protocol-lab': {
|
||||
runs: 200,
|
||||
timeout: 15000
|
||||
}
|
||||
}
|
||||
```bash
|
||||
apophis qualify --workspace --profile oauth-nightly
|
||||
```
|
||||
|
||||
## `--generation-profile` Flag
|
||||
|
||||
Control test data generation depth independently from the qualification profile:
|
||||
|
||||
```bash
|
||||
apophis qualify --profile oauth-nightly --generation-profile quick
|
||||
```
|
||||
|
||||
-226
@@ -1,226 +0,0 @@
|
||||
# Quality Engines
|
||||
|
||||
APOPHIS includes three quality engines for advanced testing: chaos injection, flake detection, and mutation testing. All require `NODE_ENV=test`.
|
||||
|
||||
## Chaos Injection
|
||||
|
||||
Inject controlled failures into contract tests to validate resilience guarantees. Chaos events are generated by fast-check alongside test data, making them shrinkable — when a test fails, fast-check finds the minimal chaos event that causes the failure.
|
||||
|
||||
### Usage
|
||||
|
||||
```javascript
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 50,
|
||||
chaos: {
|
||||
delay: { probability: 0.1, minMs: 100, maxMs: 500 },
|
||||
error: { probability: 0.1, statusCode: 503 },
|
||||
dropout: { probability: 0.05 },
|
||||
corruption: { probability: 0.1 },
|
||||
},
|
||||
})
|
||||
```
|
||||
|
||||
### Event Types
|
||||
|
||||
| Type | Effect | Tests |
|
||||
|------|--------|-------|
|
||||
| `delay` | Artificial latency | `response_time(this) < 1000` |
|
||||
| `error` | Forces HTTP status code | Error-handling contracts |
|
||||
| `dropout` | Network failure (status 0 or 504) | Fallback contracts |
|
||||
| `corruption` | Mutates response bodies | Parsing robustness |
|
||||
|
||||
### Corruption Strategies
|
||||
|
||||
| Strategy | Effect |
|
||||
|----------|--------|
|
||||
| `truncate` | Cuts response body in half |
|
||||
| `malformed` | Returns invalid JSON (`{"broken":`) |
|
||||
| `field-corrupt` | Sets a random field to `null` |
|
||||
|
||||
### Programmatic API
|
||||
|
||||
```javascript
|
||||
import {
|
||||
applyChaosToExecution,
|
||||
createChaosEventArbitrary,
|
||||
formatChaosEvents,
|
||||
} from 'apophis-fastify'
|
||||
|
||||
// Apply pre-generated chaos events to a context
|
||||
const result = applyChaosToExecution(ctx, events)
|
||||
|
||||
// Generate deterministic chaos events
|
||||
const arb = createChaosEventArbitrary(config, contractNames)
|
||||
const events = fc.sample(arb, { numRuns: 1, seed: 42 })[0]
|
||||
|
||||
// Format for diagnostics
|
||||
console.log(formatChaosEvents(events))
|
||||
```
|
||||
|
||||
### Best Practices
|
||||
|
||||
1. Start small: `probability: 0.05` (5% of requests)
|
||||
2. Test one failure mode at a time
|
||||
3. Verify contracts handle chaos: `if status:503 then response_code(GET /health) == 200`
|
||||
4. Use seeds for reproducibility: `seed: 42`
|
||||
|
||||
## Flake Detection
|
||||
|
||||
Automatically rerun failing tests with varied seeds to detect non-deterministic contracts. A "flake" is a test that fails on one run but passes on another with the same or different seed.
|
||||
|
||||
### Usage
|
||||
|
||||
```javascript
|
||||
import { FlakeDetector } from 'apophis-fastify'
|
||||
|
||||
const detector = new FlakeDetector({
|
||||
sameSeedReruns: 1, // Rerun with same seed
|
||||
seedVariations: 3, // Try 3 additional seeds
|
||||
})
|
||||
|
||||
const report = await detector.detectFlake(
|
||||
originalFailingResult,
|
||||
async (seed) => {
|
||||
const suite = await fastify.apophis.contract({ seed })
|
||||
return { passed: suite.summary.failed === 0 }
|
||||
},
|
||||
originalSeed
|
||||
)
|
||||
|
||||
if (report.isFlaky) {
|
||||
console.log(`Flaky with ${report.confidence} confidence`)
|
||||
console.log('Reruns:', report.reruns)
|
||||
}
|
||||
```
|
||||
|
||||
### Report Structure
|
||||
|
||||
```javascript
|
||||
{
|
||||
isFlaky: true,
|
||||
confidence: 'high', // 'high' | 'medium' | 'low'
|
||||
reruns: [
|
||||
{ seed: 42, passed: false },
|
||||
{ seed: 43, passed: true },
|
||||
]
|
||||
}
|
||||
```
|
||||
|
||||
### Confidence Scoring
|
||||
|
||||
| Pass Rate | Confidence |
|
||||
|-----------|------------|
|
||||
| 0% pass | `high` (deterministic failure) |
|
||||
| < 50% pass | `medium` |
|
||||
| >= 50% pass | `low` (likely flaky) |
|
||||
|
||||
## Mutation Testing
|
||||
|
||||
Measure contract strength by injecting synthetic bugs. A "mutation" is a small change to a contract (e.g., flip `==` to `!=`). If the test suite catches the mutation (fails), the mutation is "killed". If it passes, the mutation "survives" — indicating weak coverage.
|
||||
|
||||
### Usage
|
||||
|
||||
```javascript
|
||||
import { runMutationTesting } from 'apophis-fastify/quality/mutation'
|
||||
|
||||
const report = await runMutationTesting(fastify, {
|
||||
runs: 10,
|
||||
seed: 42,
|
||||
maxMutationsPerContract: 5,
|
||||
routes: ['/items'], // Optional: only test these routes
|
||||
})
|
||||
|
||||
console.log(`Mutation score: ${report.score}%`)
|
||||
console.log(`Killed: ${report.killed}, Survived: ${report.survived}`)
|
||||
console.log('Weak contracts:', report.weakContracts)
|
||||
```
|
||||
|
||||
### Mutation Operators
|
||||
|
||||
| Type | Example |
|
||||
|------|---------|
|
||||
| `flip-operator` | `== 201` → `!= 201` |
|
||||
| `change-number` | `== 200` → `== 201` |
|
||||
| `remove-clause` | `A && B` → `A` |
|
||||
| `negate-boolean` | `== true` → `== false` |
|
||||
| `swap-variable` | `response_body` → `request_body` |
|
||||
| `remove-ensures` | Remove one ensures clause entirely |
|
||||
|
||||
### Report Structure
|
||||
|
||||
```javascript
|
||||
{
|
||||
score: 85, // 0-100
|
||||
killed: 17,
|
||||
survived: 3,
|
||||
durationMs: 4500,
|
||||
weakContracts: ['POST /items'], // Routes where no mutations were killed
|
||||
mutations: [
|
||||
{
|
||||
mutation: {
|
||||
id: 'm0',
|
||||
route: 'POST /items',
|
||||
original: 'response_code(this) == 201',
|
||||
mutated: 'response_code(this) != 201',
|
||||
type: 'flip-operator',
|
||||
},
|
||||
killed: true,
|
||||
durationMs: 120,
|
||||
}
|
||||
]
|
||||
}
|
||||
```
|
||||
|
||||
### Single Mutation Test
|
||||
|
||||
Test a specific mutation without running the full suite:
|
||||
|
||||
```javascript
|
||||
import { testMutation } from 'apophis-fastify/quality/mutation'
|
||||
|
||||
const killed = await testMutation(fastify, contract, mutation, {
|
||||
runs: 10,
|
||||
seed: 42,
|
||||
})
|
||||
```
|
||||
|
||||
## Environment Guard
|
||||
|
||||
All quality engines require `NODE_ENV=test`:
|
||||
|
||||
```
|
||||
Error: chaos is only available in test environment.
|
||||
Set NODE_ENV=test to enable quality features.
|
||||
```
|
||||
|
||||
This prevents accidental execution in production or development.
|
||||
|
||||
## Integration Example
|
||||
|
||||
Run all three engines in a CI pipeline:
|
||||
|
||||
```javascript
|
||||
// 1. Standard contract tests
|
||||
const suite = await fastify.apophis.contract({ runs: 50, seed: 42 })
|
||||
|
||||
// 2. Chaos tests
|
||||
const chaosSuite = await fastify.apophis.contract({
|
||||
runs: 50,
|
||||
seed: 42,
|
||||
chaos: { error: { probability: 0.1, statusCode: 503 } },
|
||||
})
|
||||
|
||||
// 3. Flake detection on failures
|
||||
for (const test of suite.tests.filter(t => !t.ok)) {
|
||||
const report = await detector.detectFlake(test, rerunFn, 42)
|
||||
if (report.isFlaky) {
|
||||
console.warn(`Flaky test detected: ${test.name}`)
|
||||
}
|
||||
}
|
||||
|
||||
// 4. Mutation testing
|
||||
const mutationReport = await runMutationTesting(fastify, { runs: 10 })
|
||||
if (mutationReport.score < 80) {
|
||||
console.warn(`Low mutation score: ${mutationReport.score}%`)
|
||||
}
|
||||
```
|
||||
+6
-18
@@ -81,7 +81,7 @@ Run only routes modified in the current git branch:
|
||||
apophis verify --profile ci --changed
|
||||
```
|
||||
|
||||
If no routes changed, exits 2 with a message.
|
||||
If no routes changed, exits 0 with a message.
|
||||
|
||||
## Failure Output Format
|
||||
|
||||
@@ -165,7 +165,7 @@ export default {
|
||||
},
|
||||
presets: {
|
||||
'safe-ci': {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
timeout: 5000
|
||||
}
|
||||
}
|
||||
@@ -184,22 +184,10 @@ apophis verify --workspace --profile quick --format json
|
||||
|
||||
Output includes per-package pass/fail summaries. Fails if any package fails.
|
||||
|
||||
## Test Budget
|
||||
## `--generation-profile` Flag
|
||||
|
||||
The `runs` field in your preset controls how many property-based tests execute per route. Default is 50. Lower for faster CI feedback, higher for deeper exploration:
|
||||
Control test data generation depth independently from the verification profile:
|
||||
|
||||
```javascript
|
||||
profiles: {
|
||||
quick: {
|
||||
mode: 'verify',
|
||||
preset: 'safe-ci',
|
||||
routes: ['POST /users']
|
||||
}
|
||||
},
|
||||
presets: {
|
||||
'safe-ci': {
|
||||
runs: 10,
|
||||
timeout: 5000
|
||||
}
|
||||
}
|
||||
```bash
|
||||
apophis verify --profile quick --generation-profile quick
|
||||
```
|
||||
|
||||
@@ -1,4 +1,3 @@
|
||||
import crypto from 'crypto'
|
||||
import type { FastifyInstance } from 'fastify'
|
||||
|
||||
export async function userRoutes(fastify: FastifyInstance) {
|
||||
@@ -31,7 +30,7 @@ export async function userRoutes(fastify: FastifyInstance) {
|
||||
},
|
||||
}, async (req, reply) => {
|
||||
const { name, email } = req.body as { name: string; email: string }
|
||||
const id = `user-${crypto.createHash('sha256').update(email).digest('hex').slice(0, 8)}`
|
||||
const id = `user-${Date.now()}`
|
||||
const user = { id, name, email }
|
||||
fastify.db.users.set(id, user)
|
||||
reply.status(201)
|
||||
|
||||
@@ -1,6 +0,0 @@
|
||||
|
||||
Mercedes Rodriguez
|
||||
#BOAF9148155679Z
|
||||
1500 - 00251743R
|
||||
2000 - 00361Z903R
|
||||
|
||||
+2
-2
@@ -1,5 +1,5 @@
|
||||
{
|
||||
"name": "@apophis/fastify",
|
||||
"name": "apophis-fastify",
|
||||
"version": "2.0.0",
|
||||
"description": "Contract-driven API testing plugin for Fastify with property-based testing, timeout enforcement, redirect capture, and deterministic concurrency",
|
||||
"main": "dist/index.js",
|
||||
@@ -50,7 +50,7 @@
|
||||
"benchmark:cli": "npm run build && node scripts/bench/cli.mjs",
|
||||
"benchmark:hot": "npm run build && node scripts/bench/hot-paths.mjs",
|
||||
"profile:qualify": "npm run build && mkdir -p .profiles && node --cpu-prof --cpu-prof-dir=.profiles --cpu-prof-name=qualify.cpuprofile dist/cli/index.js qualify --cwd src/cli/__fixtures__/protocol-lab --profile oauth-nightly --seed 42 --quiet",
|
||||
"profile:qualify:quick": "npm run build && mkdir -p .profiles && node --cpu-prof --cpu-prof-dir=.profiles --cpu-prof-name=qualify-quick.cpuprofile dist/cli/index.js qualify --cwd src/cli/__fixtures__/protocol-lab --profile oauth-nightly --seed 42 --quiet",
|
||||
"profile:qualify:quick": "npm run build && mkdir -p .profiles && node --cpu-prof --cpu-prof-dir=.profiles --cpu-prof-name=qualify-quick.cpuprofile dist/cli/index.js qualify --cwd src/cli/__fixtures__/protocol-lab --profile oauth-nightly --generation-profile quick --seed 42 --quiet",
|
||||
"clean": "rm -rf dist",
|
||||
"apophis:verify": "apophis verify --profile quick",
|
||||
"apophis:doctor": "apophis doctor"
|
||||
|
||||
+18
-1
@@ -8,13 +8,30 @@ const __dirname = fileURLToPath(new URL('.', import.meta.url))
|
||||
const repoRoot = resolve(__dirname, '..', '..')
|
||||
|
||||
const options = getBenchOptions()
|
||||
const generationProfiles = (process.env.BENCH_GENERATION_PROFILES ?? 'default,quick,standard,thorough')
|
||||
.split(',')
|
||||
.map((value) => value.trim())
|
||||
.filter(Boolean)
|
||||
|
||||
function withGenerationProfile(baseArgs, profile) {
|
||||
if (profile === 'default') {
|
||||
return baseArgs
|
||||
}
|
||||
return [...baseArgs, '--generation-profile', profile]
|
||||
}
|
||||
|
||||
const scenarios = [
|
||||
{ name: 'cli.help', args: ['--help'] },
|
||||
{ name: 'cli.version', args: ['--version'] },
|
||||
{ name: 'cli.doctor', args: ['doctor', '--cwd', 'src/cli/__fixtures__/tiny-fastify', '--quiet'] },
|
||||
{ name: 'cli.observe.check', args: ['observe', '--cwd', 'src/cli/__fixtures__/observe-config', '--profile', 'staging-observe', '--check-config', '--quiet'] },
|
||||
{ name: 'cli.qualify', args: ['qualify', '--cwd', 'src/cli/__fixtures__/protocol-lab', '--profile', 'oauth-nightly', '--seed', '42', '--quiet'] },
|
||||
...generationProfiles.map((profile) => ({
|
||||
name: `cli.qualify.profile[${profile}]`,
|
||||
args: withGenerationProfile(
|
||||
['qualify', '--cwd', 'src/cli/__fixtures__/protocol-lab', '--profile', 'oauth-nightly', '--seed', '42', '--quiet'],
|
||||
profile,
|
||||
),
|
||||
})),
|
||||
]
|
||||
|
||||
async function run() {
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
/**
|
||||
* APOPHIS configuration for broken-behavior fixture.
|
||||
*/
|
||||
|
||||
export default {
|
||||
mode: "verify",
|
||||
profiles: {
|
||||
@@ -14,6 +15,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -2,9 +2,11 @@
|
||||
* LEGACY APOPHIS configuration (old-style, for migration tests).
|
||||
* This uses deprecated field names that should be detected by `apophis migrate`.
|
||||
*/
|
||||
|
||||
export default {
|
||||
// Deprecated: 'mode' used to be 'testMode'
|
||||
testMode: "verify",
|
||||
|
||||
// Deprecated: 'profiles' used to be 'testProfiles'
|
||||
testProfiles: {
|
||||
quick: {
|
||||
@@ -15,6 +17,7 @@ export default {
|
||||
routeFilter: ["GET /legacy"],
|
||||
},
|
||||
},
|
||||
|
||||
// Deprecated: 'presets' used to be 'testPresets'
|
||||
testPresets: {
|
||||
"safe-ci": {
|
||||
@@ -25,6 +28,7 @@ export default {
|
||||
maxDuration: 5000,
|
||||
},
|
||||
},
|
||||
|
||||
// Deprecated: 'environments' used to be 'envPolicies'
|
||||
envPolicies: {
|
||||
local: {
|
||||
|
||||
@@ -2,6 +2,7 @@
|
||||
* Root-level APOPHIS config for monorepo.
|
||||
* Packages can override with their own configs.
|
||||
*/
|
||||
|
||||
export default {
|
||||
mode: "verify",
|
||||
profiles: {
|
||||
@@ -19,6 +20,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
/**
|
||||
* APOPHIS configuration for observe-config fixture.
|
||||
*/
|
||||
|
||||
export default {
|
||||
mode: "observe",
|
||||
profiles: {
|
||||
@@ -14,6 +15,7 @@ export default {
|
||||
presets: {
|
||||
"observe-safe": {
|
||||
name: "observe-safe",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
/**
|
||||
* APOPHIS configuration for protocol-lab fixture.
|
||||
*/
|
||||
|
||||
export default {
|
||||
mode: "qualify",
|
||||
profiles: {
|
||||
@@ -14,6 +15,7 @@ export default {
|
||||
presets: {
|
||||
deep: {
|
||||
name: "deep",
|
||||
depth: "deep",
|
||||
timeout: 30000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
/**
|
||||
* APOPHIS configuration for tiny-fastify fixture.
|
||||
*/
|
||||
|
||||
export default {
|
||||
mode: "verify",
|
||||
profiles: {
|
||||
@@ -14,6 +15,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -11,6 +11,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -11,6 +11,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -11,6 +11,7 @@ export default {
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
name: "safe-ci",
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
|
||||
@@ -116,8 +116,8 @@ export async function checkRouteDiscovery(options: RouteCheckOptions): Promise<R
|
||||
name: 'route-discovery',
|
||||
status: 'fail',
|
||||
message: `App file ${appFile} does not export a valid object.`,
|
||||
detail: 'Ensure the app file exports a Fastify instance or a factory function.',
|
||||
remediation: 'Export your Fastify instance: export default app; or export const createApp = () => app; or module.exports = app;',
|
||||
detail: 'Ensure the app file exports a Fastify instance as default.',
|
||||
remediation: 'Export your Fastify instance as default: export default app;',
|
||||
mode: 'all',
|
||||
};
|
||||
}
|
||||
|
||||
@@ -26,16 +26,6 @@ import { runDocsChecks } from './checks/docs.js';
|
||||
|
||||
import { renderJson } from '../../renderers/json.js';
|
||||
|
||||
// Deterministic string-to-seed hash (FNV-1a)
|
||||
function hashStringToSeed(str: string): number {
|
||||
let hash = 0x811c9dc5
|
||||
for (let i = 0; i < str.length; i++) {
|
||||
hash ^= str.charCodeAt(i)
|
||||
hash = Math.imul(hash, 0x01000193)
|
||||
}
|
||||
return Math.abs(hash >>> 0)
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Types
|
||||
// ---------------------------------------------------------------------------
|
||||
@@ -213,7 +203,7 @@ async function runPackageChecks(
|
||||
}
|
||||
|
||||
// 6. Determinism trust signal
|
||||
const testSeed = hashStringToSeed(packageName + cwd);
|
||||
const testSeed = Math.floor(Math.random() * 0x7fffffff);
|
||||
checks.push({
|
||||
name: 'determinism',
|
||||
status: 'pass',
|
||||
|
||||
@@ -17,6 +17,7 @@ export interface ScaffoldResult {
|
||||
export function safeCiScaffold(): ScaffoldResult {
|
||||
const preset: PresetDefinition = {
|
||||
name: 'safe-ci',
|
||||
depth: 'quick',
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
@@ -94,6 +95,7 @@ If \`apophis verify\` says "No behavioral contracts found", it means your routes
|
||||
export function platformObserveScaffold(): ScaffoldResult {
|
||||
const preset: PresetDefinition = {
|
||||
name: 'platform-observe',
|
||||
depth: 'standard',
|
||||
timeout: 10000,
|
||||
parallel: true,
|
||||
chaos: false,
|
||||
@@ -178,6 +180,7 @@ This project was scaffolded with \`apophis init --preset platform-observe\`.
|
||||
export function llmSafeScaffold(): ScaffoldResult {
|
||||
const preset: PresetDefinition = {
|
||||
name: 'llm-safe',
|
||||
depth: 'quick',
|
||||
timeout: 3000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
@@ -255,6 +258,7 @@ If \`apophis verify\` says "No behavioral contracts found", it means your routes
|
||||
export function protocolLabScaffold(): ScaffoldResult {
|
||||
const preset: PresetDefinition = {
|
||||
name: 'protocol-lab',
|
||||
depth: 'deep',
|
||||
timeout: 15000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
|
||||
@@ -19,7 +19,7 @@
|
||||
import type { CliContext } from '../../core/context.js'
|
||||
import { loadConfig } from '../../core/config-loader.js'
|
||||
import { PolicyEngine, detectEnvironment } from '../../core/policy-engine.js'
|
||||
|
||||
import { resolveGenerationProfileOverride, GenerationProfileResolutionError } from '../../core/generation-profile.js'
|
||||
import { SUCCESS, BEHAVIORAL_FAILURE, USAGE_ERROR, INTERNAL_ERROR } from '../../core/exit-codes.js'
|
||||
import type { CommandResult, Artifact, FailureRecord } from '../../core/types.js'
|
||||
import { classifyError, ErrorTaxonomy } from '../../core/error-taxonomy.js'
|
||||
@@ -54,6 +54,13 @@ function isReplayCompatibleRoute(route: string): boolean {
|
||||
return ROUTE_IDENTITY_PATTERN.test(route)
|
||||
}
|
||||
|
||||
function coerceDepth(value: unknown): TestConfig['depth'] {
|
||||
if (value === 'quick' || value === 'standard' || value === 'thorough') {
|
||||
return value
|
||||
}
|
||||
return 'standard'
|
||||
}
|
||||
|
||||
function coerceTimeout(value: unknown): number | undefined {
|
||||
return typeof value === 'number' ? value : undefined
|
||||
}
|
||||
@@ -64,6 +71,7 @@ function coerceTimeout(value: unknown): number | undefined {
|
||||
|
||||
export interface QualifyOptions {
|
||||
profile?: string
|
||||
generationProfile?: string
|
||||
seed?: number
|
||||
config?: string
|
||||
cwd?: string
|
||||
@@ -521,6 +529,7 @@ export async function qualifyCommand(
|
||||
): Promise<CommandResult> {
|
||||
const {
|
||||
profile,
|
||||
generationProfile,
|
||||
seed: explicitSeed,
|
||||
config: configPath,
|
||||
cwd,
|
||||
@@ -549,6 +558,7 @@ export async function qualifyCommand(
|
||||
}
|
||||
|
||||
const config = loadResult.config
|
||||
const resolvedGenerationProfile = resolveGenerationProfileOverride(generationProfile, config)
|
||||
|
||||
// 2. Run policy engine checks
|
||||
const policyEngine = new PolicyEngine({
|
||||
@@ -590,9 +600,12 @@ export async function qualifyCommand(
|
||||
// 6. Build stateful config
|
||||
const presetName = profileDef?.preset
|
||||
const preset = presetName ? config.presets?.[presetName] : undefined
|
||||
const presetDepth = coerceDepth((preset as { depth?: unknown } | undefined)?.depth)
|
||||
const presetTimeout = coerceTimeout((preset as { timeout?: unknown } | undefined)?.timeout)
|
||||
const statefulConfig: TestConfig | undefined = gates.stateful
|
||||
? {
|
||||
depth: presetDepth,
|
||||
generationProfile: resolvedGenerationProfile,
|
||||
seed,
|
||||
timeout: presetTimeout,
|
||||
routes: profileDef?.routes,
|
||||
@@ -629,7 +642,7 @@ export async function qualifyCommand(
|
||||
}
|
||||
return {
|
||||
exitCode: USAGE_ERROR,
|
||||
message: 'No Fastify app found. Ensure app.js exports a Fastify instance or a factory function.\n\nSupported patterns:\n export default app\n export const createApp = () => app\n module.exports = app',
|
||||
message: 'No Fastify app found. Ensure app.js exports a Fastify instance.',
|
||||
}
|
||||
}
|
||||
|
||||
@@ -739,6 +752,12 @@ export async function qualifyCommand(
|
||||
}
|
||||
}
|
||||
} catch (error) {
|
||||
if (error instanceof GenerationProfileResolutionError) {
|
||||
return {
|
||||
exitCode: USAGE_ERROR,
|
||||
message: error.message,
|
||||
}
|
||||
}
|
||||
const message = error instanceof Error ? error.message : String(error)
|
||||
return {
|
||||
exitCode: INTERNAL_ERROR,
|
||||
@@ -761,6 +780,7 @@ export async function handleQualify(
|
||||
): Promise<number> {
|
||||
const options: QualifyOptions = {
|
||||
profile: ctx.options.profile || undefined,
|
||||
generationProfile: ctx.options.generationProfile,
|
||||
seed: undefined,
|
||||
config: ctx.options.config || undefined,
|
||||
cwd: ctx.cwd,
|
||||
@@ -778,6 +798,11 @@ export async function handleQualify(
|
||||
}
|
||||
}
|
||||
|
||||
const generationProfileIdx = args.indexOf('--generation-profile')
|
||||
if (generationProfileIdx !== -1 && args[generationProfileIdx + 1]) {
|
||||
options.generationProfile = args[generationProfileIdx + 1]
|
||||
}
|
||||
|
||||
const result = await qualifyCommand(options, ctx)
|
||||
const format = options.format || ctx.options.format || 'human'
|
||||
const machineMode = format === 'json' || format === 'ndjson' || format === 'json-summary' || format === 'ndjson-summary'
|
||||
|
||||
@@ -11,22 +11,12 @@
|
||||
*/
|
||||
|
||||
import { runStatefulTests } from '../../../test/stateful-runner.js'
|
||||
import { CleanupManager } from '../../../infrastructure/cleanup-manager.js'
|
||||
import type {
|
||||
TestConfig,
|
||||
TestSuite,
|
||||
ScopeRegistry,
|
||||
} from '../../../types.js'
|
||||
import type { QualifyRunnerDeps, StepTrace } from './runner.js'
|
||||
|
||||
const minimalScopeRegistry: ScopeRegistry = {
|
||||
scopes: new Map(),
|
||||
defaultScope: { headers: {} },
|
||||
register() {},
|
||||
deriveFromRequest() { return { headers: {} } },
|
||||
getHeaders() { return {} },
|
||||
}
|
||||
|
||||
/**
|
||||
* Run stateful tests with the given config.
|
||||
* Wraps the existing stateful runner.
|
||||
@@ -37,15 +27,13 @@ export async function runStatefulWithTraces(
|
||||
): Promise<{ result: TestSuite; traces: StepTrace[] }> {
|
||||
const started = Date.now()
|
||||
|
||||
const cleanupManager = new CleanupManager(deps.fastify as any, minimalScopeRegistry, false)
|
||||
|
||||
const result = await runStatefulTests(
|
||||
deps.fastify,
|
||||
config,
|
||||
cleanupManager,
|
||||
minimalScopeRegistry,
|
||||
undefined, // cleanupManager — injected if needed by caller
|
||||
undefined, // scopeRegistry
|
||||
deps.extensionRegistry,
|
||||
undefined, // pluginContractRegistry — will be passed from runner when available
|
||||
undefined, // pluginContractRegistry
|
||||
undefined, // outboundContractRegistry
|
||||
)
|
||||
|
||||
|
||||
@@ -22,7 +22,7 @@
|
||||
import type { CliContext } from '../../core/context.js'
|
||||
import { loadConfig, findWorkspacePackages } from '../../core/config-loader.js'
|
||||
import { PolicyEngine, detectEnvironment } from '../../core/policy-engine.js'
|
||||
|
||||
import { resolveGenerationProfileOverride, GenerationProfileResolutionError } from '../../core/generation-profile.js'
|
||||
import { SUCCESS, BEHAVIORAL_FAILURE, USAGE_ERROR, INTERNAL_ERROR } from '../../core/exit-codes.js'
|
||||
import type { CommandResult, Artifact, FailureRecord, RouteResult, WorkspaceRun, WorkspaceResult } from '../../core/types.js'
|
||||
import { classifyError, ErrorTaxonomy } from '../../core/error-taxonomy.js'
|
||||
@@ -54,6 +54,7 @@ function isReplayCompatibleRoute(route: string): boolean {
|
||||
|
||||
export interface VerifyOptions {
|
||||
profile?: string
|
||||
generationProfile?: string
|
||||
routes?: string
|
||||
seed?: number
|
||||
changed?: boolean
|
||||
@@ -380,6 +381,7 @@ export async function verifyCommand(
|
||||
): Promise<CommandResult> {
|
||||
const {
|
||||
profile,
|
||||
generationProfile,
|
||||
routes: routesFlag,
|
||||
seed: explicitSeed,
|
||||
changed,
|
||||
@@ -410,6 +412,7 @@ export async function verifyCommand(
|
||||
}
|
||||
|
||||
const config = loadResult.config
|
||||
const resolvedGenerationProfile = resolveGenerationProfileOverride(generationProfile, config)
|
||||
|
||||
// 2a. Resolve profile — if explicitly requested but missing, list available ones
|
||||
if (profile && !config.profiles?.[profile]) {
|
||||
@@ -465,7 +468,7 @@ export async function verifyCommand(
|
||||
const errorMessage = err instanceof Error ? err.message : String(err)
|
||||
return {
|
||||
exitCode: USAGE_ERROR,
|
||||
message: `No Fastify app found. Ensure app.js exports a Fastify instance or a factory function.\n\nSupported patterns:\n export default app\n export const createApp = () => app\n module.exports = app\n\nError: ${errorMessage}\n\nNext:\n Run \`apophis init\` to scaffold a working app.js and config.`,
|
||||
message: `No Fastify app found. Ensure app.js exports a Fastify instance.\n\nError: ${errorMessage}\n\nNext:\n Run \`apophis init\` to scaffold a working app.js and config.`,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -473,6 +476,7 @@ export async function verifyCommand(
|
||||
const runResult = await runVerify({
|
||||
fastify: fastify as any,
|
||||
seed,
|
||||
generationProfile: resolvedGenerationProfile,
|
||||
timeout: typeof config.presets?.[loadResult.presetName || '']?.timeout === 'number'
|
||||
? (config.presets[loadResult.presetName || ''] as { timeout?: number }).timeout
|
||||
: undefined,
|
||||
@@ -548,7 +552,12 @@ export async function verifyCommand(
|
||||
message: `Config validation failed: ${message}`,
|
||||
}
|
||||
}
|
||||
|
||||
if (error instanceof GenerationProfileResolutionError) {
|
||||
return {
|
||||
exitCode: USAGE_ERROR,
|
||||
message,
|
||||
}
|
||||
}
|
||||
return {
|
||||
exitCode: INTERNAL_ERROR,
|
||||
message: `Internal error in verify command: ${message}`,
|
||||
@@ -570,6 +579,7 @@ export async function handleVerify(
|
||||
): Promise<number> {
|
||||
const options: VerifyOptions = {
|
||||
profile: ctx.options.profile || undefined,
|
||||
generationProfile: ctx.options.generationProfile,
|
||||
routes: undefined,
|
||||
seed: undefined,
|
||||
changed: false,
|
||||
@@ -601,6 +611,11 @@ export async function handleVerify(
|
||||
options.changed = true
|
||||
}
|
||||
|
||||
const generationProfileIdx = args.indexOf('--generation-profile')
|
||||
if (generationProfileIdx !== -1 && args[generationProfileIdx + 1]) {
|
||||
options.generationProfile = args[generationProfileIdx + 1]
|
||||
}
|
||||
|
||||
const workspaceMode = args.includes('--workspace')
|
||||
|
||||
if (workspaceMode) {
|
||||
|
||||
@@ -53,6 +53,7 @@ export interface VerifyRunResult {
|
||||
export interface VerifyRunnerDeps {
|
||||
fastify: FastifyInjectInstance
|
||||
seed: number
|
||||
generationProfile?: 'quick' | 'standard' | 'thorough'
|
||||
timeout?: number
|
||||
routeFilters?: string[]
|
||||
changed?: boolean
|
||||
|
||||
@@ -30,6 +30,7 @@ export interface Config {
|
||||
environments?: Record<string, EnvironmentPolicy>;
|
||||
profiles?: Record<string, ProfileDefinition>;
|
||||
presets?: Record<string, PresetDefinition>;
|
||||
generationProfiles?: Record<string, 'quick' | 'standard' | 'thorough' | { base: 'quick' | 'standard' | 'thorough' }>;
|
||||
[key: string]: unknown;
|
||||
}
|
||||
|
||||
@@ -106,6 +107,11 @@ const CONFIG_SCHEMA: Record<string, SchemaField> = {
|
||||
optional: true,
|
||||
properties: {},
|
||||
},
|
||||
generationProfiles: {
|
||||
type: 'object',
|
||||
optional: true,
|
||||
properties: {},
|
||||
},
|
||||
packs: {
|
||||
type: 'array',
|
||||
optional: true,
|
||||
@@ -145,6 +151,7 @@ const PROFILE_SCHEMA: Record<string, SchemaField> = {
|
||||
// Schema for PresetDefinition values (inside presets.<name>)
|
||||
const PRESET_SCHEMA: Record<string, SchemaField> = {
|
||||
name: { type: 'string', optional: false },
|
||||
depth: { type: 'string', optional: true, enumValues: ['quick', 'standard', 'deep'] },
|
||||
timeout: { type: 'number', optional: true, min: 0 },
|
||||
parallel: { type: 'boolean', optional: true },
|
||||
chaos: { type: 'boolean', optional: true },
|
||||
@@ -153,9 +160,12 @@ const PRESET_SCHEMA: Record<string, SchemaField> = {
|
||||
sampling: { type: 'number', optional: true },
|
||||
blocking: { type: 'boolean', optional: true },
|
||||
sinks: { type: 'object', optional: true },
|
||||
runs: { type: 'number', optional: true, min: 1 },
|
||||
};
|
||||
|
||||
const GENERATION_PROFILE_ALIAS_SCHEMA: Record<string, SchemaField> = {
|
||||
base: { type: 'string', optional: false, enumValues: ['quick', 'standard', 'thorough'] },
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Config discovery
|
||||
// ---------------------------------------------------------------------------
|
||||
@@ -249,6 +259,7 @@ function getDynamicContainerSchema(path: string): Record<string, SchemaField> |
|
||||
if (path === 'profiles') return PROFILE_SCHEMA;
|
||||
if (path === 'presets') return PRESET_SCHEMA;
|
||||
if (path === 'environments') return ENVIRONMENT_POLICY_SCHEMA;
|
||||
if (path === 'generationProfiles') return GENERATION_PROFILE_ALIAS_SCHEMA;
|
||||
return null;
|
||||
}
|
||||
|
||||
@@ -256,7 +267,7 @@ function getDynamicContainerSchema(path: string): Record<string, SchemaField> |
|
||||
* Check if a path is inside a dynamic container (e.g., profiles.foo, presets.bar).
|
||||
*/
|
||||
function isInsideDynamicContainer(path: string): boolean {
|
||||
return path.startsWith('profiles.') || path.startsWith('presets.') || path.startsWith('environments.');
|
||||
return path.startsWith('profiles.') || path.startsWith('presets.') || path.startsWith('environments.') || path.startsWith('generationProfiles.');
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -368,11 +379,18 @@ export function validateConfigAgainstSchema(
|
||||
|
||||
// Handle dynamic containers: profiles, presets, environments
|
||||
// The keys are user-defined names; their values have specific schemas
|
||||
const isDynamicContainer = path === 'profiles' || path === 'presets' || path === 'environments';
|
||||
const isDynamicContainer = path === 'profiles' || path === 'presets' || path === 'environments' || path === 'generationProfiles';
|
||||
if (!fieldSchema && isDynamicContainer) {
|
||||
const childSchema = getDynamicContainerSchema(path);
|
||||
const fieldValue = obj[key];
|
||||
if (childSchema && fieldValue !== null && typeof fieldValue === 'object') {
|
||||
if (path === 'generationProfiles' && typeof fieldValue === 'string') {
|
||||
validateType(
|
||||
fieldValue,
|
||||
{ type: 'string', optional: false, enumValues: ['quick', 'standard', 'thorough'] },
|
||||
currentPath,
|
||||
key,
|
||||
);
|
||||
} else if (childSchema && fieldValue !== null && typeof fieldValue === 'object') {
|
||||
// Validate the dynamic container value against its specific schema
|
||||
validateConfigAgainstSchema(fieldValue, childSchema, currentPath);
|
||||
} else if (childSchema) {
|
||||
@@ -615,6 +633,19 @@ export function validateConfigSemantics(config: Config): void {
|
||||
);
|
||||
}
|
||||
}
|
||||
if (preset.depth !== undefined) {
|
||||
const validDepths = ['quick', 'standard', 'deep'];
|
||||
const depthValue = preset.depth;
|
||||
if (typeof depthValue === 'string' && !validDepths.includes(depthValue as string)) {
|
||||
throw new ConfigValidationError(
|
||||
`Preset "${presetName}" has invalid depth: "${depthValue}"`,
|
||||
`presets.${presetName}.depth`,
|
||||
'depth',
|
||||
depthValue,
|
||||
`Must be one of: ${validDepths.join(', ')}.`,
|
||||
);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -101,6 +101,10 @@ export function createContext(options: Record<string, unknown> = {}): CliContext
|
||||
? options.color
|
||||
: 'auto';
|
||||
|
||||
const generationProfile = typeof options.generationProfile === 'string'
|
||||
? options.generationProfile
|
||||
: undefined;
|
||||
|
||||
return {
|
||||
cwd,
|
||||
env: {
|
||||
@@ -115,6 +119,7 @@ export function createContext(options: Record<string, unknown> = {}): CliContext
|
||||
options: {
|
||||
config: typeof options.config === 'string' ? options.config : undefined,
|
||||
profile: typeof options.profile === 'string' ? options.profile : undefined,
|
||||
generationProfile,
|
||||
format,
|
||||
color,
|
||||
quiet: options.quiet === true,
|
||||
|
||||
@@ -0,0 +1,51 @@
|
||||
import type { Config } from './config-loader.js'
|
||||
|
||||
export type ResolvedGenerationProfile = 'quick' | 'standard' | 'thorough'
|
||||
|
||||
export class GenerationProfileResolutionError extends Error {
|
||||
constructor(message: string) {
|
||||
super(message)
|
||||
this.name = 'GenerationProfileResolutionError'
|
||||
}
|
||||
}
|
||||
|
||||
function isBuiltInProfile(value: string): value is ResolvedGenerationProfile {
|
||||
return value === 'quick' || value === 'standard' || value === 'thorough'
|
||||
}
|
||||
|
||||
export function resolveGenerationProfileOverride(
|
||||
rawProfile: string | undefined,
|
||||
config: Config,
|
||||
): ResolvedGenerationProfile | undefined {
|
||||
if (!rawProfile) {
|
||||
return undefined
|
||||
}
|
||||
|
||||
if (isBuiltInProfile(rawProfile)) {
|
||||
return rawProfile
|
||||
}
|
||||
|
||||
const aliases = config.generationProfiles
|
||||
if (!aliases) {
|
||||
throw new GenerationProfileResolutionError(
|
||||
`Unknown generation profile "${rawProfile}". Use one of: quick, standard, thorough, or define an alias in config.generationProfiles.`,
|
||||
)
|
||||
}
|
||||
|
||||
const alias = aliases[rawProfile]
|
||||
if (!alias) {
|
||||
const available = Object.keys(aliases).join(', ') || 'none'
|
||||
throw new GenerationProfileResolutionError(
|
||||
`Unknown generation profile "${rawProfile}". Built-ins: quick, standard, thorough. Config aliases: ${available}.`,
|
||||
)
|
||||
}
|
||||
|
||||
const target = typeof alias === 'string' ? alias : alias.base
|
||||
if (!isBuiltInProfile(target)) {
|
||||
throw new GenerationProfileResolutionError(
|
||||
`Invalid generation profile alias "${rawProfile}". Alias must resolve to quick, standard, or thorough.`,
|
||||
)
|
||||
}
|
||||
|
||||
return target
|
||||
}
|
||||
@@ -22,6 +22,7 @@ const HELP_HEADER = `
|
||||
${pc.dim('Global Options:')}
|
||||
--config <path> Config file path
|
||||
--profile <name> Profile name from config
|
||||
--generation-profile <name> Generation budget profile (built-in or config alias)
|
||||
--cwd <path> Working directory override
|
||||
--format <mode> Output format: human | json | ndjson (default: human)
|
||||
--color <mode> Color mode: auto | always | never (default: auto)
|
||||
@@ -70,6 +71,7 @@ function getCommandHelp(command: string): string {
|
||||
|
||||
${pc.dim('Options:')}
|
||||
--profile <name> Profile name from config
|
||||
--generation-profile <name> Generation budget profile (built-in or config alias)
|
||||
--routes <filter> Route filter pattern
|
||||
--seed <number> Deterministic seed
|
||||
--changed Filter to git-modified routes
|
||||
@@ -101,6 +103,7 @@ function getCommandHelp(command: string): string {
|
||||
|
||||
${pc.dim('Options:')}
|
||||
--profile <name> Profile name from config
|
||||
--generation-profile <name> Generation budget profile (built-in or config alias)
|
||||
--seed <number> Deterministic seed
|
||||
|
||||
${pc.dim('Examples:')}
|
||||
@@ -222,6 +225,7 @@ export async function main(argv: string[] = process.argv.slice(2)): Promise<numb
|
||||
// Global flags
|
||||
cli.option('--config <path>', 'Config file path');
|
||||
cli.option('--profile <name>', 'Profile name from config');
|
||||
cli.option('--generation-profile <name>', 'Generation budget profile (built-in or config alias)');
|
||||
cli.option('--cwd <path>', 'Working directory override');
|
||||
cli.option('--format <mode>', 'Output format: human | json | ndjson', { default: 'human' });
|
||||
cli.option('--color <mode>', 'Color mode: auto | always | never', { default: 'auto' });
|
||||
@@ -266,6 +270,7 @@ export async function main(argv: string[] = process.argv.slice(2)): Promise<numb
|
||||
break;
|
||||
case 'verify':
|
||||
cmd.option('--profile <name>', 'Profile name from config');
|
||||
cmd.option('--generation-profile <name>', 'Generation budget profile (built-in or config alias)');
|
||||
cmd.option('--routes <filter>', 'Route filter pattern');
|
||||
cmd.option('--seed <number>', 'Deterministic seed');
|
||||
cmd.option('--changed', 'Filter to git-modified routes');
|
||||
@@ -276,6 +281,7 @@ export async function main(argv: string[] = process.argv.slice(2)): Promise<numb
|
||||
break;
|
||||
case 'qualify':
|
||||
cmd.option('--profile <name>', 'Profile name from config');
|
||||
cmd.option('--generation-profile <name>', 'Generation budget profile (built-in or config alias)');
|
||||
cmd.option('--seed <number>', 'Deterministic seed');
|
||||
break;
|
||||
case 'replay':
|
||||
@@ -367,15 +373,16 @@ export async function main(argv: string[] = process.argv.slice(2)): Promise<numb
|
||||
// Handle unknown flags
|
||||
const knownGlobalFlags = new Set([
|
||||
'--config', '--profile', '--cwd', '--format', '--color',
|
||||
'--generation-profile',
|
||||
'--quiet', '--verbose', '--artifact-dir', '--workspace',
|
||||
'-v', '--version', '-h', '--help',
|
||||
]);
|
||||
|
||||
const commandSpecificFlags: Record<string, Set<string>> = {
|
||||
init: new Set(['--preset', '--force', '--noninteractive']),
|
||||
verify: new Set(['--profile', '--routes', '--seed', '--changed', '--workspace']),
|
||||
verify: new Set(['--profile', '--generation-profile', '--routes', '--seed', '--changed', '--workspace']),
|
||||
observe: new Set(['--profile', '--check-config', '--workspace']),
|
||||
qualify: new Set(['--profile', '--seed', '--workspace']),
|
||||
qualify: new Set(['--profile', '--generation-profile', '--seed', '--workspace']),
|
||||
replay: new Set(['--artifact']),
|
||||
doctor: new Set(['--mode', '--strict', '--workspace']),
|
||||
migrate: new Set(['--check', '--dry-run', '--write']),
|
||||
|
||||
@@ -29,6 +29,7 @@ export interface CliContext {
|
||||
options: {
|
||||
config: string | undefined;
|
||||
profile: string | undefined;
|
||||
generationProfile?: string;
|
||||
format: OutputFormat;
|
||||
color: ColorMode;
|
||||
quiet: boolean;
|
||||
@@ -131,6 +132,7 @@ export interface ProfileDefinition {
|
||||
* required: ["name"],
|
||||
* properties: {
|
||||
* name: { type: "string" },
|
||||
* depth: { type: "string", enum: ["quick", "standard", "deep"] },
|
||||
* timeout: { type: "number" },
|
||||
* parallel: { type: "boolean" },
|
||||
* chaos: { type: "boolean" },
|
||||
@@ -141,6 +143,7 @@ export interface ProfileDefinition {
|
||||
*/
|
||||
export interface PresetDefinition {
|
||||
name: string;
|
||||
depth?: "quick" | "standard" | "deep";
|
||||
timeout?: number;
|
||||
parallel?: boolean;
|
||||
chaos?: boolean;
|
||||
|
||||
@@ -183,6 +183,29 @@ function generateNextSteps(failure: FailureRecord): string {
|
||||
// Progress and summary rendering
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
/**
|
||||
* Render progress for a running command.
|
||||
* Safe for CI (no spinners, just text updates).
|
||||
*/
|
||||
export function renderProgress(
|
||||
current: number,
|
||||
total: number,
|
||||
label: string,
|
||||
ctx: OutputContext,
|
||||
): string {
|
||||
const c = getColorizer(ctx);
|
||||
const pct = total > 0 ? Math.round((current / total) * 100) : 0;
|
||||
|
||||
if (ctx.isCI || !ctx.isTTY) {
|
||||
// CI mode: simple text, no spinner
|
||||
return `${label} [${current}/${total}] ${pct}%`;
|
||||
}
|
||||
|
||||
// TTY mode: with color
|
||||
const bar = renderProgressBar(current, total, 20, ctx);
|
||||
return `${c.dim(label)} ${bar} ${c.bold(`${pct}%`)}`;
|
||||
}
|
||||
|
||||
/**
|
||||
* Render a simple ASCII progress bar.
|
||||
*/
|
||||
|
||||
@@ -87,3 +87,9 @@ export const discoverRoutes = (instance: { routes?: Array<{ method: string; url:
|
||||
// Fastify 5 fallback: routes registered before plugin
|
||||
return discoverRoutesFallback(instance)
|
||||
}
|
||||
/**
|
||||
* Clear captured routes for an instance (useful for testing).
|
||||
*/
|
||||
export const clearCapturedRoutes = (instance: object): void => {
|
||||
capturedRoutes.delete(instance)
|
||||
}
|
||||
|
||||
@@ -143,6 +143,46 @@ export class PluginContractRegistry {
|
||||
}
|
||||
}
|
||||
// ============================================================================
|
||||
// Built-in Plugin Contracts
|
||||
// ============================================================================
|
||||
export const BUILTIN_PLUGIN_CONTRACTS: Record<string, PluginContractSpec> = {
|
||||
'@fastify/auth': {
|
||||
appliesTo: '**',
|
||||
hooks: {
|
||||
onRequest: {
|
||||
requires: ['request_headers(this).authorization != null'],
|
||||
},
|
||||
},
|
||||
},
|
||||
'@fastify/compress': {
|
||||
appliesTo: '**',
|
||||
hooks: {
|
||||
onSend: {
|
||||
ensures: ['response_headers(this).content-encoding != null'],
|
||||
},
|
||||
},
|
||||
},
|
||||
'@fastify/cors': {
|
||||
appliesTo: '**',
|
||||
hooks: {
|
||||
onRequest: {
|
||||
ensures: ['response_headers(this).access-control-allow-origin != null'],
|
||||
},
|
||||
},
|
||||
},
|
||||
'@fastify/rate-limit': {
|
||||
appliesTo: '**',
|
||||
hooks: {
|
||||
onRequest: {
|
||||
ensures: [
|
||||
'response_headers(this).x-ratelimit-limit != null',
|
||||
'response_headers(this).x-ratelimit-remaining != null',
|
||||
],
|
||||
},
|
||||
},
|
||||
},
|
||||
}
|
||||
// ============================================================================
|
||||
// Factory
|
||||
// ============================================================================
|
||||
export function createPluginContractRegistry(): PluginContractRegistry {
|
||||
|
||||
@@ -7,9 +7,13 @@ import type { Arbitrary } from 'fast-check'
|
||||
import * as fc from 'fast-check'
|
||||
import { CONTENT_TYPE } from '../infrastructure/http-executor.js'
|
||||
|
||||
export type GenerationProfile = 'quick' | 'standard' | 'thorough'
|
||||
|
||||
export interface SchemaToArbOptions {
|
||||
/** 'request' skips readOnly, 'response' skips writeOnly */
|
||||
readonly context: 'request' | 'response'
|
||||
/** Generation budget profile: quick favors speed, thorough favors breadth */
|
||||
readonly generationProfile?: GenerationProfile
|
||||
}
|
||||
|
||||
interface ContextCache {
|
||||
@@ -25,32 +29,45 @@ const patternRegexCache = new Map<string, RegExp>()
|
||||
const STABLE_SCHEMA_CACHE_LIMIT = 512
|
||||
const PATTERN_REGEX_CACHE_LIMIT = 256
|
||||
|
||||
// Fixed defaults for generated data size (previously controlled by generationProfile tier)
|
||||
const DEFAULT_STRING_MAX = 128
|
||||
const DEFAULT_ARRAY_MAX = 10
|
||||
const DEFAULT_ADDITIONAL_PROPS_MAX = 6
|
||||
|
||||
function defaultStringMaxLength(): number | undefined {
|
||||
return DEFAULT_STRING_MAX
|
||||
function normalizeProfile(profile: GenerationProfile | undefined): GenerationProfile {
|
||||
return profile ?? 'standard'
|
||||
}
|
||||
|
||||
function defaultArrayMaxLength(): number | undefined {
|
||||
return DEFAULT_ARRAY_MAX
|
||||
function defaultStringMaxLength(profile: GenerationProfile): number | undefined {
|
||||
if (profile === 'quick') return 48
|
||||
if (profile === 'standard') return 128
|
||||
return undefined
|
||||
}
|
||||
|
||||
function additionalPropsMaxKeys(): number {
|
||||
return DEFAULT_ADDITIONAL_PROPS_MAX
|
||||
function defaultArrayMaxLength(profile: GenerationProfile): number | undefined {
|
||||
if (profile === 'quick') return 4
|
||||
if (profile === 'standard') return 10
|
||||
return undefined
|
||||
}
|
||||
|
||||
function buildFallbackAnyArb(): Arbitrary<unknown> {
|
||||
function additionalPropsMaxKeys(profile: GenerationProfile): number {
|
||||
if (profile === 'quick') return 3
|
||||
if (profile === 'standard') return 6
|
||||
return 10
|
||||
}
|
||||
|
||||
function buildFallbackAnyArb(profile: GenerationProfile): Arbitrary<unknown> {
|
||||
if (profile === 'thorough') {
|
||||
return fc.anything()
|
||||
}
|
||||
|
||||
const stringMax = profile === 'quick' ? 24 : 64
|
||||
const arrayMax = profile === 'quick' ? 3 : 6
|
||||
const dictMax = profile === 'quick' ? 2 : 4
|
||||
|
||||
return fc.oneof(
|
||||
fc.constant(null),
|
||||
fc.boolean(),
|
||||
fc.integer(),
|
||||
fc.double({ noNaN: true }),
|
||||
fc.string({ maxLength: 64 }),
|
||||
fc.array(fc.string({ maxLength: 16 }), { maxLength: 6 }),
|
||||
fc.dictionary(fc.string({ maxLength: 16 }), fc.string({ maxLength: 24 }), { maxKeys: 4 }),
|
||||
fc.string({ maxLength: stringMax }),
|
||||
fc.array(fc.string({ maxLength: 16 }), { maxLength: arrayMax }),
|
||||
fc.dictionary(fc.string({ maxLength: 16 }), fc.string({ maxLength: 24 }), { maxKeys: dictMax }),
|
||||
)
|
||||
}
|
||||
|
||||
@@ -89,6 +106,7 @@ const getObject = (schema: unknown, key: string): Record<string, unknown> | unde
|
||||
|
||||
const buildStringArb = (
|
||||
schema: Record<string, unknown>,
|
||||
profile: GenerationProfile,
|
||||
): Arbitrary<string> => {
|
||||
const minLength = getNumber(schema, 'minLength')
|
||||
const maxLength = getNumber(schema, 'maxLength')
|
||||
@@ -131,7 +149,7 @@ const buildStringArb = (
|
||||
if (minLength !== undefined) constraints.minLength = minLength
|
||||
if (maxLength !== undefined) constraints.maxLength = maxLength
|
||||
else {
|
||||
const capped = defaultStringMaxLength()
|
||||
const capped = defaultStringMaxLength(profile)
|
||||
if (capped !== undefined) constraints.maxLength = capped
|
||||
}
|
||||
|
||||
@@ -280,13 +298,14 @@ function getSchemaFingerprint(schema: Record<string, unknown>): string | undefin
|
||||
function getStableCachedArbitrary(
|
||||
schema: Record<string, unknown>,
|
||||
context: SchemaToArbOptions['context'],
|
||||
profile: GenerationProfile,
|
||||
): Arbitrary<unknown> | undefined {
|
||||
const fingerprint = getSchemaFingerprint(schema)
|
||||
if (!fingerprint) {
|
||||
return undefined
|
||||
}
|
||||
|
||||
const key = `${context}:${fingerprint}`
|
||||
const key = `${context}:${profile}:${fingerprint}`
|
||||
const cached = stableSchemaArbitraryCache.get(key)
|
||||
if (!cached) {
|
||||
return undefined
|
||||
@@ -300,6 +319,7 @@ function getStableCachedArbitrary(
|
||||
function setStableCachedArbitrary(
|
||||
schema: Record<string, unknown>,
|
||||
context: SchemaToArbOptions['context'],
|
||||
profile: GenerationProfile,
|
||||
arbitrary: Arbitrary<unknown>,
|
||||
): void {
|
||||
const fingerprint = getSchemaFingerprint(schema)
|
||||
@@ -307,7 +327,7 @@ function setStableCachedArbitrary(
|
||||
return
|
||||
}
|
||||
|
||||
const key = `${context}:${fingerprint}`
|
||||
const key = `${context}:${profile}:${fingerprint}`
|
||||
if (stableSchemaArbitraryCache.has(key)) {
|
||||
stableSchemaArbitraryCache.delete(key)
|
||||
}
|
||||
@@ -333,18 +353,19 @@ const buildIntegerArb = (schema: Record<string, unknown>): Arbitrary<number> =>
|
||||
const buildArrayArb = (
|
||||
schema: Record<string, unknown>,
|
||||
options: SchemaToArbOptions,
|
||||
profile: GenerationProfile,
|
||||
): Arbitrary<unknown[]> => {
|
||||
const itemsSchema = getObject(schema, 'items')
|
||||
const itemArb = itemsSchema !== undefined
|
||||
? convertSchemaInternal(itemsSchema, options, false)
|
||||
: buildFallbackAnyArb()
|
||||
: buildFallbackAnyArb(profile)
|
||||
const minItems = getNumber(schema, 'minItems')
|
||||
const maxItems = getNumber(schema, 'maxItems')
|
||||
const constraints: { minLength?: number; maxLength?: number } = {}
|
||||
if (minItems !== undefined) constraints.minLength = minItems
|
||||
if (maxItems !== undefined) constraints.maxLength = maxItems
|
||||
else {
|
||||
const capped = defaultArrayMaxLength()
|
||||
const capped = defaultArrayMaxLength(profile)
|
||||
if (capped !== undefined) constraints.maxLength = capped
|
||||
}
|
||||
return fc.array(itemArb, constraints)
|
||||
@@ -353,6 +374,7 @@ const buildArrayArb = (
|
||||
const buildObjectArb = (
|
||||
schema: Record<string, unknown>,
|
||||
options: SchemaToArbOptions,
|
||||
profile: GenerationProfile,
|
||||
): Arbitrary<Record<string, unknown>> => {
|
||||
const properties = getObject(schema, 'properties') ?? {}
|
||||
const required = new Set(getArray(schema, 'required') as string[] ?? [])
|
||||
@@ -376,14 +398,14 @@ const buildObjectArb = (
|
||||
const baseArb = fc.record(arbs)
|
||||
|
||||
if (additionalProperties === true) {
|
||||
const extraValueArb = buildFallbackAnyArb()
|
||||
const keyMaxLength = 32
|
||||
const extraValueArb = buildFallbackAnyArb(profile)
|
||||
const keyMaxLength = profile === 'quick' ? 16 : 32
|
||||
return fc.tuple(
|
||||
baseArb,
|
||||
fc.dictionary(
|
||||
fc.string({ maxLength: keyMaxLength }),
|
||||
extraValueArb,
|
||||
{ maxKeys: additionalPropsMaxKeys() },
|
||||
{ maxKeys: additionalPropsMaxKeys(profile) },
|
||||
),
|
||||
).map(([base, extra]) => ({
|
||||
...base,
|
||||
@@ -396,6 +418,7 @@ const buildObjectArb = (
|
||||
|
||||
const buildMultipartArb = (
|
||||
schema: Record<string, unknown>,
|
||||
profile: GenerationProfile,
|
||||
): Arbitrary<{ fields: Record<string, unknown>; files: Record<string, { originalname: string; mimetype: string; size: number; buffer: Buffer } | { originalname: string; mimetype: string; size: number; buffer: Buffer }[]> }> => {
|
||||
const fieldsSchema = getObject(schema, 'x-multipart-fields') ?? {}
|
||||
const filesSchema = getObject(schema, 'x-multipart-files') ?? {}
|
||||
@@ -403,7 +426,7 @@ const buildMultipartArb = (
|
||||
const fieldArbs: Record<string, Arbitrary<unknown>> = {}
|
||||
for (const [key, propSchema] of Object.entries(fieldsSchema)) {
|
||||
if (isObject(propSchema)) {
|
||||
fieldArbs[key] = convertSchemaInternal(propSchema, { context: 'request' }, false)
|
||||
fieldArbs[key] = convertSchemaInternal(propSchema, { context: 'request', generationProfile: profile }, false)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -440,6 +463,7 @@ const convertSchemaInternal = (
|
||||
options: SchemaToArbOptions,
|
||||
useStableCache: boolean,
|
||||
): Arbitrary<unknown> => {
|
||||
const profile = normalizeProfile(options.generationProfile)
|
||||
const cacheKey = options.context
|
||||
const cachedBySchema = schemaArbitraryCache.get(schema)
|
||||
const cached = cachedBySchema?.[cacheKey]
|
||||
@@ -448,7 +472,7 @@ const convertSchemaInternal = (
|
||||
}
|
||||
|
||||
if (useStableCache) {
|
||||
const stableCached = getStableCachedArbitrary(schema, cacheKey)
|
||||
const stableCached = getStableCachedArbitrary(schema, cacheKey, profile)
|
||||
if (stableCached) {
|
||||
const contextCache = cachedBySchema ?? {}
|
||||
contextCache[cacheKey] = stableCached
|
||||
@@ -465,11 +489,11 @@ const convertSchemaInternal = (
|
||||
let arb: Arbitrary<unknown>
|
||||
|
||||
if (contentType === CONTENT_TYPE.MULTIPART) {
|
||||
arb = buildMultipartArb(schema)
|
||||
arb = buildMultipartArb(schema, profile)
|
||||
} else if (enumValues !== undefined && enumValues.length > 0) {
|
||||
arb = fc.constantFrom(...enumValues)
|
||||
} else if (type === 'string') {
|
||||
arb = buildStringArb(schema)
|
||||
arb = buildStringArb(schema, profile)
|
||||
} else if (type === 'integer') {
|
||||
arb = buildIntegerArb(schema)
|
||||
} else if (type === 'number') {
|
||||
@@ -477,11 +501,11 @@ const convertSchemaInternal = (
|
||||
} else if (type === 'boolean') {
|
||||
arb = fc.boolean()
|
||||
} else if (type === 'array') {
|
||||
arb = buildArrayArb(schema, options)
|
||||
arb = buildArrayArb(schema, options, profile)
|
||||
} else if (type === 'object') {
|
||||
arb = buildObjectArb(schema, options)
|
||||
arb = buildObjectArb(schema, options, profile)
|
||||
} else {
|
||||
arb = buildFallbackAnyArb()
|
||||
arb = buildFallbackAnyArb(profile)
|
||||
}
|
||||
|
||||
if (nullable === true) {
|
||||
@@ -492,7 +516,7 @@ const convertSchemaInternal = (
|
||||
contextCache[cacheKey] = arb
|
||||
schemaArbitraryCache.set(schema, contextCache)
|
||||
if (useStableCache) {
|
||||
setStableCachedArbitrary(schema, cacheKey, arb)
|
||||
setStableCachedArbitrary(schema, cacheKey, profile, arb)
|
||||
}
|
||||
|
||||
return arb
|
||||
|
||||
@@ -220,6 +220,7 @@ function setNestedValue(obj: Record<string, unknown>, path: string, value: unkno
|
||||
function createConditionalDependencyArbitrary(
|
||||
contract: ResolvedOutboundContract,
|
||||
request: Record<string, unknown>,
|
||||
generationProfile: 'quick' | 'standard' | 'thorough',
|
||||
): Arbitrary<DependencyResponseSample> {
|
||||
const statuses = Object.keys(contract.response).map(Number)
|
||||
if (statuses.length === 0) {
|
||||
@@ -228,7 +229,7 @@ function createConditionalDependencyArbitrary(
|
||||
return fc.integer({ min: 0, max: statuses.length - 1 }).chain((statusIndex) => {
|
||||
const statusCode = statuses[statusIndex]!
|
||||
const schema = contract.response[statusCode]
|
||||
const bodyArb = convertSchema(schema ?? {}, { context: 'response' })
|
||||
const bodyArb = convertSchema(schema ?? {}, { context: 'response', generationProfile })
|
||||
return bodyArb.map((rawBody) => ({
|
||||
contractName: contract.name,
|
||||
statusCode,
|
||||
@@ -238,10 +239,11 @@ function createConditionalDependencyArbitrary(
|
||||
}
|
||||
function createRequestArbitrary(
|
||||
route: RouteContract,
|
||||
generationProfile: 'quick' | 'standard' | 'thorough',
|
||||
): Arbitrary<Record<string, unknown>> {
|
||||
const bodySchema = route.schema?.body as Record<string, unknown> | undefined
|
||||
const bodyArb = bodySchema !== undefined
|
||||
? convertSchema(bodySchema, { context: 'request' })
|
||||
? convertSchema(bodySchema, { context: 'request', generationProfile })
|
||||
: fc.constant({})
|
||||
const pathParams = route.path.match(/:([a-zA-Z_][a-zA-Z0-9_]*)/g) ?? []
|
||||
const pathParamArbs: Record<string, fc.Arbitrary<string>> = {}
|
||||
@@ -262,10 +264,11 @@ function createRequestArbitrary(
|
||||
function createConditionalDependenciesArbitrary(
|
||||
contracts: ResolvedOutboundContract[],
|
||||
request: Record<string, unknown>,
|
||||
generationProfile: 'quick' | 'standard' | 'thorough',
|
||||
): Arbitrary<ReadonlyArray<DependencyResponseSample>> {
|
||||
if (contracts.length === 0) return fc.constant([])
|
||||
const arbs = contracts.map((contract) =>
|
||||
createConditionalDependencyArbitrary(contract, request)
|
||||
createConditionalDependencyArbitrary(contract, request, generationProfile)
|
||||
)
|
||||
return fc.tuple(...arbs)
|
||||
}
|
||||
@@ -289,10 +292,11 @@ export function createTripleBoundaryArbitrary(
|
||||
route: RouteContract,
|
||||
contracts: ResolvedOutboundContract[],
|
||||
chaosConfig: ChaosConfig,
|
||||
generationProfile: 'quick' | 'standard' | 'thorough' = 'standard',
|
||||
): Arbitrary<TripleBoundaryCommand> {
|
||||
const requestArb = createRequestArbitrary(route)
|
||||
const requestArb = createRequestArbitrary(route, generationProfile)
|
||||
return requestArb.chain((request) => {
|
||||
const depArb = createConditionalDependenciesArbitrary(contracts, request)
|
||||
const depArb = createConditionalDependenciesArbitrary(contracts, request, generationProfile)
|
||||
const chaosArb = createChaosEventArbitrary(route, contracts, chaosConfig)
|
||||
return fc.tuple(depArb, chaosArb).map(([dependencyResponses, chaosEvents]) => ({
|
||||
route,
|
||||
@@ -356,3 +360,47 @@ export function applyChaosToAllResponses(
|
||||
)
|
||||
})
|
||||
}
|
||||
// ============================================================================
|
||||
// Formatting
|
||||
// ============================================================================
|
||||
export function formatTripleBoundaryCounterexample(result: TripleBoundaryResult): string {
|
||||
const lines: string[] = []
|
||||
lines.push('Triple-boundary counterexample:')
|
||||
lines.push('')
|
||||
lines.push(`Route: ${result.command.route.method} ${result.command.route.path}`)
|
||||
lines.push('')
|
||||
lines.push('Request:')
|
||||
lines.push(JSON.stringify(result.command.request, null, 2))
|
||||
lines.push('')
|
||||
if (result.command.dependencyResponses.length > 0) {
|
||||
lines.push('Dependency responses:')
|
||||
for (const dep of result.command.dependencyResponses) {
|
||||
lines.push(` ${dep.contractName}: ${dep.statusCode}`)
|
||||
lines.push(` ${JSON.stringify(dep.body)}`)
|
||||
}
|
||||
lines.push('')
|
||||
}
|
||||
if (result.command.chaosEvents.length > 0) {
|
||||
lines.push('Chaos events:')
|
||||
for (const chaos of result.command.chaosEvents) {
|
||||
if (chaos.type === 'none') continue
|
||||
lines.push(` ${chaos.type}`)
|
||||
if (chaos.contractName) lines.push(` Target: ${chaos.contractName}`)
|
||||
if (chaos.delayMs) lines.push(` Delay: ${chaos.delayMs}ms`)
|
||||
if (chaos.statusCode) lines.push(` Status: ${chaos.statusCode}`)
|
||||
if (chaos.corruptionStrategy) lines.push(` Corruption: ${chaos.corruptionStrategy}`)
|
||||
if (chaos.corruptionField) lines.push(` Field: ${chaos.corruptionField}`)
|
||||
}
|
||||
lines.push('')
|
||||
}
|
||||
if (result.failureBoundary) {
|
||||
lines.push(`Failure boundary: ${result.failureBoundary}`)
|
||||
}
|
||||
if (result.failureDescription) {
|
||||
lines.push(`Description: ${result.failureDescription}`)
|
||||
}
|
||||
if (result.error) {
|
||||
lines.push(`Error: ${result.error}`)
|
||||
}
|
||||
return lines.join('\n')
|
||||
}
|
||||
|
||||
@@ -285,7 +285,7 @@ function evaluateQuantified(
|
||||
bodyFn: (item: unknown) => boolean
|
||||
): boolean {
|
||||
if (!Array.isArray(collection)) {
|
||||
return false
|
||||
throw new Error(`Quantified expression requires an array collection, got: ${typeof collection}`)
|
||||
}
|
||||
if (quantifier === 'for') {
|
||||
return collection.every(bodyFn)
|
||||
@@ -299,7 +299,7 @@ async function evaluateQuantifiedAsync(
|
||||
bodyFn: (item: unknown) => Promise<boolean>
|
||||
): Promise<boolean> {
|
||||
if (!Array.isArray(collection)) {
|
||||
return false
|
||||
throw new Error(`Quantified expression requires an array collection, got: ${typeof collection}`)
|
||||
}
|
||||
if (quantifier === 'for') {
|
||||
for (const item of collection) {
|
||||
|
||||
+1
-20
@@ -13,33 +13,14 @@ export default fp(apophisPlugin, {
|
||||
|
||||
export * from './types.js'
|
||||
|
||||
// Quality engines
|
||||
// Chaos-v3: Pure chaos application for property-based testing
|
||||
export {
|
||||
applyChaosToExecution,
|
||||
applyChaosToAllResponses,
|
||||
createChaosEventArbitrary,
|
||||
extractDelays,
|
||||
sleep,
|
||||
hasAppliedChaos,
|
||||
formatChaosEvents,
|
||||
type ChaosEvent,
|
||||
type ChaosEventType,
|
||||
type ChaosApplicationResult,
|
||||
} from './quality/chaos-v3.js'
|
||||
|
||||
export {
|
||||
FlakeDetector,
|
||||
type FlakeReport,
|
||||
type FlakeRerun,
|
||||
type FlakeOptions,
|
||||
} from './quality/flake.js'
|
||||
|
||||
export {
|
||||
runMutationTesting,
|
||||
testMutation,
|
||||
type Mutation,
|
||||
type MutationType,
|
||||
type MutationResult,
|
||||
type MutationReport,
|
||||
type MutationConfig,
|
||||
} from './quality/mutation.js'
|
||||
|
||||
@@ -29,6 +29,7 @@ export interface OutboundMockRuntime {
|
||||
interface OutboundMockOptions {
|
||||
readonly contracts: ResolvedOutboundContract[]
|
||||
readonly mode: 'example' | 'property'
|
||||
readonly generationProfile?: 'quick' | 'standard' | 'thorough'
|
||||
readonly overrides?: Record<string, {
|
||||
readonly forceStatus?: number
|
||||
readonly headers?: Record<string, string>
|
||||
@@ -76,7 +77,7 @@ export function createOutboundMockRuntime(opts: OutboundMockOptions): OutboundMo
|
||||
const schema = contract.response[statusCode]
|
||||
if (!schema) return null
|
||||
// Generate base response from schema
|
||||
const arb = convertSchema(schema, { context: 'response' })
|
||||
const arb = convertSchema(schema, { context: 'response', generationProfile: opts.generationProfile })
|
||||
const samples = fc.sample(arb, { numRuns: 1, seed: opts.seed + calls.length })
|
||||
let body = samples[0] ?? null
|
||||
if (typeof body !== 'object' || body === null) return body
|
||||
|
||||
@@ -27,7 +27,7 @@ import { parse } from '../formula/parser.js'
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
const normalizeTestConfig = (opts: TestConfig = {}): TestConfig => ({
|
||||
runs: opts.runs,
|
||||
depth: opts.depth ?? 'standard',
|
||||
scope: opts.scope,
|
||||
seed: opts.seed,
|
||||
timeout: opts.timeout,
|
||||
|
||||
+5
-1
@@ -13,7 +13,7 @@ import { registerValidationHooks, storeRouteContract } from '../infrastructure/h
|
||||
import { extractContract } from '../domain/contract.js'
|
||||
import { createExtensionRegistry } from '../extension/registry.js'
|
||||
import type { ApophisExtension } from '../extension/types.js'
|
||||
import { createPluginContractRegistry } from '../domain/plugin-contracts.js'
|
||||
import { createPluginContractRegistry, BUILTIN_PLUGIN_CONTRACTS } from '../domain/plugin-contracts.js'
|
||||
import type { PluginContractRegistry } from '../domain/plugin-contracts.js'
|
||||
import { OutboundContractRegistry } from '../domain/outbound-contracts.js'
|
||||
import { createOutboundMockRuntime, type OutboundMockRuntime } from '../infrastructure/outbound-mock-runtime.js'
|
||||
@@ -75,6 +75,10 @@ export const apophisPlugin = async (fastify: FastifyInstance, opts: ApophisOptio
|
||||
// Initialize scope registry with explicit config or empty
|
||||
const scope = new ScopeRegistry(opts.scopes ?? {})
|
||||
const cleanupManager = new CleanupManager(fastify, scope, opts.cleanup ?? false)
|
||||
// Register built-in plugin contracts
|
||||
for (const [name, spec] of Object.entries(BUILTIN_PLUGIN_CONTRACTS)) {
|
||||
pluginContractRegistry.register(name, spec)
|
||||
}
|
||||
// Register user-provided plugin contracts
|
||||
if (opts.pluginContracts) {
|
||||
for (const [name, spec] of Object.entries(opts.pluginContracts)) {
|
||||
|
||||
@@ -80,6 +80,7 @@ export function oauth21ProfilePack(opts: PackOptions = {}): Partial<Config> {
|
||||
presets: {
|
||||
'protocol-lab': {
|
||||
name: 'protocol-lab',
|
||||
depth: 'deep',
|
||||
timeout: opts.timeout ?? 15000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
@@ -87,6 +88,7 @@ export function oauth21ProfilePack(opts: PackOptions = {}): Partial<Config> {
|
||||
},
|
||||
'safe-ci': {
|
||||
name: 'safe-ci',
|
||||
depth: 'quick',
|
||||
timeout: opts.timeout ?? 5000,
|
||||
parallel: false,
|
||||
chaos: false,
|
||||
@@ -116,6 +118,7 @@ export function rfc8628DeviceAuthorizationPack(opts: PackOptions = {}): Partial<
|
||||
presets: {
|
||||
'protocol-lab': {
|
||||
name: 'protocol-lab',
|
||||
depth: 'deep',
|
||||
timeout: opts.timeout ?? 20000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
@@ -145,6 +148,7 @@ export function rfc8693TokenExchangePack(opts: PackOptions = {}): Partial<Config
|
||||
presets: {
|
||||
'protocol-lab': {
|
||||
name: 'protocol-lab',
|
||||
depth: 'deep',
|
||||
timeout: opts.timeout ?? 15000,
|
||||
parallel: false,
|
||||
chaos: true,
|
||||
|
||||
+14
-20
@@ -7,7 +7,7 @@
|
||||
* If the test suite passes, the mutation "survives" — indicating a gap in coverage.
|
||||
*
|
||||
* Usage:
|
||||
* const report = await runMutationTesting(fastify, { runs: 10 })
|
||||
* const report = await runMutationTesting(fastify, { depth: 'quick' })
|
||||
* console.log(`Mutation score: ${report.score}%`)
|
||||
*/
|
||||
import type { FastifyInstance } from 'fastify'
|
||||
@@ -44,7 +44,7 @@ export interface MutationReport {
|
||||
readonly weakContracts: string[] // contracts that survived all mutations
|
||||
}
|
||||
export interface MutationConfig {
|
||||
readonly runs?: number
|
||||
readonly depth?: TestConfig['depth']
|
||||
readonly seed?: number
|
||||
/** Max mutations per contract (default: 5) */
|
||||
readonly maxMutationsPerContract?: number
|
||||
@@ -214,7 +214,7 @@ export async function runMutationTesting(
|
||||
const suite = await runPetitTestsWithMutation(
|
||||
fastify as unknown as FastifyInjectInstance,
|
||||
{
|
||||
runs: config.runs ?? 10,
|
||||
depth: config.depth ?? 'quick',
|
||||
seed: config.seed,
|
||||
},
|
||||
mutatedContract
|
||||
@@ -256,26 +256,20 @@ export async function runMutationTesting(
|
||||
}
|
||||
/**
|
||||
* Run petit tests with a mutated contract.
|
||||
* Injects the mutated contract so the runner uses it instead of discovering from Fastify.
|
||||
* This is a simplified version that tests a single mutated contract.
|
||||
*/
|
||||
async function runPetitTestsWithMutation(
|
||||
fastify: FastifyInjectInstance,
|
||||
config: { runs?: number; seed?: number },
|
||||
config: { depth?: TestConfig['depth']; seed?: number },
|
||||
mutatedContract: RouteContract
|
||||
): Promise<TestSuite> {
|
||||
return runPetitTests(
|
||||
fastify,
|
||||
{
|
||||
runs: config.runs ?? 10,
|
||||
seed: config.seed,
|
||||
routes: [`${mutatedContract.method} ${mutatedContract.path}`],
|
||||
},
|
||||
undefined,
|
||||
undefined,
|
||||
undefined,
|
||||
undefined,
|
||||
[mutatedContract]
|
||||
)
|
||||
// For now, run the full suite - the mutated contract will be discovered
|
||||
// In a real implementation, you'd inject the mutated contract into the discovery
|
||||
return runPetitTests(fastify, {
|
||||
depth: config.depth ?? 'quick',
|
||||
seed: config.seed,
|
||||
routes: [`${mutatedContract.method} ${mutatedContract.path}`],
|
||||
})
|
||||
}
|
||||
/**
|
||||
* Quick mutation test for a single contract formula.
|
||||
@@ -285,14 +279,14 @@ export async function testMutation(
|
||||
fastify: FastifyInstance,
|
||||
contract: RouteContract,
|
||||
mutation: Mutation,
|
||||
config: Pick<MutationConfig, 'runs' | 'seed'> = {}
|
||||
config: Pick<MutationConfig, 'depth' | 'seed'> = {}
|
||||
): Promise<boolean> {
|
||||
const mutatedContract = applyMutation(contract, mutation)
|
||||
try {
|
||||
const suite = await runPetitTestsWithMutation(
|
||||
fastify as unknown as FastifyInjectInstance,
|
||||
{
|
||||
runs: config.runs ?? 10,
|
||||
depth: config.depth ?? 'quick',
|
||||
seed: config.seed,
|
||||
},
|
||||
mutatedContract
|
||||
|
||||
@@ -57,7 +57,7 @@ async function expectLoadConfigError(
|
||||
test('schema: accepts minimal valid configs', () => {
|
||||
validateConfigAgainstSchema({}, CONFIG_SCHEMA);
|
||||
validateConfigAgainstSchema({ mode: 'verify', routes: ['GET /users'], seed: 42 }, CONFIG_SCHEMA);
|
||||
validateConfigAgainstSchema({ presets: { quick: { runs: 10, timeout: 5000 } } }, CONFIG_SCHEMA);
|
||||
validateConfigAgainstSchema({ presets: { quick: { depth: 'quick', timeout: 5000 } } }, CONFIG_SCHEMA);
|
||||
});
|
||||
|
||||
test('schema: rejects unknown keys with guidance', () => {
|
||||
@@ -131,7 +131,11 @@ test('schema: rejects enum and numeric range violations with clear guidance', ()
|
||||
path: 'profiles.default.mode',
|
||||
expectedGuidance: ['verify', 'observe', 'qualify'],
|
||||
},
|
||||
|
||||
{
|
||||
value: { presets: { quick: { depth: 'super-deep' } } },
|
||||
path: 'presets.quick.depth',
|
||||
expectedGuidance: ['quick', 'standard', 'deep'],
|
||||
},
|
||||
] as const;
|
||||
|
||||
for (const c of enumCases) {
|
||||
@@ -211,9 +215,9 @@ test('semantic: validates cross-reference and value rules', () => {
|
||||
{ value: { seed: 3.14 }, path: 'seed', guidance: 'integer' },
|
||||
{ value: { presets: { quick: { timeout: -100 } } }, path: 'presets.quick.timeout', guidance: 'non-negative' },
|
||||
{
|
||||
value: { presets: { quick: { timeout: -100 } } },
|
||||
path: 'presets.quick.timeout',
|
||||
guidance: 'non-negative',
|
||||
value: { presets: { quick: { depth: 'super-deep' } } },
|
||||
path: 'presets.quick.depth',
|
||||
guidance: 'quick, standard, deep',
|
||||
},
|
||||
] as const;
|
||||
|
||||
@@ -228,7 +232,7 @@ test('semantic: validates cross-reference and value rules', () => {
|
||||
{ routes: ['GET /users', 'POST /items'] },
|
||||
{ seed: -42 },
|
||||
{ seed: 0 },
|
||||
{ presets: { quick: { timeout: 0 } } },
|
||||
{ presets: { quick: { timeout: 0, depth: 'standard' } } },
|
||||
];
|
||||
|
||||
for (const value of acceptCases) {
|
||||
@@ -318,7 +322,7 @@ test('loadConfig: resolves profile and preset and applies profile overrides', as
|
||||
},
|
||||
presets: {
|
||||
quick: {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
timeout: 5000,
|
||||
parallel: false,
|
||||
},
|
||||
|
||||
@@ -23,10 +23,12 @@
|
||||
* - Property and state model-based testing focused on confidence
|
||||
* - Iterative small steps with rapid feedback loops
|
||||
*/
|
||||
|
||||
import { test } from 'node:test';
|
||||
import assert from 'node:assert';
|
||||
import { writeFileSync, readFileSync } from 'node:fs';
|
||||
import { resolve } from 'node:path';
|
||||
|
||||
import {
|
||||
migrateCommand,
|
||||
detectAllLegacyPatterns,
|
||||
@@ -34,25 +36,31 @@ import {
|
||||
type MigrateOptions,
|
||||
type MigrationItem,
|
||||
} from '../../cli/commands/migrate/index.js';
|
||||
|
||||
import {
|
||||
rewriteConfigFile,
|
||||
detectLegacyConfigFields,
|
||||
detectLegacyFieldsNoEquivalent,
|
||||
detectMixedLegacyModernFields,
|
||||
} from '../../cli/commands/migrate/rewriters/config-rewriter.js';
|
||||
|
||||
import {
|
||||
rewriteRouteAnnotations,
|
||||
detectLegacyRouteAnnotations,
|
||||
detectAmbiguousRoutePatterns,
|
||||
} from '../../cli/commands/migrate/rewriters/route-rewriter.js';
|
||||
|
||||
import {
|
||||
rewriteCodePatterns,
|
||||
detectLegacyCodePatterns,
|
||||
detectAmbiguousCodePatterns,
|
||||
} from '../../cli/commands/migrate/rewriters/code-rewriter.js';
|
||||
|
||||
import { createTempDir, cleanup, makeCtx } from './helpers.js';
|
||||
|
||||
test('migrate --check detects broad legacy config field set', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
@@ -74,9 +82,12 @@ test('migrate --check detects broad legacy config field set', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 when legacy patterns are found');
|
||||
const legacyNames = result.items.map((item) => item.legacy);
|
||||
assert.ok(legacyNames.includes('testMode'), 'Should detect testMode');
|
||||
@@ -92,23 +103,29 @@ test('migrate --check detects broad legacy config field set', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 1: Mixed legacy and modern config detection
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate detects mixed legacy and modern config fields', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
// Config with both legacy and modern fields present
|
||||
const mixedConfig = `export default {
|
||||
// Legacy field
|
||||
testMode: "verify",
|
||||
|
||||
// Modern field (conflicts with legacy)
|
||||
mode: "observe",
|
||||
|
||||
profiles: {
|
||||
quick: {
|
||||
preset: "safe-ci",
|
||||
},
|
||||
},
|
||||
|
||||
// Legacy container
|
||||
testProfiles: {
|
||||
old: {
|
||||
@@ -116,17 +133,22 @@ test('migrate detects mixed legacy and modern config fields', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), mixedConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
// Should detect legacy patterns
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 when legacy patterns found');
|
||||
assert.ok(result.items.length > 0, 'Should detect legacy items');
|
||||
|
||||
// Check that mixed fields are reported
|
||||
const legacyNames = result.items.map((item) => item.legacy);
|
||||
assert.ok(legacyNames.includes('testMode'), 'Should detect testMode');
|
||||
assert.ok(legacyNames.includes('testProfiles'), 'Should detect testProfiles');
|
||||
assert.ok(legacyNames.includes('usesPreset'), 'Should detect usesPreset');
|
||||
|
||||
// Verify guidance mentions the conflict
|
||||
const testModeItem = result.items.find((item) => item.legacy === 'testMode');
|
||||
assert.ok(testModeItem, 'Should have testMode item');
|
||||
@@ -135,15 +157,19 @@ test('migrate detects mixed legacy and modern config fields', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 2: Dry-run shows exact rewrites
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate dry-run shows exact file path, line number, legacy text, replacement text', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
// Line 2
|
||||
testMode: "verify",
|
||||
|
||||
profiles: {
|
||||
quick: {
|
||||
// Line 7
|
||||
@@ -151,27 +177,36 @@ test('migrate dry-run shows exact file path, line number, legacy text, replaceme
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ dryRun: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 when legacy patterns found');
|
||||
assert.ok(result.message, 'Should have output message');
|
||||
|
||||
// Verify dry-run output contains exact details
|
||||
assert.ok(result.message.includes('Dry run'), 'Should indicate dry run');
|
||||
assert.ok(result.message.includes('testMode'), 'Should show legacy text');
|
||||
assert.ok(result.message.includes('mode'), 'Should show replacement text');
|
||||
assert.ok(result.message.includes('usesPreset'), 'Should show usesPreset');
|
||||
assert.ok(result.message.includes('preset'), 'Should show preset replacement');
|
||||
|
||||
// Verify file path is shown
|
||||
assert.ok(result.message.includes('apophis.config.js'), 'Should show file path');
|
||||
|
||||
// Verify line numbers are shown
|
||||
assert.ok(result.message.includes(':2') || result.message.includes(': 2'), 'Should show line number');
|
||||
|
||||
// Verify total count
|
||||
assert.ok(result.message.includes('Total:'), 'Should show total count');
|
||||
assert.ok(result.message.includes('3'), 'Should show correct total (3 items)');
|
||||
|
||||
// Verify files would be modified
|
||||
assert.ok(result.filesWouldBeModified, 'Should list files that would be modified');
|
||||
assert.strictEqual(result.filesWouldBeModified.length, 1, 'Should show 1 file would be modified');
|
||||
|
||||
// Verify file was NOT modified
|
||||
const content = readFileSync(resolve(dir, 'apophis.config.js'), 'utf-8');
|
||||
assert.ok(content.includes('testMode'), 'File should still have testMode');
|
||||
@@ -180,11 +215,14 @@ test('migrate dry-run shows exact file path, line number, legacy text, replaceme
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 3: Write performs rewrites correctly
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate write performs rewrites correctly', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
@@ -194,12 +232,16 @@ test('migrate write performs rewrites correctly', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ write: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 when rewrites performed');
|
||||
assert.ok(result.completed.length > 0, 'Should have completed items');
|
||||
assert.ok(result.filesModified && result.filesModified.length > 0, 'Should list modified files');
|
||||
|
||||
// Verify file WAS modified
|
||||
const content = readFileSync(resolve(dir, 'apophis.config.js'), 'utf-8');
|
||||
assert.ok(!content.includes('testMode'), 'File should not have testMode');
|
||||
@@ -212,26 +254,35 @@ test('migrate write performs rewrites correctly', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 4: Ambiguous rewrite stops and shows context
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate ambiguous rewrite stops and shows surrounding context', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
// Create a file with an ambiguous code pattern
|
||||
const ambiguousCode = `import Fastify from 'fastify';
|
||||
const app = Fastify();
|
||||
|
||||
// This is ambiguous: what does oldApi() mean here?
|
||||
app.register(oldApi());
|
||||
|
||||
export default app;`;
|
||||
|
||||
writeFileSync(resolve(dir, 'app.js'), ambiguousCode);
|
||||
|
||||
// Also create a config file so migration has something to work with
|
||||
const config = `export default {
|
||||
mode: "verify",
|
||||
};`;
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), config);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ write: true }, ctx);
|
||||
|
||||
// Should stop with exit code 2 (USAGE_ERROR) because ambiguous patterns found
|
||||
assert.strictEqual(result.exitCode, 2, 'Should exit 2 when ambiguous patterns found in write mode');
|
||||
assert.ok(result.remaining.length > 0, 'Should have remaining items');
|
||||
@@ -239,17 +290,21 @@ export default app;`;
|
||||
assert.ok(result.message.includes('Ambiguous'), 'Should mention ambiguous patterns');
|
||||
assert.ok(result.message.includes('oldApi()'), 'Should show the ambiguous pattern');
|
||||
assert.ok(result.message.includes('manual choice'), 'Should mention manual choice');
|
||||
|
||||
// Verify context is shown (surrounding lines)
|
||||
assert.ok(result.message.includes('app.register'), 'Should show surrounding context');
|
||||
} finally {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 5: Legacy field with no equivalent emits guidance
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate legacy field with no direct equivalent emits human guidance', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
// Config with a legacy field that has no direct equivalent
|
||||
const legacyConfig = `export default {
|
||||
@@ -262,12 +317,16 @@ test('migrate legacy field with no direct equivalent emits human guidance', asyn
|
||||
// This field is deprecated with no direct equivalent
|
||||
legacyField: true,
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
// Should detect the legacy field with no equivalent
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 when legacy patterns found');
|
||||
assert.ok(result.items.length > 0, 'Should detect legacy items');
|
||||
|
||||
const legacyFieldItem = result.items.find((item) => item.legacy === 'legacyField');
|
||||
assert.ok(legacyFieldItem, 'Should detect legacyField');
|
||||
assert.ok(legacyFieldItem.guidance, 'Should have guidance for legacyField');
|
||||
@@ -284,11 +343,14 @@ test('migrate legacy field with no direct equivalent emits human guidance', asyn
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 6: Partial migration reports completed and remaining
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate partial migration reports completed and remaining items', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
@@ -298,9 +360,12 @@ test('migrate partial migration reports completed and remaining items', async ()
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ write: true }, ctx);
|
||||
|
||||
assert.ok(result.completed.length > 0, 'Should have completed items');
|
||||
assert.ok(result.message, 'Should have output message');
|
||||
assert.ok(result.message.includes('Completed'), 'Should mention completed');
|
||||
@@ -309,16 +374,20 @@ test('migrate partial migration reports completed and remaining items', async ()
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 7: Preserves comments/formatting where feasible
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate preserves comments and formatting where feasible', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
// Config with specific formatting (comments, indentation)
|
||||
const legacyConfig = `export default {
|
||||
// This is a comment about testMode
|
||||
testMode: "verify",
|
||||
|
||||
/*
|
||||
* Block comment about testProfiles
|
||||
*/
|
||||
@@ -329,14 +398,19 @@ test('migrate preserves comments and formatting where feasible', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ write: true }, ctx);
|
||||
|
||||
const content = readFileSync(resolve(dir, 'apophis.config.js'), 'utf-8');
|
||||
|
||||
// Verify comments are preserved
|
||||
assert.ok(content.includes('// This is a comment about testMode'), 'Should preserve line comment');
|
||||
assert.ok(content.includes('Block comment about testProfiles'), 'Should preserve block comment');
|
||||
assert.ok(content.includes('// Inline comment'), 'Should preserve inline comment');
|
||||
|
||||
// Verify replacements were made
|
||||
assert.ok(content.includes('mode:'), 'Should have mode');
|
||||
assert.ok(content.includes('profiles:'), 'Should have profiles');
|
||||
@@ -345,11 +419,14 @@ test('migrate preserves comments and formatting where feasible', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 8: Migrate exits 0 when config is already modern
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate exits 0 when config is already modern', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const modernConfig = `export default {
|
||||
mode: "verify",
|
||||
@@ -361,7 +438,7 @@ test('migrate exits 0 when config is already modern', async () => {
|
||||
},
|
||||
presets: {
|
||||
"safe-ci": {
|
||||
,
|
||||
depth: "quick",
|
||||
timeout: 5000,
|
||||
},
|
||||
},
|
||||
@@ -371,9 +448,12 @@ test('migrate exits 0 when config is already modern', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), modernConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 0, 'Should exit 0 for modern config');
|
||||
assert.strictEqual(result.items.length, 0, 'Should have no items');
|
||||
assert.ok(result.message, 'Should have message');
|
||||
@@ -382,25 +462,35 @@ test('migrate exits 0 when config is already modern', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 9: Migrate exits 2 when ambiguous in write mode
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate exits 2 when ambiguous patterns found in write mode', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const config = `export default {
|
||||
mode: "verify",
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), config);
|
||||
|
||||
// Create app with an ambiguous pattern
|
||||
const code = `import Fastify from 'fastify';
|
||||
const app = Fastify();
|
||||
|
||||
// Ambiguous pattern
|
||||
app.register(oldApi());
|
||||
|
||||
export default app;`;
|
||||
|
||||
writeFileSync(resolve(dir, 'app.js'), code);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ write: true }, ctx);
|
||||
|
||||
// Should exit 2 because ambiguous patterns found
|
||||
assert.strictEqual(result.exitCode, 2, 'Should exit 2 when ambiguous patterns found in write mode');
|
||||
assert.ok(result.remaining.length > 0, 'Should have remaining ambiguous items');
|
||||
@@ -409,11 +499,14 @@ export default app;`;
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 10: Migrate emits guidance for each legacy field
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate emits guidance for each legacy field', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
@@ -423,10 +516,14 @@ test('migrate emits guidance for each legacy field', async () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.ok(result.items.length > 0, 'Should have items');
|
||||
|
||||
for (const item of result.items) {
|
||||
assert.ok(item.guidance, `Item ${item.legacy} should have guidance`);
|
||||
assert.ok(
|
||||
@@ -438,11 +535,14 @@ test('migrate emits guidance for each legacy field', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 11: Config rewriter replaces legacy fields
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('config rewriter replaces legacy fields', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `export default {
|
||||
testMode: "verify",
|
||||
@@ -452,13 +552,17 @@ test('config rewriter replaces legacy fields', () => {
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.config.js'), content);
|
||||
|
||||
const items = detectLegacyConfigFields(content, 'test.config.js');
|
||||
assert.strictEqual(items.length, 3, 'Should detect 3 legacy fields');
|
||||
|
||||
const result = rewriteConfigFile(
|
||||
resolve(dir, 'test.config.js'),
|
||||
items,
|
||||
);
|
||||
|
||||
assert.strictEqual(result.modified, true, 'Should modify content');
|
||||
assert.ok(result.content.includes('mode:'), 'Should have mode');
|
||||
assert.ok(result.content.includes('profiles:'), 'Should have profiles');
|
||||
@@ -468,28 +572,35 @@ test('config rewriter replaces legacy fields', () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 12: Route rewriter detects x-validate-runtime annotation
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('route rewriter detects x-validate-runtime annotation', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `export default {
|
||||
schema: {
|
||||
'x-validate-runtime': true,
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.routes.js'), content);
|
||||
|
||||
const items = detectLegacyRouteAnnotations(content, 'test.routes.js');
|
||||
assert.strictEqual(items.length, 1, 'Should detect 1 legacy annotation');
|
||||
const firstItem = items[0];
|
||||
assert.ok(firstItem, 'Expected one migration item');
|
||||
assert.strictEqual(firstItem.legacy, 'x-validate-runtime');
|
||||
assert.strictEqual(firstItem.replacement, 'runtime');
|
||||
|
||||
const result = rewriteRouteAnnotations(
|
||||
resolve(dir, 'test.routes.js'),
|
||||
items,
|
||||
);
|
||||
|
||||
assert.strictEqual(result.modified, true, 'Should modify content');
|
||||
assert.ok(result.content.includes("'runtime'"), 'Should have runtime');
|
||||
assert.ok(!result.content.includes('x-validate-runtime'), 'Should not have legacy annotation');
|
||||
@@ -497,25 +608,34 @@ test('route rewriter detects x-validate-runtime annotation', () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 13: Code rewriter detects legacy patterns
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('code rewriter detects legacy patterns', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `import Fastify from 'fastify';
|
||||
const app = Fastify();
|
||||
|
||||
app.register(contract());
|
||||
app.register(stateful());
|
||||
app.register(scenario());
|
||||
|
||||
export default app;`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.app.js'), content);
|
||||
|
||||
const items = detectLegacyCodePatterns(content, 'test.app.js');
|
||||
assert.strictEqual(items.length, 3, 'Should detect 3 legacy patterns');
|
||||
|
||||
const result = rewriteCodePatterns(
|
||||
resolve(dir, 'test.app.js'),
|
||||
items,
|
||||
);
|
||||
|
||||
assert.strictEqual(result.modified, true, 'Should modify content');
|
||||
assert.ok(result.content.includes("verify({ kind: 'contract' })"), 'Should have verify');
|
||||
assert.ok(result.content.includes("qualify({ kind: 'stateful' })"), 'Should have qualify stateful');
|
||||
@@ -524,21 +644,28 @@ export default app;`;
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 14: Dry-run default mode (safe by default)
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate defaults to dry-run mode (safe by default)', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
// No mode specified — should default to dry-run
|
||||
const result = await migrateCommand({}, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1 in dry-run mode');
|
||||
assert.ok(result.message?.includes('Dry run'), 'Should indicate dry run');
|
||||
|
||||
// Verify file was NOT modified
|
||||
const content = readFileSync(resolve(dir, 'apophis.config.js'), 'utf-8');
|
||||
assert.ok(content.includes('testMode'), 'File should still have testMode');
|
||||
@@ -546,30 +673,38 @@ test('migrate defaults to dry-run mode (safe by default)', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 15: Mixed legacy/modern field detection at rewriter level
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('config rewriter detects mixed legacy and modern fields', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `export default {
|
||||
// Both legacy and modern present
|
||||
testMode: "verify",
|
||||
mode: "observe",
|
||||
|
||||
testProfiles: {
|
||||
quick: {
|
||||
usesPreset: "safe-ci",
|
||||
},
|
||||
},
|
||||
|
||||
profiles: {
|
||||
modern: {
|
||||
preset: "safe-ci",
|
||||
},
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.config.js'), content);
|
||||
|
||||
const mixedReports = detectMixedLegacyModernFields(content, 'test.config.js');
|
||||
assert.ok(mixedReports.length > 0, 'Should detect mixed fields');
|
||||
|
||||
const testModeReport = mixedReports.find((r) => r.legacy === 'testMode');
|
||||
assert.ok(testModeReport, 'Should report testMode as mixed');
|
||||
assert.ok(testModeReport.guidance.includes('testMode'), 'Guidance should mention testMode');
|
||||
@@ -578,11 +713,14 @@ test('config rewriter detects mixed legacy and modern fields', () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 16: Ambiguous route pattern detection
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('route rewriter detects ambiguous route patterns with context', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `export default {
|
||||
schema: {
|
||||
@@ -590,9 +728,12 @@ test('route rewriter detects ambiguous route patterns with context', () => {
|
||||
'x-validate': true,
|
||||
},
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.routes.js'), content);
|
||||
|
||||
const items = detectAmbiguousRoutePatterns(content, 'test.routes.js');
|
||||
assert.strictEqual(items.length, 1, 'Should detect 1 ambiguous pattern');
|
||||
|
||||
const firstItem = items[0];
|
||||
assert.ok(firstItem, 'Expected one migration item');
|
||||
assert.strictEqual(firstItem.legacy, 'x-validate');
|
||||
@@ -603,20 +744,28 @@ test('route rewriter detects ambiguous route patterns with context', () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 17: Ambiguous code pattern detection with context
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('code rewriter detects ambiguous code patterns with surrounding context', () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const content = `import Fastify from 'fastify';
|
||||
const app = Fastify();
|
||||
|
||||
// Ambiguous pattern
|
||||
app.register(oldApi());
|
||||
|
||||
export default app;`;
|
||||
|
||||
writeFileSync(resolve(dir, 'test.app.js'), content);
|
||||
|
||||
const items = detectAmbiguousCodePatterns(content, 'test.app.js');
|
||||
assert.strictEqual(items.length, 1, 'Should detect 1 ambiguous pattern');
|
||||
|
||||
const firstItem = items[0];
|
||||
assert.ok(firstItem, 'Expected one migration item');
|
||||
assert.strictEqual(firstItem.legacy, 'oldApi()');
|
||||
@@ -628,32 +777,42 @@ export default app;`;
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 18: Legacy fixture detection
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate detects legacy patterns in fixture config', async () => {
|
||||
const ctx = makeCtx({ cwd: 'src/cli/__fixtures__/legacy-config' });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should detect legacy patterns in fixture');
|
||||
assert.ok(result.items.length > 0, 'Should find legacy items');
|
||||
|
||||
const legacyNames = result.items.map((item) => item.legacy);
|
||||
assert.ok(legacyNames.includes('testMode'), 'Should detect testMode in fixture');
|
||||
assert.ok(legacyNames.includes('testProfiles'), 'Should detect testProfiles in fixture');
|
||||
assert.ok(legacyNames.includes('testPresets'), 'Should detect testPresets in fixture');
|
||||
assert.ok(legacyNames.includes('envPolicies'), 'Should detect envPolicies in fixture');
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 19: JSON output format
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate outputs JSON format with all fields', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const legacyConfig = `export default {
|
||||
testMode: "verify",
|
||||
};`;
|
||||
|
||||
writeFileSync(resolve(dir, 'apophis.config.js'), legacyConfig);
|
||||
|
||||
const ctx = makeCtx({ cwd: dir, options: { ...makeCtx().options, format: 'json' } });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 1, 'Should exit 1');
|
||||
assert.ok(result.items.length > 0, 'Should have items');
|
||||
assert.ok(result.totalRewrites, 'Should have totalRewrites');
|
||||
@@ -662,14 +821,18 @@ test('migrate outputs JSON format with all fields', async () => {
|
||||
cleanup(dir);
|
||||
}
|
||||
});
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Test 20: No files found returns usage error
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('migrate returns usage error when no files found', async () => {
|
||||
const dir = createTempDir();
|
||||
|
||||
try {
|
||||
const ctx = makeCtx({ cwd: dir });
|
||||
const result = await migrateCommand({ check: true }, ctx);
|
||||
|
||||
assert.strictEqual(result.exitCode, 2, 'Should exit 2 when no files found');
|
||||
assert.ok(result.message?.includes('No config or app files found'), 'Should mention no files found');
|
||||
} finally {
|
||||
|
||||
@@ -152,7 +152,7 @@ test('contract runner supports cross-operation APOSTL ensures', async () => {
|
||||
await fastify.register(apophisPlugin, {})
|
||||
registerItemApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({ runs: 10, seed: 7 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick', seed: 7 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
assert.strictEqual(failures.length, 0)
|
||||
} finally {
|
||||
@@ -169,7 +169,7 @@ test('stateful runner supports cross-operation APOSTL ensures', async () => {
|
||||
await fastify.register(apophisPlugin, {})
|
||||
registerItemApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.stateful({ runs: 10, seed: 11 })
|
||||
const result = await fastify.apophis.stateful({ depth: 'quick', seed: 11 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
assert.strictEqual(failures.length, 0)
|
||||
} finally {
|
||||
@@ -217,7 +217,7 @@ test('contract runner applies chaos injection when configured', async () => {
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
seed: 3,
|
||||
chaos: {
|
||||
probability: 1,
|
||||
@@ -241,7 +241,7 @@ test('contract runner supports previous(...) with response_body(this) path place
|
||||
await fastify.register(apophisPlugin, {})
|
||||
registerPlanApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({ runs: 10, seed: 17 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick', seed: 17 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
assert.strictEqual(failures.length, 0)
|
||||
} finally {
|
||||
@@ -258,7 +258,7 @@ test('stateful runner supports previous(...) with response_body(this) path place
|
||||
await fastify.register(apophisPlugin, {})
|
||||
registerPlanApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.stateful({ runs: 10, seed: 19 })
|
||||
const result = await fastify.apophis.stateful({ depth: 'quick', seed: 19 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
assert.strictEqual(failures.length, 0)
|
||||
} finally {
|
||||
@@ -406,7 +406,7 @@ test('contract runner validates hypermedia links with route_exists', async () =>
|
||||
})
|
||||
registerHypermediaApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({ runs: 10, seed: 23 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick', seed: 23 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
if (failures.length > 0) {
|
||||
console.log('Contract failures:', failures.map((f: TestResult) => ({
|
||||
@@ -441,7 +441,7 @@ test('stateful runner validates cross-route relationships', async () => {
|
||||
})
|
||||
registerHypermediaApi(fastify)
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.stateful({ runs: 10, seed: 29 })
|
||||
const result = await fastify.apophis.stateful({ depth: 'quick', seed: 29 })
|
||||
const failures = result.tests.filter((entry: TestResult) => !entry.ok)
|
||||
if (failures.length > 0) {
|
||||
console.log('Stateful failures:', failures.map((f: TestResult) => ({
|
||||
|
||||
@@ -36,7 +36,7 @@ test('APOPHIS_DEBUG=1 logs requests and responses', async () => {
|
||||
originalDebug(msg, _obj)
|
||||
}
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should have tests')
|
||||
|
||||
// Should have logged at least one request and one response
|
||||
@@ -80,7 +80,7 @@ test('APOPHIS_DEBUG=0 does not log requests', async () => {
|
||||
originalDebug(msg, _obj)
|
||||
}
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should have tests')
|
||||
|
||||
// Should not have any request/response logs
|
||||
|
||||
@@ -30,7 +30,7 @@ test('example: minimal API compiles and runs', async () => {
|
||||
|
||||
await fastify.ready()
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should have test results')
|
||||
console.log('Minimal example:', result.summary)
|
||||
} finally {
|
||||
@@ -111,7 +111,7 @@ test('example: CRUD API with contracts compiles and runs', async () => {
|
||||
|
||||
await fastify.ready()
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should have test results')
|
||||
console.log('CRUD example:', result.summary)
|
||||
} finally {
|
||||
@@ -149,7 +149,7 @@ test('example: prefix registration works', async () => {
|
||||
const itemContract = contracts.find(c => c.path === '/api/v1/items')
|
||||
assert.ok(itemContract, 'should discover prefixed route')
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should run tests on prefixed routes')
|
||||
} finally {
|
||||
await fastify.close()
|
||||
|
||||
@@ -523,35 +523,11 @@ test('evaluate: returns error for missing previous context', () => {
|
||||
assert.strictEqual(result.success, false)
|
||||
assert.ok((result as { success: false; error: string }).error.includes('No previous context'))
|
||||
})
|
||||
test('evaluate: returns false for non-array in quantified expression', () => {
|
||||
test('evaluate: returns error for non-array in quantified expression', () => {
|
||||
const ast = parse('for x in response_code(this): x == 1')
|
||||
const result = evaluate(ast.ast, makeContext())
|
||||
assert.strictEqual(result.success, true)
|
||||
assert.strictEqual((result as { success: true; value: unknown }).value, false)
|
||||
})
|
||||
test('evaluate: returns false for undefined collection in quantified expression', () => {
|
||||
const ast = parse('for x in response_body(this): x == 1')
|
||||
const result = evaluate(ast.ast, makeContext({ response: { body: undefined, headers: {}, statusCode: 200, responseTime: 0 } }))
|
||||
assert.strictEqual(result.success, true)
|
||||
assert.strictEqual((result as { success: true; value: unknown }).value, false)
|
||||
})
|
||||
test('evaluate: returns false for error object collection in quantified expression', () => {
|
||||
const ast = parse('for x in response_body(this): x == 1')
|
||||
const result = evaluate(ast.ast, makeContext({ response: { body: { error: 'Chaos error: forced 503' }, headers: {}, statusCode: 200, responseTime: 0 } }))
|
||||
assert.strictEqual(result.success, true)
|
||||
assert.strictEqual((result as { success: true; value: unknown }).value, false)
|
||||
})
|
||||
test('evaluate: quantified nested in conditional handles undefined gracefully', () => {
|
||||
const ast = parse('if status:503 then for x in response_body(this): x == 1 else true')
|
||||
const result = evaluate(ast.ast, makeContext({ response: { statusCode: 503, body: undefined, headers: {}, responseTime: 0 } }))
|
||||
assert.strictEqual(result.success, true)
|
||||
assert.strictEqual((result as { success: true; value: unknown }).value, false)
|
||||
})
|
||||
test('evaluate: exists returns false for non-array collection', () => {
|
||||
const ast = parse('exists x in response_body(this): x == 1')
|
||||
const result = evaluate(ast.ast, makeContext({ response: { body: { error: 'fail' }, headers: {}, statusCode: 200, responseTime: 0 } }))
|
||||
assert.strictEqual(result.success, true)
|
||||
assert.strictEqual((result as { success: true; value: unknown }).value, false)
|
||||
assert.strictEqual(result.success, false)
|
||||
assert.ok((result as { success: false; error: string }).error.includes('array collection'))
|
||||
})
|
||||
// ============================================================================
|
||||
// Unit Tests: Substitutor
|
||||
|
||||
@@ -243,7 +243,7 @@ test('petit-runner executes tests against real API', async () => {
|
||||
]
|
||||
const fastifyWithRoutes = Object.assign(fastify, { routes: mockRoutes })
|
||||
const result = await runPetitTests(fastifyWithRoutes as any, {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: undefined
|
||||
})
|
||||
@@ -367,7 +367,7 @@ test('full integration: plugin + routes + test execution', async () => {
|
||||
const createUserContract = contracts.find(c => c.path === '/users' && c.method === 'POST')
|
||||
assert.ok(createUserContract, 'create user contract should exist')
|
||||
assert.strictEqual(createUserContract.category, 'constructor')
|
||||
const testResult = await fastify.apophis.contract({ runs: 10 })
|
||||
const testResult = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(Array.isArray(testResult.tests), 'tests should be an array')
|
||||
assert.ok(testResult.tests.length > 0, 'tests should not be empty')
|
||||
await fastify.apophis.cleanup()
|
||||
@@ -439,7 +439,7 @@ test('mode filtering: stateful mode only runs constructor/mutator routes', async
|
||||
}, async () => ({ status: 'ok' }))
|
||||
await fastify.ready()
|
||||
// Run in stateful mode
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
// In stateful mode, utility routes should be excluded
|
||||
// The test should only run constructor and mutator routes
|
||||
assert.ok(Array.isArray(result.tests), 'tests should be an array')
|
||||
@@ -474,7 +474,7 @@ test('failing contract produces ContractViolation with suggestion', async () =>
|
||||
return { status: 'created' } // Returns 200, not 201
|
||||
})
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
// Find the failing test
|
||||
const failingTests = result.tests.filter(t => !t.ok)
|
||||
assert.ok(failingTests.length > 0, 'should have at least one failing test')
|
||||
@@ -647,7 +647,7 @@ test('integration: contract routes option limits tested routes', async () => {
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
routes: ['GET /included'],
|
||||
})
|
||||
const includedTests = result.tests.filter(t => t.name.includes('GET /included'))
|
||||
@@ -673,7 +673,7 @@ test('integration: contract variants are tagged and run in declared order', asyn
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
variants: [
|
||||
{ name: 'json', headers: { accept: 'application/json' } },
|
||||
{ name: 'xml', headers: { accept: 'application/xml' } },
|
||||
@@ -715,7 +715,7 @@ test('integration: variant headers override scope headers', async () => {
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
variants: [
|
||||
{ name: 'xml', headers: { accept: 'application/xml' } },
|
||||
],
|
||||
@@ -744,7 +744,7 @@ test('integration: route-level x-variants are extracted and executed', async ()
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
// No call-site variants; route-level variants should drive execution
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
const jsonTests = result.tests.filter((t) => t.name.includes('[variant:json]'))
|
||||
const xmlTests = result.tests.filter((t) => t.name.includes('[variant:xml]'))
|
||||
assert.ok(jsonTests.length > 0, 'route json variant should produce tests')
|
||||
@@ -781,7 +781,7 @@ test('integration: inferred contracts are guarded by status code', async () => {
|
||||
return { error: 'not found' }
|
||||
})
|
||||
await fastify.ready()
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
// Should pass because the inferred const contract is guarded:
|
||||
// response_code(this) == 200 => response_body(this).status == "success"
|
||||
// The 404 response doesn't trigger the antecedent, so the implication holds.
|
||||
|
||||
+13
-31
@@ -7,7 +7,7 @@
|
||||
import { convertSchema } from '../domain/schema-to-arbitrary.js'
|
||||
import { lookupCache, storeCache } from '../incremental/cache.js'
|
||||
import type { ApiCommand } from '../domain/stateful.js'
|
||||
import type { RouteContract, RunConfig } from '../types.js'
|
||||
import type { DepthConfig, RouteContract } from '../types.js'
|
||||
import * as fc from 'fast-check'
|
||||
|
||||
const buildCommand = (route: RouteContract, data: unknown): ApiCommand => ({
|
||||
@@ -19,10 +19,11 @@ const buildCommand = (route: RouteContract, data: unknown): ApiCommand => ({
|
||||
|
||||
export const generateCommands = (
|
||||
routes: RouteContract[],
|
||||
runConfig: RunConfig,
|
||||
depth: DepthConfig,
|
||||
seed?: number,
|
||||
generationProfile: 'quick' | 'standard' | 'thorough' = 'standard',
|
||||
): { commands: ApiCommand[][], cacheHits: number, cacheMisses: number } => {
|
||||
const commandsPerRoute = Math.max(1, Math.floor(runConfig.contractRuns / Math.max(routes.length, 1)))
|
||||
const commandsPerRoute = Math.max(1, Math.floor(depth.contractRuns / Math.max(routes.length, 1)))
|
||||
let cacheHits = 0
|
||||
let cacheMisses = 0
|
||||
const allCommands = routes.map((route) => {
|
||||
@@ -39,7 +40,7 @@ export const generateCommands = (
|
||||
cacheMisses++
|
||||
const bodySchema = route.schema?.body as Record<string, unknown> | undefined
|
||||
const bodyArb = bodySchema !== undefined
|
||||
? convertSchema(bodySchema, { context: 'request' })
|
||||
? convertSchema(bodySchema, { context: 'request', generationProfile })
|
||||
: fc.constant({})
|
||||
const pathParams = route.path.match(/:([a-zA-Z_][a-zA-Z0-9_]*)/g) ?? []
|
||||
const pathParamArbs: Record<string, fc.Arbitrary<string>> = {}
|
||||
@@ -81,7 +82,7 @@ import { buildPetitSuite, filterPetitRoutes } from './route-filter.js'
|
||||
import { executePetitCommandStep } from './petit-command-step.js'
|
||||
import { runTripleBoundaryPropertyTest } from './triple-boundary-runner.js'
|
||||
import { makeTrackedResource } from '../domain/state-operations.js'
|
||||
import { resolveRuns } from '../types.js'
|
||||
import { resolveDepth, resolveGenerationProfile } from '../types.js'
|
||||
import type {
|
||||
EvalContext,
|
||||
FastifyInjectInstance,
|
||||
@@ -104,37 +105,17 @@ export const runPetitTests = async (
|
||||
config: TestConfig,
|
||||
scopeRegistry?: ScopeRegistry,
|
||||
extensionRegistry?: ExtensionRegistry,
|
||||
pluginContractRegistry?: import('../domain/plugin-contracts.js').PluginContractRegistry,
|
||||
outboundContractRegistry?: OutboundContractRegistry,
|
||||
overrideContracts?: RouteContract[]
|
||||
_pluginContractRegistry?: import('../domain/plugin-contracts.js').PluginContractRegistry,
|
||||
outboundContractRegistry?: OutboundContractRegistry
|
||||
): Promise<TestSuite> => {
|
||||
const startTime = Date.now()
|
||||
if (extensionRegistry) await extensionRegistry.runSuiteStartHooks(config)
|
||||
|
||||
const allRoutes = overrideContracts ?? discoverRoutes(fastify)
|
||||
const allRoutes = discoverRoutes(fastify)
|
||||
const { routes, skippedRoutes } = filterPetitRoutes(allRoutes, config)
|
||||
|
||||
// Merge plugin contracts into route contracts
|
||||
if (pluginContractRegistry) {
|
||||
for (const route of routes) {
|
||||
const composed = pluginContractRegistry.composeContracts(route)
|
||||
for (const phase of Object.values(composed.phases)) {
|
||||
for (const req of phase.requires) {
|
||||
if (!route.requires.includes(req.formula)) {
|
||||
route.requires.push(req.formula)
|
||||
}
|
||||
}
|
||||
for (const ens of phase.ensures) {
|
||||
if (!route.ensures.includes(ens.formula)) {
|
||||
route.ensures.push(ens.formula)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
const runConfig = resolveRuns(config.runs)
|
||||
const { commands: commandGroups, cacheHits, cacheMisses } = generateCommands(routes, runConfig, config.seed)
|
||||
const depth = resolveDepth(config.depth ?? 'standard')
|
||||
const generationProfile = config.generationProfile ?? resolveGenerationProfile(config.depth)
|
||||
const { commands: commandGroups, cacheHits, cacheMisses } = generateCommands(routes, depth, config.seed, generationProfile)
|
||||
const allCommands = commandGroups.flat()
|
||||
const rng = config.seed !== undefined ? new SeededRng(config.seed) : undefined
|
||||
|
||||
@@ -180,6 +161,7 @@ export const runPetitTests = async (
|
||||
? createOutboundMockRuntime({
|
||||
contracts: outboundContractRegistry.resolve(Array.from(outboundNames)),
|
||||
mode: config.outboundMocks?.mode ?? 'example',
|
||||
generationProfile,
|
||||
overrides: config.outboundMocks?.overrides,
|
||||
unmatched: config.outboundMocks?.unmatched ?? 'error',
|
||||
seed: config.seed !== undefined ? hashCombine(config.seed, 0x6d6f636b) : Math.floor(Math.random() * 0xffffffff),
|
||||
|
||||
@@ -1,425 +0,0 @@
|
||||
/**
|
||||
* Quality Engine Tests — Flake Detection, Mutation Testing, Chaos
|
||||
*/
|
||||
import { test } from 'node:test'
|
||||
import assert from 'node:assert'
|
||||
import Fastify from 'fastify'
|
||||
import swagger from '@fastify/swagger'
|
||||
import apophisPlugin from '../index.js'
|
||||
import { FlakeDetector } from '../quality/flake.js'
|
||||
import { runMutationTesting, testMutation, type Mutation } from '../quality/mutation.js'
|
||||
import * as fc from 'fast-check'
|
||||
import {
|
||||
applyChaosToExecution,
|
||||
applyChaosToAllResponses,
|
||||
applyChaosToDependencyResponse,
|
||||
createChaosEventArbitrary,
|
||||
extractDelays,
|
||||
sleep,
|
||||
hasAppliedChaos,
|
||||
formatChaosEvents,
|
||||
type ChaosEvent,
|
||||
} from '../quality/chaos-v3.js'
|
||||
import type { EvalContext, TestResult, RouteContract } from '../types.js'
|
||||
|
||||
// Quality engines require test environment
|
||||
process.env.NODE_ENV = 'test'
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Helpers
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
function makeCtx(overrides?: Partial<EvalContext['response']>): EvalContext {
|
||||
return {
|
||||
request: { body: {}, headers: {}, query: {}, params: {} },
|
||||
response: { body: { id: '1' }, headers: {}, statusCode: 200, ...overrides },
|
||||
} as EvalContext
|
||||
}
|
||||
|
||||
function makeTestResult(ok: boolean): TestResult {
|
||||
return {
|
||||
ok,
|
||||
name: 'test',
|
||||
id: 1,
|
||||
diagnostics: ok ? {} : { error: 'failed' },
|
||||
}
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Chaos Tests
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('applyChaosToExecution: no chaos when events are empty', () => {
|
||||
const ctx = makeCtx()
|
||||
const result = applyChaosToExecution(ctx, [])
|
||||
assert.strictEqual(result.applied, false)
|
||||
assert.strictEqual(result.ctx.response.statusCode, 200)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: inbound error changes status code', () => {
|
||||
const ctx = makeCtx()
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-error', target: 'inbound', statusCode: 503 },
|
||||
])
|
||||
assert.strictEqual(result.applied, true)
|
||||
assert.strictEqual(result.ctx.response.statusCode, 503)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: inbound dropout simulates gateway timeout', () => {
|
||||
const ctx = makeCtx()
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-dropout', target: 'inbound' },
|
||||
])
|
||||
assert.strictEqual(result.ctx.response.statusCode, 504)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: inbound corruption truncates response body', () => {
|
||||
const ctx = makeCtx({ body: { id: '1', name: 'Alice', email: 'a@b.com' } })
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-corruption', target: 'inbound', corruptionStrategy: 'truncate' },
|
||||
])
|
||||
const body = result.ctx.response.body as Record<string, unknown>
|
||||
assert.ok(Object.keys(body).length <= 2)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: inbound corruption with field-corrupt', () => {
|
||||
const ctx = makeCtx({ body: { id: '1', name: 'Alice' } })
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-corruption', target: 'inbound', corruptionStrategy: 'field-corrupt', corruptionField: 'name' },
|
||||
])
|
||||
const body = result.ctx.response.body as Record<string, unknown>
|
||||
assert.strictEqual(body.name, null)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: inbound corruption malformed', () => {
|
||||
const ctx = makeCtx({ body: { id: '1' } })
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-corruption', target: 'inbound', corruptionStrategy: 'malformed' },
|
||||
])
|
||||
assert.strictEqual(result.ctx.response.body, '{"broken":')
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: ignores none events', () => {
|
||||
const ctx = makeCtx()
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'none', target: 'inbound' },
|
||||
])
|
||||
assert.strictEqual(result.applied, false)
|
||||
})
|
||||
|
||||
test('applyChaosToExecution: only applies first non-delay event', () => {
|
||||
const ctx = makeCtx()
|
||||
const result = applyChaosToExecution(ctx, [
|
||||
{ type: 'inbound-error', target: 'inbound', statusCode: 503 },
|
||||
{ type: 'inbound-dropout', target: 'inbound' },
|
||||
])
|
||||
assert.strictEqual(result.ctx.response.statusCode, 503)
|
||||
})
|
||||
|
||||
test('applyChaosToDependencyResponse: outbound error changes status', () => {
|
||||
const response = { contractName: 'api', statusCode: 200, body: { ok: true } }
|
||||
const result = applyChaosToDependencyResponse(response, [
|
||||
{ type: 'outbound-error', target: 'outbound', contractName: 'api', statusCode: 503 },
|
||||
])
|
||||
assert.strictEqual(result.statusCode, 503)
|
||||
})
|
||||
|
||||
test('applyChaosToDependencyResponse: ignores events for other contracts', () => {
|
||||
const response = { contractName: 'api', statusCode: 200, body: {} }
|
||||
const result = applyChaosToDependencyResponse(response, [
|
||||
{ type: 'outbound-error', target: 'outbound', contractName: 'other', statusCode: 503 },
|
||||
])
|
||||
assert.strictEqual(result.statusCode, 200)
|
||||
})
|
||||
|
||||
test('applyChaosToAllResponses: applies chaos to multiple responses', () => {
|
||||
const responses = [
|
||||
{ contractName: 'a', statusCode: 200, body: {} },
|
||||
{ contractName: 'b', statusCode: 200, body: {} },
|
||||
]
|
||||
const events = [
|
||||
{ type: 'outbound-error', target: 'outbound', contractName: 'a', statusCode: 503 },
|
||||
{ type: 'outbound-error', target: 'outbound', contractName: 'b', statusCode: 504 },
|
||||
] as ChaosEvent[]
|
||||
const result = applyChaosToAllResponses(responses, events)
|
||||
assert.strictEqual(result[0]!.statusCode, 503)
|
||||
assert.strictEqual(result[1]!.statusCode, 504)
|
||||
})
|
||||
|
||||
test('extractDelays: computes total delay', () => {
|
||||
const events = [
|
||||
{ type: 'inbound-delay', target: 'inbound', delayMs: 100 },
|
||||
{ type: 'outbound-delay', target: 'outbound', delayMs: 200 },
|
||||
] as ChaosEvent[]
|
||||
const result = extractDelays(events)
|
||||
assert.strictEqual(result.totalMs, 300)
|
||||
assert.strictEqual(result.events.length, 2)
|
||||
})
|
||||
|
||||
test('sleep: resolves after specified ms', async () => {
|
||||
const start = Date.now()
|
||||
await sleep(10)
|
||||
const elapsed = Date.now() - start
|
||||
assert.ok(elapsed >= 8 && elapsed < 100)
|
||||
})
|
||||
|
||||
test('hasAppliedChaos: detects applied chaos', () => {
|
||||
assert.strictEqual(hasAppliedChaos([{ type: 'none', target: 'inbound' }]), false)
|
||||
assert.strictEqual(hasAppliedChaos([{ type: 'inbound-error', target: 'inbound' }]), true)
|
||||
})
|
||||
|
||||
test('formatChaosEvents: formats events for diagnostics', () => {
|
||||
const events = [
|
||||
{ type: 'inbound-error', target: 'inbound', statusCode: 503 },
|
||||
] as ChaosEvent[]
|
||||
const formatted = formatChaosEvents(events)
|
||||
assert.ok(formatted.includes('inbound-error'))
|
||||
assert.ok(formatted.includes('503'))
|
||||
})
|
||||
|
||||
test('formatChaosEvents: returns "No chaos applied" for empty events', () => {
|
||||
assert.strictEqual(formatChaosEvents([]), 'No chaos applied')
|
||||
assert.strictEqual(formatChaosEvents([{ type: 'none', target: 'inbound' }]), 'No chaos applied')
|
||||
})
|
||||
|
||||
test('createChaosEventArbitrary: generates deterministic events with seed', () => {
|
||||
const arb = createChaosEventArbitrary(
|
||||
{ probability: 1, delay: { probability: 1, minMs: 100, maxMs: 200 } },
|
||||
[]
|
||||
)
|
||||
const sample1 = fc.sample(arb, { seed: 42, numRuns: 5 })
|
||||
const sample2 = fc.sample(arb, { seed: 42, numRuns: 5 })
|
||||
assert.deepStrictEqual(sample1, sample2)
|
||||
})
|
||||
|
||||
test('createChaosEventArbitrary: returns empty array when no config', () => {
|
||||
const arb = createChaosEventArbitrary(undefined, [])
|
||||
const sample = fc.sample(arb, { numRuns: 10 })
|
||||
assert.ok(sample.every((events) => events.length === 0 || events.every((e) => e.type === 'none')))
|
||||
})
|
||||
|
||||
test('createChaosEventArbitrary: generates outbound events for contracts', () => {
|
||||
const arb = createChaosEventArbitrary(
|
||||
{
|
||||
probability: 1,
|
||||
error: { probability: 1, statusCode: 503 },
|
||||
outbound: [
|
||||
{
|
||||
target: 'api',
|
||||
error: {
|
||||
probability: 1,
|
||||
responses: [{ statusCode: 500 }],
|
||||
},
|
||||
},
|
||||
],
|
||||
},
|
||||
['api']
|
||||
)
|
||||
const sample = fc.sample(arb, { numRuns: 20 })
|
||||
const hasOutbound = sample.some((events) =>
|
||||
events.some((e) => e.target === 'outbound')
|
||||
)
|
||||
assert.ok(hasOutbound, 'expected some outbound events')
|
||||
})
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Flake Detection Tests
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
test('FlakeDetector: deterministic failure returns high confidence', async () => {
|
||||
const detector = new FlakeDetector({ sameSeedReruns: 1, seedVariations: 2 })
|
||||
const report = await detector.detectFlake(
|
||||
makeTestResult(false),
|
||||
async () => ({ passed: false }),
|
||||
42
|
||||
)
|
||||
assert.strictEqual(report.isFlaky, false)
|
||||
assert.strictEqual(report.confidence, 'high')
|
||||
assert.strictEqual(report.reruns.length, 3)
|
||||
})
|
||||
|
||||
test('FlakeDetector: passing same-seed rerun flags flaky', async () => {
|
||||
const detector = new FlakeDetector({ sameSeedReruns: 1, seedVariations: 0 })
|
||||
let callCount = 0
|
||||
const report = await detector.detectFlake(
|
||||
makeTestResult(false),
|
||||
async () => {
|
||||
callCount++
|
||||
return { passed: callCount === 1 }
|
||||
},
|
||||
42
|
||||
)
|
||||
assert.strictEqual(report.isFlaky, true)
|
||||
assert.strictEqual(report.confidence, 'low')
|
||||
})
|
||||
|
||||
test('FlakeDetector: medium confidence when some pass', async () => {
|
||||
const detector = new FlakeDetector({ sameSeedReruns: 1, seedVariations: 2 })
|
||||
let callCount = 0
|
||||
const report = await detector.detectFlake(
|
||||
makeTestResult(false),
|
||||
async () => {
|
||||
callCount++
|
||||
return { passed: callCount % 2 === 0 }
|
||||
},
|
||||
42
|
||||
)
|
||||
assert.strictEqual(report.isFlaky, true)
|
||||
assert.strictEqual(report.confidence, 'medium')
|
||||
})
|
||||
|
||||
test('FlakeDetector: uses Date.now() when no seed provided', async () => {
|
||||
const detector = new FlakeDetector({ sameSeedReruns: 0, seedVariations: 1 })
|
||||
const report = await detector.detectFlake(
|
||||
makeTestResult(false),
|
||||
async () => ({ passed: false })
|
||||
)
|
||||
assert.strictEqual(report.reruns.length, 1)
|
||||
assert.ok(report.reruns[0]!.seed > 0)
|
||||
})
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Mutation Testing Tests
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
function makeContract(ensures: string[], requires: string[] = []): RouteContract {
|
||||
return {
|
||||
path: '/items',
|
||||
method: 'POST',
|
||||
category: 'constructor',
|
||||
requires,
|
||||
ensures,
|
||||
invariants: [],
|
||||
regexPatterns: {},
|
||||
validateRuntime: false,
|
||||
schema: {
|
||||
body: {
|
||||
type: 'object',
|
||||
properties: { name: { type: 'string' } },
|
||||
},
|
||||
response: {
|
||||
201: {
|
||||
type: 'object',
|
||||
properties: { id: { type: 'string' } },
|
||||
},
|
||||
},
|
||||
},
|
||||
}
|
||||
}
|
||||
|
||||
test('testMutation: detects flipped operator mutation', async () => {
|
||||
const fastify = Fastify()
|
||||
await fastify.register(swagger)
|
||||
await fastify.register(apophisPlugin)
|
||||
fastify.post('/items', {
|
||||
schema: {
|
||||
body: { type: 'object', properties: { name: { type: 'string' } } },
|
||||
response: { 201: { type: 'object', properties: { id: { type: 'string' } } } },
|
||||
'x-ensures': ['response_code(this) == 201'],
|
||||
},
|
||||
}, async (request, reply) => {
|
||||
reply.status(201)
|
||||
return { id: '1' }
|
||||
})
|
||||
await fastify.ready()
|
||||
|
||||
const mutation: Mutation = {
|
||||
id: 'm1',
|
||||
route: 'POST /items',
|
||||
original: 'response_code(this) == 201',
|
||||
mutated: 'response_code(this) != 201',
|
||||
type: 'flip-operator',
|
||||
}
|
||||
|
||||
const killed = await testMutation(fastify, makeContract(['response_code(this) == 201']), mutation)
|
||||
assert.strictEqual(killed, true)
|
||||
})
|
||||
|
||||
test('runMutationTesting: produces report with score', async () => {
|
||||
const fastify = Fastify()
|
||||
await fastify.register(swagger)
|
||||
await fastify.register(apophisPlugin)
|
||||
fastify.post('/items', {
|
||||
schema: {
|
||||
body: { type: 'object', properties: { name: { type: 'string' } } },
|
||||
response: { 201: { type: 'object', properties: { id: { type: 'string' } } } },
|
||||
'x-ensures': ['response_code(this) == 201'],
|
||||
},
|
||||
}, async (request, reply) => {
|
||||
reply.status(201)
|
||||
return { id: '1' }
|
||||
})
|
||||
await fastify.ready()
|
||||
|
||||
const report = await runMutationTesting(fastify, { runs: 5, seed: 42, maxMutationsPerContract: 3 })
|
||||
assert.ok(typeof report.score === 'number')
|
||||
assert.ok(report.score >= 0 && report.score <= 100)
|
||||
assert.ok(report.mutations.length > 0)
|
||||
assert.ok(report.durationMs > 0)
|
||||
})
|
||||
|
||||
test('runMutationTesting: skips routes without contracts', async () => {
|
||||
const fastify = Fastify()
|
||||
await fastify.register(swagger)
|
||||
await fastify.register(apophisPlugin)
|
||||
fastify.get('/health', {}, async () => 'ok')
|
||||
await fastify.ready()
|
||||
|
||||
const report = await runMutationTesting(fastify, { runs: 5, seed: 42 })
|
||||
assert.strictEqual(report.mutations.length, 0)
|
||||
assert.strictEqual(report.score, 0)
|
||||
})
|
||||
|
||||
test('runMutationTesting: filters by route', async () => {
|
||||
const fastify = Fastify()
|
||||
await fastify.register(swagger)
|
||||
await fastify.register(apophisPlugin)
|
||||
fastify.post('/items', {
|
||||
schema: {
|
||||
body: { type: 'object', properties: { name: { type: 'string' } } },
|
||||
response: { 201: { type: 'object', properties: { id: { type: 'string' } } } },
|
||||
'x-ensures': ['response_code(this) == 201'],
|
||||
},
|
||||
}, async (request, reply) => {
|
||||
reply.status(201)
|
||||
return { id: '1' }
|
||||
})
|
||||
fastify.post('/other', {
|
||||
schema: {
|
||||
body: { type: 'object', properties: { name: { type: 'string' } } },
|
||||
response: { 201: { type: 'object', properties: { id: { type: 'string' } } } },
|
||||
'x-ensures': ['response_code(this) == 201'],
|
||||
},
|
||||
}, async (request, reply) => {
|
||||
reply.status(201)
|
||||
return { id: '2' }
|
||||
})
|
||||
await fastify.ready()
|
||||
|
||||
const report = await runMutationTesting(fastify, { runs: 5, seed: 42, routes: ['/items'] })
|
||||
const itemsMutations = report.mutations.filter((m) => m.mutation.route.includes('/items'))
|
||||
const otherMutations = report.mutations.filter((m) => m.mutation.route.includes('/other'))
|
||||
assert.ok(itemsMutations.length > 0)
|
||||
assert.strictEqual(otherMutations.length, 0)
|
||||
})
|
||||
|
||||
test('runMutationTesting: identifies weak contracts', async () => {
|
||||
const fastify = Fastify()
|
||||
await fastify.register(swagger)
|
||||
await fastify.register(apophisPlugin)
|
||||
fastify.post('/items', {
|
||||
schema: {
|
||||
body: { type: 'object', properties: { name: { type: 'string' } } },
|
||||
response: { 201: { type: 'object', properties: { id: { type: 'string' } } } },
|
||||
'x-ensures': ['response_code(this) == 201', 'response_body(this).id != null'],
|
||||
},
|
||||
}, async (request, reply) => {
|
||||
reply.status(201)
|
||||
return { id: '1' }
|
||||
})
|
||||
await fastify.ready()
|
||||
|
||||
const report = await runMutationTesting(fastify, { runs: 5, seed: 42 })
|
||||
// All mutations should be killed since the handler always returns 201 with id
|
||||
assert.strictEqual(report.weakContracts.length, 0)
|
||||
})
|
||||
@@ -7,7 +7,7 @@ import apophisPlugin from '../index.js'
|
||||
import type { TestResult } from '../types.js'
|
||||
type TestFastifyInstance = FastifyInstance & {
|
||||
apophis: {
|
||||
contract: (opts?: { runs?: number; scope?: string; seed?: number }) => Promise<any>
|
||||
contract: (opts?: { depth?: string; scope?: string; seed?: number }) => Promise<any>
|
||||
spec: () => Record<string, unknown>
|
||||
}
|
||||
}
|
||||
@@ -44,19 +44,19 @@ test('scope isolation: routes with x-scope are filtered by scope parameter', asy
|
||||
}, async () => ({ user: true }))
|
||||
await fastify.ready()
|
||||
// Test with no scope - should discover all 3 routes
|
||||
const allResult = await fastify.apophis.contract({ runs: 10, scope: undefined })
|
||||
const allResult = await fastify.apophis.contract({ depth: 'quick', scope: undefined })
|
||||
const allPaths = new Set(allResult.tests.map((t: TestResult) => t.name.split(' ')[1]))
|
||||
assert.ok(allPaths.has('/public'), 'public route should be in all scope')
|
||||
assert.ok(allPaths.has('/admin'), 'admin route should be in all scope')
|
||||
assert.ok(allPaths.has('/user'), 'user route should be in all scope')
|
||||
// Test with admin scope - should only get public + admin
|
||||
const adminResult = await fastify.apophis.contract({ runs: 10, scope: 'admin' })
|
||||
const adminResult = await fastify.apophis.contract({ depth: 'quick', scope: 'admin' })
|
||||
const adminPaths = new Set(adminResult.tests.map((t: TestResult) => t.name.split(' ')[1]))
|
||||
assert.ok(adminPaths.has('/public'), 'public route should be in admin scope')
|
||||
assert.ok(adminPaths.has('/admin'), 'admin route should be in admin scope')
|
||||
assert.ok(!adminPaths.has('/user'), 'user route should NOT be in admin scope')
|
||||
// Test with user scope - should only get public + user
|
||||
const userResult = await fastify.apophis.contract({ runs: 10, scope: 'user' })
|
||||
const userResult = await fastify.apophis.contract({ depth: 'quick', scope: 'user' })
|
||||
const userPaths = new Set(userResult.tests.map((t: TestResult) => t.name.split(' ')[1]))
|
||||
assert.ok(userPaths.has('/public'), 'public route should be in user scope')
|
||||
assert.ok(!userPaths.has('/admin'), 'admin route should NOT be in user scope')
|
||||
@@ -88,7 +88,7 @@ test('scope isolation: scope headers are passed to requests', async () => {
|
||||
headers: { 'x-custom-header': 'test-value' },
|
||||
metadata: {}
|
||||
})
|
||||
await fastify.apophis.contract({ runs: 10, scope: 'test' })
|
||||
await fastify.apophis.contract({ depth: 'quick', scope: 'test' })
|
||||
assert.strictEqual(receivedHeaders['x-custom-header'], 'test-value', 'scope header should be passed to request')
|
||||
} finally {
|
||||
await fastify.close()
|
||||
@@ -130,7 +130,7 @@ test('scope isolation: non-matching scope returns empty test suite', async () =>
|
||||
}, async () => ({ ok: true }))
|
||||
await fastify.ready()
|
||||
// Test with non-matching scope
|
||||
const result = await fastify.apophis.contract({ runs: 10, scope: 'other' })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick', scope: 'other' })
|
||||
assert.strictEqual(result.tests.length, 0, 'no tests should run for non-matching scope')
|
||||
assert.strictEqual(result.summary.passed, 0, 'no tests should pass')
|
||||
assert.strictEqual(result.summary.failed, 0, 'no tests should fail')
|
||||
|
||||
@@ -32,7 +32,7 @@ test('serverless: fastify.ready() without listen works', async () => {
|
||||
await fastify.ready()
|
||||
|
||||
// Should be able to run tests
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should have tests')
|
||||
|
||||
// Should be able to get spec
|
||||
@@ -115,7 +115,7 @@ test('serverless: multiple ready() calls are safe', async () => {
|
||||
// Second ready() should be safe (idempotent)
|
||||
await fastify.ready()
|
||||
|
||||
const result = await fastify.apophis.contract({ runs: 10 })
|
||||
const result = await fastify.apophis.contract({ depth: 'quick' })
|
||||
assert.ok(result.tests.length > 0, 'should still work after multiple ready() calls')
|
||||
} finally {
|
||||
await fastify.close()
|
||||
|
||||
@@ -22,7 +22,7 @@ test('stateful runner handles empty routes', async () => {
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -62,7 +62,7 @@ test('stateful runner executes commands', async () => {
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -103,7 +103,7 @@ test('stateful runner detects status code violations', async () => {
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -139,7 +139,7 @@ test('stateful runner evaluates APOSTL formulas', async () => {
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -189,7 +189,7 @@ test('stateful runner tracks resource state', async () => {
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -264,7 +264,7 @@ test('stateful runner substitutes path params from resource state', async () =>
|
||||
await fastify.ready()
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
@@ -308,7 +308,7 @@ test('stateful runner supports config-level variants', async () => {
|
||||
await fastify.ready()
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
variants: [
|
||||
@@ -359,7 +359,7 @@ test('stateful runner supports route-level x-variants', async () => {
|
||||
await fastify.ready()
|
||||
|
||||
const result = await runStatefulTests(fastify as any, {
|
||||
runs: 10,
|
||||
depth: 'quick',
|
||||
scope: undefined,
|
||||
seed: 42,
|
||||
})
|
||||
|
||||
+10
-27
@@ -6,7 +6,7 @@
|
||||
* generate → execute → validate → update → check-invariants
|
||||
*/
|
||||
import type { ExtensionRegistry } from '../extension/types.js'
|
||||
import { resolveRuns } from '../types.js'
|
||||
import { resolveDepth, resolveGenerationProfile } from '../types.js'
|
||||
import { discoverRoutes } from '../domain/discovery.js'
|
||||
import { convertSchema } from '../domain/schema-to-arbitrary.js'
|
||||
import { SeededRng } from '../infrastructure/seeded-rng.js'
|
||||
@@ -22,7 +22,7 @@ import type { OutboundContractRegistry } from '../domain/outbound-contracts.js'
|
||||
import * as fc from 'fast-check'
|
||||
import type { ModelState } from '../domain/stateful.js'
|
||||
import type { CleanupManager } from '../infrastructure/cleanup-manager.js'
|
||||
import type { EvalContext, FastifyInjectInstance, RouteContract, ScopeRegistry, TestConfig, TestResult, TestSuite } from '../types.js'
|
||||
import type { DepthConfig, EvalContext, FastifyInjectInstance, RouteContract, ScopeRegistry, TestConfig, TestResult, TestSuite } from '../types.js'
|
||||
|
||||
// Pure: hash helpers for deterministic sub-seeds
|
||||
// ---------------------------------------------------------------------------
|
||||
@@ -61,6 +61,7 @@ class ApiOperation implements StatefulApiOperation {
|
||||
// ============================================================================
|
||||
const createCommandArbitrary = (
|
||||
routes: RouteContract[],
|
||||
generationProfile: 'quick' | 'standard' | 'thorough',
|
||||
): { arb: fc.Arbitrary<ApiOperation>, cacheHits: number, cacheMisses: number } => {
|
||||
let cacheHits = 0
|
||||
let cacheMisses = 0
|
||||
@@ -73,7 +74,7 @@ const createCommandArbitrary = (
|
||||
cacheMisses++
|
||||
const bodySchema = route.schema?.body as Record<string, unknown> | undefined
|
||||
const arb = bodySchema !== undefined
|
||||
? convertSchema(bodySchema, { context: 'request' })
|
||||
? convertSchema(bodySchema, { context: 'request', generationProfile })
|
||||
: fc.constant({})
|
||||
return arb.map((params) => new ApiOperation(route, params as Record<string, unknown>))
|
||||
})
|
||||
@@ -92,7 +93,8 @@ export const runStatefulTests = async (
|
||||
outboundContractRegistry?: OutboundContractRegistry
|
||||
): Promise<TestSuite> => {
|
||||
const startTime = Date.now()
|
||||
const runConfig = resolveRuns(config.runs)
|
||||
const depth = resolveDepth(config.depth ?? 'standard')
|
||||
const generationProfile = config.generationProfile ?? resolveGenerationProfile(config.depth)
|
||||
if (extensionRegistry) {
|
||||
await extensionRegistry.runSuiteStartHooks(config)
|
||||
}
|
||||
@@ -100,26 +102,6 @@ export const runStatefulTests = async (
|
||||
// Skip HEAD routes — auto-generated by Fastify for GET routes, no response body
|
||||
const filteredRoutes = allRoutes.filter((r) => r.category !== 'utility' && r.method !== 'HEAD')
|
||||
const routes = filterByScope(filteredRoutes, config.scope)
|
||||
|
||||
// Merge plugin contracts into route contracts
|
||||
if (pluginContractRegistry) {
|
||||
for (const route of routes) {
|
||||
const composed = pluginContractRegistry.composeContracts(route)
|
||||
for (const phase of Object.values(composed.phases)) {
|
||||
for (const req of phase.requires) {
|
||||
if (!route.requires.includes(req.formula)) {
|
||||
route.requires.push(req.formula)
|
||||
}
|
||||
}
|
||||
for (const ens of phase.ensures) {
|
||||
if (!route.ensures.includes(ens.formula)) {
|
||||
route.ensures.push(ens.formula)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (routes.length === 0) {
|
||||
return {
|
||||
tests: [],
|
||||
@@ -149,7 +131,7 @@ export const runStatefulTests = async (
|
||||
variantName ? `[variant:${variantName}] ${name}` : name
|
||||
// Get scope headers for test requests
|
||||
const baseScopeHeaders = scopeRegistry?.getHeaders(config.scope ?? null) ?? {}
|
||||
const { arb: commandArb, cacheHits, cacheMisses } = createCommandArbitrary(routes)
|
||||
const { arb: commandArb, cacheHits, cacheMisses } = createCommandArbitrary(routes, generationProfile)
|
||||
let allResults: TestResult[] = []
|
||||
let globalTestId = 0
|
||||
// Create seeded RNG for reproducible path param selection
|
||||
@@ -176,6 +158,7 @@ export const runStatefulTests = async (
|
||||
suiteMockRuntime = createOutboundMockRuntime({
|
||||
contracts: allResolved,
|
||||
mode: config.outboundMocks?.mode ?? 'example',
|
||||
generationProfile,
|
||||
overrides: config.outboundMocks?.overrides,
|
||||
unmatched: config.outboundMocks?.unmatched ?? 'error',
|
||||
seed: outboundSeed,
|
||||
@@ -183,7 +166,7 @@ export const runStatefulTests = async (
|
||||
suiteMockRuntime.install()
|
||||
}
|
||||
// Run property-based stateful tests per variant
|
||||
const numRuns = runConfig.statefulRuns
|
||||
const numRuns = depth.statefulRuns
|
||||
const seed = config.seed
|
||||
let counterexampleOutput: string | undefined
|
||||
const hashString = (s: string): number => {
|
||||
@@ -273,7 +256,7 @@ export const runStatefulTests = async (
|
||||
}
|
||||
try {
|
||||
const prop = fc.asyncProperty(
|
||||
fc.array(commandArb, { minLength: 1, maxLength: runConfig.maxCommands }),
|
||||
fc.array(commandArb, { minLength: 1, maxLength: depth.maxCommands }),
|
||||
async (cmds) => {
|
||||
await runSequence(cmds)
|
||||
return true
|
||||
|
||||
@@ -22,7 +22,7 @@ import type {
|
||||
TestConfig,
|
||||
TestResult,
|
||||
} from '../types.js'
|
||||
import { resolveRuns } from '../types.js'
|
||||
import { resolveDepth, resolveGenerationProfile } from '../types.js'
|
||||
|
||||
export const runTripleBoundaryPropertyTest = async (
|
||||
route: RouteContract,
|
||||
@@ -39,9 +39,10 @@ export const runTripleBoundaryPropertyTest = async (
|
||||
if (!config.chaos) return []
|
||||
|
||||
const results: TestResult[] = []
|
||||
const arbitrary = createTripleBoundaryArbitrary(route, contracts, config.chaos)
|
||||
const runConfig = resolveRuns(config.runs)
|
||||
const numRuns = Math.max(10, Math.floor(runConfig.contractRuns / 2))
|
||||
const generationProfile = config.generationProfile ?? resolveGenerationProfile(config.depth)
|
||||
const arbitrary = createTripleBoundaryArbitrary(route, contracts, config.chaos, generationProfile)
|
||||
const depth = resolveDepth(config.depth ?? 'standard')
|
||||
const numRuns = Math.max(10, Math.floor(depth.contractRuns / 2))
|
||||
|
||||
const property = fc.asyncProperty(arbitrary, async (cmd) => {
|
||||
const testId = testIdBase + results.length + 1
|
||||
|
||||
+5
-2
@@ -39,15 +39,18 @@ export type {
|
||||
|
||||
// Formula and test configuration types
|
||||
export type {
|
||||
TestDepth,
|
||||
TestConfig,
|
||||
ResolvedOutboundContract,
|
||||
OutboundChaosConfig,
|
||||
ChaosConfig,
|
||||
RunConfig,
|
||||
DepthConfig,
|
||||
} from './types/formula.js'
|
||||
|
||||
export {
|
||||
resolveRuns,
|
||||
DEPTH_CONFIGS,
|
||||
resolveDepth,
|
||||
resolveGenerationProfile,
|
||||
} from './types/formula.js'
|
||||
|
||||
// Extension types
|
||||
|
||||
+1
-1
@@ -201,7 +201,7 @@ export interface ApophisTestDecorations {
|
||||
|
||||
// Forward declarations to avoid circular deps — these are defined in sibling modules
|
||||
export interface TestConfig {
|
||||
readonly runs?: number
|
||||
readonly depth?: import('./formula.js').TestDepth
|
||||
readonly scope?: string
|
||||
readonly seed?: number
|
||||
readonly timeout?: number
|
||||
|
||||
+35
-11
@@ -7,8 +7,11 @@
|
||||
// Test: Configuration
|
||||
// ============================================================================
|
||||
|
||||
export type TestDepth = 'quick' | 'standard' | 'thorough' | { runs: number }
|
||||
|
||||
export interface TestConfig {
|
||||
readonly runs?: number
|
||||
readonly depth?: TestDepth
|
||||
readonly generationProfile?: GenerationProfile
|
||||
readonly scope?: string
|
||||
readonly seed?: number
|
||||
readonly timeout?: number
|
||||
@@ -201,26 +204,47 @@ export interface ChaosConfig {
|
||||
}
|
||||
|
||||
// ============================================================================
|
||||
// Test run configuration
|
||||
// Depth Configuration
|
||||
// ============================================================================
|
||||
|
||||
export interface RunConfig {
|
||||
export interface DepthConfig {
|
||||
readonly contractRuns: number
|
||||
readonly propertyRuns: number
|
||||
readonly statefulRuns: number
|
||||
readonly maxCommands: number
|
||||
}
|
||||
|
||||
const DEFAULT_RUNS = 50
|
||||
export type GenerationProfile = 'quick' | 'standard' | 'thorough'
|
||||
|
||||
export function resolveRuns(runs: number | undefined): RunConfig {
|
||||
const r = runs ?? DEFAULT_RUNS
|
||||
return {
|
||||
contractRuns: r,
|
||||
propertyRuns: r * 2,
|
||||
statefulRuns: Math.max(1, Math.floor(r / 10)),
|
||||
maxCommands: Math.max(5, Math.floor(r / 2)),
|
||||
export const DEPTH_CONFIGS: Record<'quick' | 'standard' | 'thorough', DepthConfig> = {
|
||||
quick: { contractRuns: 10, propertyRuns: 50, statefulRuns: 5, maxCommands: 10 },
|
||||
standard: { contractRuns: 50, propertyRuns: 100, statefulRuns: 20, maxCommands: 30 },
|
||||
thorough: { contractRuns: 200, propertyRuns: 1000, statefulRuns: 100, maxCommands: 50 }
|
||||
}
|
||||
|
||||
export function resolveDepth(depth: TestDepth): DepthConfig {
|
||||
if (typeof depth === 'string') {
|
||||
return DEPTH_CONFIGS[depth]
|
||||
}
|
||||
return {
|
||||
contractRuns: depth.runs,
|
||||
propertyRuns: depth.runs,
|
||||
statefulRuns: Math.max(1, Math.floor(depth.runs / 10)),
|
||||
maxCommands: Math.max(5, Math.floor(depth.runs / 5)),
|
||||
}
|
||||
}
|
||||
|
||||
export function resolveGenerationProfile(depth: TestDepth | undefined): GenerationProfile {
|
||||
if (depth === undefined) {
|
||||
return 'standard'
|
||||
}
|
||||
if (typeof depth === 'string') {
|
||||
return depth
|
||||
}
|
||||
|
||||
if (depth.runs <= 25) return 'quick'
|
||||
if (depth.runs >= 250) return 'thorough'
|
||||
return 'standard'
|
||||
}
|
||||
|
||||
// ============================================================================
|
||||
|
||||
Reference in New Issue
Block a user