Additional vector tests

[nkorinek]

Have to merge https://github.com/earthlab/matplotcheck/pull/203 before this one.

[codecov[bot]]

Codecov Report

Merging #212 into master will not change coverage by %. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   77.61%   77.61%           
=======================================
  Files          18       18           
  Lines        1747     1747           
=======================================
  Hits         1356     1356           
  Misses        391      391           

---

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 44f4334...44f4334. Read the comment docs.

[mbjoseph]

Heya @nkorinek - not 100% sure whether this is a fix, but do you need to set the matplotlib backend to get these commented out tests to work? e.g., in test_vector.py at the top:

... # other imports
import matplotlib
matplotlib.use('Agg')

# awesome tests
...

[nkorinek]

@lwasser Alright I think I'm happy with how much of the code is tested now! Let me know what you think and if I need to look into adding anymore vector tests.

[lwasser]

Heya @nkorinek - not 100% sure whether this is a fix, but do you need to set the matplotlib backend to get these commented out tests to work? e.g., in test_vector.py at the top:

... # other imports
import matplotlib
matplotlib.use('Agg')

# awesome tests
...

THANK YOU @mbjoseph !! so appreciative of your help!

[nkorinek]

Ok @lwasser , I went through and renamed variables you pointed out had confusing names, and hopefully clarified a lot of the confusion. I think this is ready for another review!

[lwasser]

ok once CI finishes I will merge this!!