search

roadster-rs / roadster

"Batteries Included" web framework for Rust designed to get you moving fast 🏎️
Apache License 2.0
3 stars 0 forks source link

Implement the health check API as a protocol agnostic `core` module #212

Closed spencewenski closed 3 weeks ago

spencewenski commented 3 weeks ago

Move the underlying implementation of the _health HTTP API to the core module and update the HTTP method to call the core impl. Also, add support for invoking the health check API from the CLI using the same underlying core impl.

Closes https://github.com/roadster-rs/roadster/issues/170

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 44.13%. Comparing base (4196c0a) to head (a3521cb).

Files Coverage Δ
src/api/http/health.rs 29.33% <ø> (+2.50%) :arrow_up:
src/api/cli/roadster/mod.rs 0.00% <0.00%> (ø)
src/api/core/health.rs 0.00% <0.00%> (ø)
src/api/cli/roadster/health.rs 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4196c0a...a3521cb. Read the comment docs.