Absolutely, there shouldn't be any print statements in the tests.
And there should be asserts where you indicated.

-Joseph


On Thu, Nov 7, 2013 at 10:33 AM, bcail <notifications@github.com> wrote:

> In test/rights_test.py:
>
> >          self.builder.addReader('jack@brown.edu')
> >          self.builder.addReader('jack@brown.edu')
> >          self.builder.addEditor('jim@brown.edu')
> >          self.builder.addReader('johnny@brown.edu').addDiscoverer('johnny@brown.edu')
> >          rights = self.builder.build()
> >          rights_str = rights.serialize(pretty=True)
> > +        print rights_str
>
> remove the print statement?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Brown-University-Library/bdrxml/pull/19/files#r7498702>
> .
>