Skip to content

Add Xquik request and response validation fixture#299

Closed
kriptoburak wants to merge 1 commit into
pb33f:mainfrom
kriptoburak:codex/add-xquik-validation-fixture
Closed

Add Xquik request and response validation fixture#299
kriptoburak wants to merge 1 commit into
pb33f:mainfrom
kriptoburak:codex/add-xquik-validation-fixture

Conversation

@kriptoburak

Copy link
Copy Markdown

Summary

  • Adds a compact Xquik OpenAPI search fixture to validator tests.
  • Verifies document validation, request validation, API key security, and response validation.

Validation

  • go test . -run TestXquikSearchFixture_ValidateRequestAndResponse
  • go test ./...

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.97%. Comparing base (e8302f2) to head (75dae8e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   97.97%   97.97%           
=======================================
  Files          67       67           
  Lines        7670     7670           
=======================================
  Hits         7515     7515           
  Misses        127      127           
  Partials       28       28           
Flag Coverage Δ
unittests 97.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@daveshanley

Copy link
Copy Markdown
Member

I can't see what value this adds?

@kriptoburak

Copy link
Copy Markdown
Author

Thanks for the direct feedback. I agree this is not adding enough maintainer value as a branded fixture, so I am closing it rather than keeping noise in your queue.

@kriptoburak kriptoburak closed this Jul 2, 2026
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.

2 participants