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

Robert Kanter commented on OOZIE-987:
-------------------------------------

The test error is unrelated. The patch only changes one file, 
TestMapReduceActionExecutor, so it couldn't have broken any other tests.
(the test error was in 
org.apache.oozie.executor.jpa.TestCoordJobGetReadyActionsJPAExecutor.testCoordActionGet)
                
> Fix minor bug in one of the uber jar tests
> ------------------------------------------
>
>                 Key: OOZIE-987
>                 URL: https://issues.apache.org/jira/browse/OOZIE-987
>             Project: Oozie
>          Issue Type: Bug
>    Affects Versions: trunk
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>            Priority: Trivial
>             Fix For: trunk
>
>         Attachments: OOZIE-987.patch
>
>
> In OOZIE-654, I added the ability to use uber jars from Oozie.  One of the 
> new tests has a try-catch block and is expected to throw and catch an 
> exception but in the case where the test fails (i.e. it doesn't throw an 
> exception) it should actually fail.  
> {code}
> try {
>     actionXml = createUberJarActionXML(getNameNodeUri() + "/app/job.jar", "");
>     conf = ae.createBaseHadoopConf(context, actionXml);
>     ae.setupActionConf(conf, context, actionXml, getFsTestCaseDir());
>     assertEquals(getNameNodeUri() + "/app/job.jar", 
> conf.get("oozie.mapreduce.uber.jar"));  // <-- SHOULD FAIL INSTEAD OF THIS
> } catch (ActionExecutorException aee) {
>     assertEquals("MR003", aee.getErrorCode());
>     assertEquals(ActionExecutorException.ErrorType.ERROR, aee.getErrorType());
>     
> assertTrue(aee.getMessage().contains("oozie.action.mapreduce.uber.jar.enable"));
>     assertTrue(aee.getMessage().contains("oozie.mapreduce.uber.jar"));
> }
> {code}

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