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

Ivan A. Veselovsky commented on HBASE-8584:
-------------------------------------------

Problem this is trying to solve is to provide >= 80% coverage of package 
org.apache.hadoop.hbase.coprocessor in HBase branches 0.94, 0.95 and trunk. 
This is the task we are assigned to solve.
As written in the suggested test description, here we only test that the class 
BaseMasterObserver can be directly instantiated and that all its methods can be 
invoked with default parameters without exceptions, nothing else.
Real MasterObserver implementations extending BaseMasterObserver never invoke 
super.xxx(), and this is correct, because all BaseMasterObserver methods are 
empty.
There is test TestMasterObserver that checks that the methods of MasterObserver 
are really invoked when needed.
So, a reasonable way to cover empty methods of BaseMasterObserver is to invoke 
them directly.
                
> increase unit-test coverage of class 
> org.apache.hadoop.hbase.coprocessor.BaseMasterObserver
> -------------------------------------------------------------------------------------------
>
>                 Key: HBASE-8584
>                 URL: https://issues.apache.org/jira/browse/HBASE-8584
>             Project: HBase
>          Issue Type: Test
>    Affects Versions: 0.98.0, 0.95.1, 0.94.9
>            Reporter: Ivan A. Veselovsky
>         Attachments: HBASE-8584-0.94--N1.patch, HBASE-8584-trunk--N1.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to