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

Terry Koch commented on MRUNIT-188:
-----------------------------------

Proposed fix:

1) Added a boolean to TestDriver to determine whether a driver has been reused 
(ie has executed run() more than once).
2) Added a check and set to preRunChecks() in each of the drivers to either 
throw an exception if is is being reused or set the boolean.

Caveat - this patch still has my Javadoc alteration from JIRA 188 (sorry, my 
bad, n00b mistake) in MapDriverBase, which also has the preRunChecks() addition 
- I've since been taught how to avoid this in the future but will back out and 
redo changes if that's an issue.d

> Throw assertion error if drivers are reused
> -------------------------------------------
>
>                 Key: MRUNIT-188
>                 URL: https://issues.apache.org/jira/browse/MRUNIT-188
>             Project: MRUnit
>          Issue Type: Bug
>            Reporter: Brock Noland
>            Assignee: Terry Koch
>            Priority: Blocker
>             Fix For: 1.1.0
>
>
> We should throw an assertion error if a driver.run method is called more than 
> once. See MRUNIT-187 for more details.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to