[
https://issues.apache.org/jira/browse/HBASE-18830?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chinmay Kulkarni updated HBASE-18830:
-------------------------------------
Attachment: HBASE-18830.001.patch
Added assertion checks to make sure that the error code for the
ToolRunner run() method is used.
Testing Done: Checked that TestCanaryTool unit tests fail when there is
an error code in the current Canary run.
[~apurtell] [[email protected]] please review.
> TestCanaryTool does not check Canary monitor's error code
> ---------------------------------------------------------
>
> Key: HBASE-18830
> URL: https://issues.apache.org/jira/browse/HBASE-18830
> Project: HBase
> Issue Type: Bug
> Reporter: Chinmay Kulkarni
> Assignee: Chinmay Kulkarni
> Attachments: HBASE-18830.001.patch
>
>
> None of the tests inside TestCanaryTool check Canary monitor's error code.
> Thus, it is possible that the monitor has registered an error and yet the
> tests pass. We should check the value returned by the _ToolRunner.run()_
> method inside each unit test.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)