[
https://issues.apache.org/jira/browse/MRUNIT-126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13424810#comment-13424810
]
Bertrand Dechoux commented on MRUNIT-126:
-----------------------------------------
My understanding of run() is that, when using it, all assertions/tests should
be done by the user. If we mix the behaviour with runTest() the usage would be
less coherent.
A warning is a good idea but for me it should be an exception for an illegal
use of the API. A warning is something that won't be necessary impact the user.
It is nice to put a warning when something is migrating : "warning this method
will be removed" but that's not the case here.
Of course it means that existent tests may break when the users of mrunit will
migrate to the newer versions.
But this is actually a good thing because it means that they would know which
test were 'nice but useless'.
> Counter tests are ignored if test is executed using run() rather than
> runTest()
> -------------------------------------------------------------------------------
>
> Key: MRUNIT-126
> URL: https://issues.apache.org/jira/browse/MRUNIT-126
> Project: MRUnit
> Issue Type: Bug
> Affects Versions: 0.9.0
> Reporter: Dave Beech
>
> An alternative way of running an mrunit test is to call run(), which returns
> the output list to you for manual validation, rather than runTest() which
> validates everything automatically.
> If you run a test using the run() method, any counter tests you have
> specified via withCounter(..) etc are silently ignored, which may falsely
> give the impression that these tests have passed.
> At the very least we should give a warning that this is happening. Otherwise,
> we should make sure the counter validation is triggered in all cases.
--
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
