[Biopython-dev] used local in Tests/test_Nexus.py
chapmanb at 50mail.com
Fri Dec 4 15:27:06 UTC 2015
Vincent and Peter;
My guess is the tests there make sure it can read the ts2 tree, then the
int_node_labels.nwk file. We don't really need to assign to the `tree` local
variable since we don't do anything with it. As Peter said, separating the
tests or adding some assertions would help make this more clear.
> Hi Vincent,
> Did you mean a reused local variable in the email title?
> It looks like Brad added this test some time back,
> Certainly the final int_node_labels.nwk snippet seems like
> it could be a separate test method, perhaps expanded with
> a sanity test like the number of leaves?
> On Fri, Dec 4, 2015 at 1:09 AM, Vincent Davis <vincent at vincentdavis.net> wrote:
>> Posting this here just to be sure I am not missing something.
>> Starting at lin
>> tree = Trees.Tree(ts2)
>> large_ex_handle = open(os.path.join(self.testfile_dir,
>> tree = Trees.Tree(large_ex_handle.read())
>> Vincent Davis
>> Biopython-dev mailing list
>> Biopython-dev at mailman.open-bio.org
More information about the Biopython-dev