Skip to content

test(inventory): report boundary projection timeout state (#157) - #374

Merged
smiggleworth merged 1 commit into
mainfrom
chore/application-inventory-projection-diagnostics
Sep 23, 2026
Merged

smiggleworth merged 1 commit into
mainfrom
chore/application-inventory-projection-diagnostics

Conversation

@smiggleworth

@smiggleworth smiggleworth commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add failure-only diagnostics to the existing split API/restarted-worker Application Inventory broker test. If the linked boundary still is not readable after the existing 45-second gate, the assertion reports the last GET status and expected error body, source boundary revision, minimum-revision read, earlier boundary view presence, BoundaryDirectoryV2 checkpoint start/change flags, and bounded worker fault metadata.
  • Keep the test sequence and timeout unchanged. The output omits unexpected response bodies and prints only worker error category, event, runner, stage, and type metadata.
  • This distinguishes projection startup/catch-up from missing source or worker faults if the intermittent failure recurs. It does not assert a root cause or change runtime behavior. The same existing assertion failed in hosted CI on runs 35910032416, 35911891833, and 35914953984.

Validation

  • Client check — exact-head Validate passed.
  • Format check — dotnet format Compliance.slnx --verify-no-changes --no-restore --include test/Compliance.Tests/E2E/ApplicationInventoryE2ETests.cs passed on local head 292bf91.
  • Release test build — dotnet build test/Compliance.Tests/Compliance.Tests.csproj -c Release --no-restore --no-dependencies passed with zero warnings.
  • Focused broker test — ./scripts/check-backend.sh focused 'FullyQualifiedName~ApplicationInventoryE2ETests.ShouldProjectDeclarationsGivenSplitApiAndRestartedWorker' passed 1/1 on 292bf91bd78923686a8c41c43cd3e8144e883a34.
  • Exact-head CI run 35917266316 passed Validate (including full broker), dependency review, and native AMD64/ARM64 at 292bf91bd78923686a8c41c43cd3e8144e883a34.

Contracts and operations

  • Authentication and authorization effects considered — no policy or request behavior changed.
  • API, persistence, scheduling, and wire compatibility considered — test-only change; no runtime contract, event, projection, schema, or worker scheduling change.
  • Deployment, health, logging, and rollback effects described — no deployment or configuration change. The extra diagnostic reads and worker log capture run in this isolated test only.

@smiggleworth
smiggleworth merged commit 3daffb0 into main Sep 23, 2026
4 checks passed
@smiggleworth
smiggleworth deleted the chore/application-inventory-projection-diagnostics branch September 23, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant