Some testing improvements

Still familiarizing myself with the code, making the test infrastructure work for me
On Fri, Aug 29, 2014 at 8:19 AM, Luca Barbato <notifications@github.com>
wrote:

> In test/test_mailbox.py:
>
> >
> > -    def test_find_project_name(self):
> > -        self.assertEqual('libav-devel.libav.org',
> > -                         find_project_name(self.email1))
> > -        self.assertEqual('libav-devel.libav.org',
> > -                         find_project_name(self.email2))
> > +    @classmethod
> > +    def tearDownClass(cls):
> > +        for filename in ('test/data/libav_ancestor.mbox',):
>
> The data set must be kept.
>

​It doesn't exist in the branch. Did you forget to commit it? If not, then
why should it be kept? I would argue that tests should always clean up
after themselves.