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

Robert Kanter commented on OOZIE-1307:
--------------------------------------

A few comments:
\- The patch removes the tests added by OOZIE-1286; any reason for this?  
\- Why is {{testOozieUserMismatch}} removed?  I know it was commented out, but 
we should either leave it for now or fix it.
\- Can you fix the line that has trailing spaces?
                
> Cover package org.apache.oozie.action.ssh with unit tests
> ---------------------------------------------------------
>
>                 Key: OOZIE-1307
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1307
>             Project: Oozie
>          Issue Type: Test
>    Affects Versions: trunk
>            Reporter: Vadim Bondarev
>            Priority: Minor
>         Attachments: OOZIE-1307-trunk-a.patch
>
>
> Improve test coverage for package org.apache.oozie.action.ssh 
> Add in class TestSshActionExecutor methods testSshCheckWithInvalidXml(), 
> testSshStartWithInvalidXml(), testJobStartAndKill()
> For success run test current user should be allowed to login by ssh without 
> password ("ssh test@localhost")

--
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