search

biocore / labadmin

Administration website for the Knight Lab
4 stars 16 forks source link

added new unit tests for the get method of handler "BarcodeUtilHandler" #159

Closed sjanssen2 closed 7 years ago

sjanssen2 commented 7 years ago

In order to address #158 I'd like to have unit tests in place. I think I found some odd things in the code and will correct them after the test PR has been accepted. Until now, I haven't touched the handler sources other than adding TODOs.

wasade commented 7 years ago

A flake8 error and it looks like a timeout again :/

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 85d53b3d9726cc6b44f79ab7a6816868a7ec5721 on sjanssen2:unittest_barcode_util into \ on biocore:master**.

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling d515e52b967996540d4af15c4f9307d0b5553415 on sjanssen2:unittest_barcode_util into \ on biocore:master**.

wasade commented 7 years ago

Just one question about the 599