[jboss-jira] [JBoss JIRA] (AS7-2954) TS: Mgmt test utils - move JUnit deps to tests.

[Jason Greene (Commented) (JIRA)]

    [ https://issues.jboss.org/browse/AS7-2954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12651792#comment-12651792 ] 

Jason Greene commented on AS7-2954:
-----------------------------------

Is there a reason why this is a bad idea? I mean all assert does is throw an exception? Are you trying to limit the usage of junit to allow usage of testng?
                
> TS: Mgmt test utils - move JUnit deps to tests.
> -----------------------------------------------
>
>                 Key: AS7-2954
>                 URL: https://issues.jboss.org/browse/AS7-2954
>             Project: Application Server 7
>          Issue Type: Quality Risk
>            Reporter: Ondrej Zizka
>            Assignee: Dominik Pospisil
>
> Calling JUnit's assert*() from a dependency is not a good idea.
> Pls make it return boolean or throw an exception, and handle it in the test sources.
> CLIWrapper.java

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira