search

FundingCircle / jackdaw

A Clojure library for the Apache Kafka distributed streaming platform.
https://fundingcircle.github.io/jackdaw/
BSD 3-Clause "New" or "Revised" License
369 stars 80 forks source link

ensure that the command given in exists #182

Closed AndreaCrotti closed 5 years ago

AndreaCrotti commented 5 years ago

It's easy to mispell a command, this PR simply throws an exception if any of the commands is unknown to jackdaw.

codecov[bot] commented 5 years ago

Codecov Report

Merging #182 into master will decrease coverage by 0.48%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #182      +/-   ##
==========================================
- Coverage   81.46%   80.98%   -0.49%     
==========================================
  Files          39       39              
  Lines        2369     2366       -3     
  Branches      149      149              
==========================================
- Hits         1930     1916      -14     
- Misses        290      301      +11     
  Partials      149      149
Impacted Files Coverage Δ
src/jackdaw/test/commands.clj 100% <100%> (+13.04%) :arrow_up:
src/jackdaw/test/fixtures.clj 55.33% <0%> (-11.55%) :arrow_down:
src/jackdaw/serdes/edn.clj 88.46% <0%> (+7.69%) :arrow_up:

Continue to review full report at Codecov.

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