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

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

I think the problem here is that we used only allow a single input kv. A second 
call to withInput forgot the previous one. If we change that, it's not 
backwards compatible.  Also is there any advantage to this over the new methods 
which take lists?
                
> 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
>
> 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: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to