[MRG + 1] Test UsageError for CLI

[vaibhavmule]

referencing: #110

[codecov-io]

Codecov Report

Merging #122 into master will increase coverage by 0.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   73.19%   73.84%   +0.64%     
==========================================
  Files          12       12              
  Lines        1231     1231              
  Branches      292      292              
==========================================
+ Hits          901      909       +8     
+ Misses        283      279       -4     
+ Partials       47       43       -4

Impacted Files	Coverage Δ
camelot/cli.py	93.47% <0%> (+8.69%)	⬆️

---

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 9d00937...1721cbb. Read the comment docs.

[vinayak-mehta]

@vaibhavmule Is this a WIP or ready for merge? Can you name the title accordingly, I'll add my review after it is ready.

EDIT: You should check out the PR contribution guidelines.

[vaibhavmule]

@vinayak-mehta it is ready for review.

[vinayak-mehta]

LGTM. In the future, we might want to break these into separate tests.

[vinayak-mehta]

@vaibhavmule Thanks for the contribution!

[vaibhavmule]

@vinayak-mehta
Welcome, Yes, deff tests need to be broken down.

Also, I tried using --plot_type but it was giving error Error: no such option: --plot_type

[vaibhavmule]

BTW, what is +1 in title?

[vinayak-mehta]

I've taken the MRG philosophy from scikit-learn's contributing guidelines, which states that at least 2 +1s on the MRG label are needed (from core devs) to merge a PR. Since I'm the only one here right now, only a +1 suffices. This isn't in the docs right now, want to add it?

[vinayak-mehta]

Also, I tried using --plot_type but it was giving error Error: no such option: --plot_type

Strange, what were the exact arguments that you used? It's working for me.