Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24034
to look at the new patch set (#2).
Change subject: Add swagger verification test target
......................................................................
Add swagger verification test target
Wire the swagger spec checker into the master test suite via a
script wrapper so it runs with regular test execution.
Alternatives considered:
- Standalone custom target (e.g. verify_swagger_spec): lower risk but opt‑in,
so it’s easy to miss in day‑to‑day testing.
- Top‑level docs target (like doxygen): runs too infrequently.
- CI‑only hook: ensures coverage in CI but not in local dev runs.
Chosen approach:
- Add a test wrapper under the master tests so it runs regularly with
ctest/test suites while keeping the checker isolated and lightweight.
Manual testing:
- ctest -R verify_swagger_spec-test
Change-Id: I4f5523eb1fa6792cd81e469169962704d0d27e3b
---
M src/kudu/master/CMakeLists.txt
A src/kudu/master/verify_swagger_spec-test.sh
2 files changed, 42 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/24034/2
--
To view, visit http://gerrit.cloudera.org:8080/24034
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4f5523eb1fa6792cd81e469169962704d0d27e3b
Gerrit-Change-Number: 24034
Gerrit-PatchSet: 2
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
