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

Brock Noland commented on MRUNIT-138:
-------------------------------------

This patch looks quite good!  I am going to do a review of the rest of the code 
before committing just to make sure we aren't missing other places.
                
> Multiple calls to withInput should be supported
> -----------------------------------------------
>
>                 Key: MRUNIT-138
>                 URL: https://issues.apache.org/jira/browse/MRUNIT-138
>             Project: MRUnit
>          Issue Type: Improvement
>    Affects Versions: 1.0.0
>            Reporter: Dave Beech
>            Priority: Blocker
>             Fix For: 1.0.0
>
>         Attachments: MRUNIT-138-0.patch
>
>
> As multiple key/val pairs are now supported for tests following MRUNIT-64, it 
> feels to me that I should be able to call withInput multiple times in 
> sequence in the same way I would usually call withOutput. This doesn't work 
> because of the way the deprecation of the old code has been handled. 
> Here's a unit test I think should pass:
> (from TestMapDriver - mapper is IdentityMapper)
>   @Test
>   public void testMultipleWithInput() throws IOException {
>     driver.withInput(new Text("foo"), new Text("bar"))
>       .withInput(new Text("bar"), new Text("baz"))
>       .withOutput(new Text("foo"), new Text("bar"))
>       .withOutput(new Text("bar"), new Text("baz"))
>       .runTest(false);
>   }

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