[ 
https://issues.apache.org/jira/browse/HADOOP-6409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785624#action_12785624
 ] 

Konstantin Boudnik commented on HADOOP-6409:
--------------------------------------------

bq. Now that I look closer, the original hudson link was to the common TestCLI, 
which again has nothing in it.

Well, I think you right. you might be right. This class isn't a test per se. It 
is a helper class which isn't intended to have any tests. However, because it 
has {{Test}} prefix is being picked up by JUnit and executed as a normal test. 
What we need to do is to rename this class to something like {{CLITestHelper}}

I still believe that your patch makes sense, actually.

> TestHDFSCLI has to check if it's running any testcases at all
> -------------------------------------------------------------
>
>                 Key: HADOOP-6409
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6409
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 0.21.0, 0.22.0
>            Reporter: Konstantin Boudnik
>            Assignee: Todd Lipcon
>            Priority: Blocker
>         Attachments: hadoop-6409.txt, hadoop-6409.txt
>
>
> There's a number of occasions when TestHDFSCLI reports a successful execution 
> however doesn't run any tests at all.
> For a typical case please take a [look 
> here|http://hudson.zones.apache.org/hudson/view/Hadoop/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/162/testReport/org.apache.hadoop.cli/TestCLI/testAll/]

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to