Agreed.


On Thu, Nov 7, 2013 at 10:39 AM, bcail <notifications@github.com> wrote:

> In test/rights_test.py:
>
> > @@ -67,36 +131,97 @@ def test_get_context(self):
> >          self.init_holder()
> >          tmp_ctext = self.rights.get_ctext_for("johnny@brown.edu")
> >          self.assertEqual(tmp_ctext.id, "rights3")
> > +
> > +    def test_index_data(self):
> > +        self.init_context("rights1", 'jack@brown.edu')
> > +        self.init_context("rights2", 'jim@brown.edu')
> > +        self.init_context("rights3", 'johnny@brown.edu')
> > +        self.assertEqual( {
> > +            'discover': [],
> > +            'display': [],
> > +            'modify': [ 'jack@brown.edu', 'jim@brown.edu', 'johnny@brown.edu'],
> > +            'delete': [ 'jack@brown.edu', 'jim@brown.edu', 'johnny@brown.edu'],
> > +            },self.rights.index_data())
> > +
> > +    def test_index_data_hydra(self):
> > +        # TODO: write code...
>
> looks like this TODO can be removed
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Brown-University-Library/bdrxml/pull/19/files#r7498931>
> .
>