[jira] [Comment Edited] (TEZ-1249) Bug fix of TestDAGVerify

["Jeff Zhang (JIRA)" <ji...@apache.org>]

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

Jeff Zhang edited comment on TEZ-1249 at 7/2/14 12:04 PM:
----------------------------------------------------------

Attach the patch.

Changes:
* testVerify3 and testVerify4 is unclear on where kind of IllegalStateException to catch, because there're many places throwing this kind of exception. Modify the testcases
** refactor the method name to indicate its purpose
** catch the exception and verify the exception message
* modify the testVerify4, because I guess it should be for test on "Unsupported scheduling type" while the testVerify3 is for "Unsupported source type"
* modify method BinaryOutput ( it should the a typo here )



was (Author: zjffdu):
Attach the patch.

Changes:
* testVerify3 and testVerify4 is unclear on where kind of IllegalStateException to catch. Modify the testcases
** refactor the method name to indicate its purpose
** catch the exception and verify the exception message
* modify the testVerify4, because I guess it should be for test on "Unsupported scheduling type" while the other is for "Unsupported source type"
* modify method BinaryOutput ( it should the a typo here )


> Bug fix of TestDAGVerify
> ------------------------
>
>                 Key: TEZ-1249
>                 URL: https://issues.apache.org/jira/browse/TEZ-1249
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.4.0
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: Tez-1249.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)