baccuslab / pyret

Python tools for analysis of neurophysiology data
https://pyret.readthedocs.io/en/master/
MIT License
35 stars 8 forks source link

Changes testing strategy for the visualization module. #92

Closed bnaecker closed 7 years ago

bnaecker commented 7 years ago

Previous versions tested the module by comparing images. The tests generated an image, which was compared pixel-to-pixel with a baseline version saved in the repository. This is complicated, error-prone, and difficult to maintain.

This commit now tests the module by calling the functions with simple and known data, and then verifying that the data in the actual plot objects rendered matches our expectations based on the inputs.

codecov-io commented 7 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4632548). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #92   +/-   ##
=========================================
  Coverage          ?   95.83%           
=========================================
  Files             ?        7           
  Lines             ?      528           
  Branches          ?        0           
=========================================
  Hits              ?      506           
  Misses            ?       22           
  Partials          ?        0

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 4632548...16b29d1. Read the comment docs.