[ 
https://issues.apache.org/jira/browse/HBASE-21272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16640551#comment-16640551
 ] 

Andrew Purtell commented on HBASE-21272:
----------------------------------------

Its fine to put them back. I removed them because these are now utility methods 
and in my judgment it’s weird to have assertions in utility methods. They 
should be in the units instead. That said certainly it doesn’t hurt to have 
them. 

> Re-add assertions for RS Group admin tests
> ------------------------------------------
>
>                 Key: HBASE-21272
>                 URL: https://issues.apache.org/jira/browse/HBASE-21272
>             Project: HBase
>          Issue Type: Test
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>            Priority: Minor
>             Fix For: 1.5.0
>
>         Attachments: 21272.branch-1.01.patch
>
>
> The checked in version of HBASE-21258 for branch-1 didn't include assertions 
> for adding / removing RS group coprocessor hook calls.
> This issue is to add the assertions to corresponding tests in 
> TestRSGroupsAdmin1



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to