[ 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