Closed ssnn-airr closed 9 years ago
Original comment by Jason Vander Heiden (Bitbucket: javh, GitHub: javh).
Thanks. Just FYI - the unit tests are still very rough. Coverage is pretty poor. I just started add tests whenever I updated/added functions with the intention of slowly fill them up. The tests in:
Are currently hardcoded to fail with self.fail()
. For now, don't worry about the AssemblePairs tests. They need some cleanup. The only one of real concern is test_calculateSetError
in tests_BuildConsensus.py
Original report by David Koppstein (Bitbucket: dkoppstein, GitHub: dkoppstein).
Hi Jason,
I noticed that the scripts haven't been converted to python 3 (although presto can be successfully imported), so I tried using 2to3 to convert the whole package.
2to3 --output-dir=presto3 -W -n presto
After that, I tried running your test suite, but noticed that there were some errors.
I then went back to the old python 2 version, and found that some of the tests were still failing.
I think some of these can be attributed to the test environment, i.e. not finding the IG_TR file. I'll keep digging and updating this issue.