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

ASF GitHub Bot commented on CAMEL-9148:
---------------------------------------

GitHub user woj-i opened a pull request:

    https://github.com/apache/camel/pull/633

    add more precise information of skipped tests [CAMEL-9148]

    add more precise information of skipped tests and append warnings on the 
console

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/woj-i/camel CAMEL-9148

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/633.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #633
    
----
commit fe41cef182410cdac723aecffbfb3bd7076dcb29
Author: woj-i <[email protected]>
Date:   2015-10-08T16:05:46Z

    add more precise information of skipped tests and append warnings on the 
console

----


> camel-hbase tests pass despite setup error
> ------------------------------------------
>
>                 Key: CAMEL-9148
>                 URL: https://issues.apache.org/jira/browse/CAMEL-9148
>             Project: Camel
>          Issue Type: Test
>          Components: camel-hbase
>            Reporter: Wojciech Indyk
>
> there is clause in tests: when system is not ready (a local test cluster 
> failed) then test is passed. It concerns all test cases. It is because a 
> setUp method is: 
> {code:java}
> try {
>             hbaseUtil.startMiniCluster(numServers);
>         } catch (Exception e) {
>             LOG.error("couldn't start HBase cluster.", e);
>             systemReady = false;
>         }
> {code}
> in my opinion this catch should be droped. When start of a miniCluster is 
> failed, then tests should failed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to