https://issues.apache.org/bugzilla/show_bug.cgi?id=50391

Chris Bowditch <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #4 from Chris Bowditch <[email protected]> 2012-01-10 
14:54:18 UTC ---
Hi Peter,

Thanks for submitting this patch. I have reviewed it and found a couple of
issues. There were some minor conflicts due to the age of the patch that I
resolved. I also had to update the tests to Junit4. My question is around the
AllTests class added to the pagination package. 2 test were added to this
package:

PageSequenceMasterTestCase
RepeatablePageMasterAlternativesTestCase

but the AllTest class only references PageSequenceMasterTestCase. Is that
intentional or should it also reference the
RepeatablePageMasterAlternativesTestCase?

Thanks,

Chris

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to