[ 
http://issues.apache.org/jira/browse/DERBY-1600?page=comments#action_12423924 ] 
            
Daniel John Debrunner commented on DERBY-1600:
----------------------------------------------

My confusion, Sorry, just getting used to Junit tests. Thanks for the quick 
feedback, helped my understanding.

I tried to close this and got the unhelpful - action 2 invalid message

> JUnit test SURQueryMixTest does not follow the typical pattern for JUnit 
> tests.
> -------------------------------------------------------------------------------
>
>                 Key: DERBY-1600
>                 URL: http://issues.apache.org/jira/browse/DERBY-1600
>             Project: Derby
>          Issue Type: Bug
>          Components: Test
>    Affects Versions: 10.2.0.0
>            Reporter: Daniel John Debrunner
>             Fix For: 10.2.0.0
>
>
> SURQueryMixTest overrides runtest instead of providing individual test 
> methods, I think this means its setUp and tearDown methods would not be 
> called.
> It does not have such methods at the present, bu if someone needed to add 
> them then they would waste time figuring out why the methods were never 
> called.
> JUnit tests in derby should follow the established pattern, see 
> http://junit.sourceforge.net/doc/cookbook/cookbook.htm
> http://wiki.apache.org/db-derby/DerbyJUnitTesting
> Slows down the progress to moving to JUnit for all tests.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to