search

glue-viz / glue-plotly

Experimental plot.ly plugin for glue
BSD 3-Clause "New" or "Revised" License
2 stars 8 forks source link

Improve test suite #34

Closed Carifio24 closed 5 months ago

Carifio24 commented 1 year ago

This is just a reminder to myself that we should improve the test suite. This is particularly true for the HTML exporters, whose tests are just basic smoketests.

With the refactoring of the various export functionality that's currently almost complete, we should be better able to write unit tests for individual pieces of the export - this is good since I'm not sure whether it will be feasible to test e.g. the output HTML.

Carifio24 commented 5 months ago

With #36 and #38 (and maybe in some other PRs as well) we now do have unit testing for most of the comnon functionality.