add ::testing #72
add ::testing #72
Conversation
|
@laggycomputer -- there are several things to note here:
|
| use crate::MultiSiteCounts; | ||
|
|
||
| #[test] | ||
| fn watterson_theta() { |
There was a problem hiding this comment.
This test is moot given the two below and can be deleted.
| // once we have a naive implementation in place | ||
| // and can test using our random data API. | ||
| #[test] | ||
| fn tajima_d() { |
There was a problem hiding this comment.
This test will be replaced by comparisons to naive impls but should be left for now.
| use std::iter::repeat_n; | ||
|
|
||
| #[test] | ||
| fn load_raw() { |
There was a problem hiding this comment.
We can probably come up with a version of this that uses our testdata mod, but that would be a later PR.
|
Took a look at the commented out pi test(s), figured we want to remove that quickly. We can proceed with deleting that. |
@laggycomputer -- you mean the one in test.rs? If so, that should be removed. The intent is to remove commented out code unless you have an objection at this point. We can actually delete all of test.rs. |
Move code into a testing module that is hopefully nicely-organized.