[rules-dev] Test cases upgraded from JUnit 3 to JUnit 4

[Geoffrey De Smet]

Hi guys,

I've updated all testcases on master to use JUnit 4 style
(except for drools-eclipse and the non-active parts of drools-grid).
The main reason is so we now let hudson report how many testcases are 
ignored (annoted with @Ignore instead of disabled).

Some notes:

    * *Don't disable testcases by prefixing them with FIXME_ or
      disabled_ or by commenting out @Test*
          o *Instead, add the @Ignore annotation*
                + This way hudson gives us a report on how many
                  testcases have been ignored.
    * *Don't use try-catch to test if an exception is thrown. Use
      @Test(expected = MyException.class)*
          o I haven't migrated this in all testcases, as I couldn't
            automate that.
          o Please fix it in any code you come across.
    * To use assertions like assertEquals(a,b), just add this import:
          o import static org.junit.Assert.*;
    * Don't import anything from junit.framework.*.
          o Import only from org.junit.*
          o Configure your Eclipse/Intellij to not propose those imports
            to keep mistakes to a minimum.
    * TestSuites have been removed.
          o Use your Eclipse/IntelliJ to single out tests, packages of
            tests or modules.
          o They bring extra maintenance (and seem to be seen as legacy
            since junit 4?)
    * Testcases that print out their own name, no longer print out their
      own name
          o The junit runner prints out the name already
                + Otherwise it looks like it's run twice (while it's not)

I hope this push (many files were changed) doesn't cause to many merge 
conflicts.
If it does: just take your revision and upgrade your file to junit 4 
manually.

Learn more:
- Junit 4 in 60 seconds: 
http://www.cavdar.net/2008/07/21/junit-4-in-60-seconds/

-- 
With kind regards,
Geoffrey De Smet

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/rules-dev/attachments/20101226/34eb3eff/attachment.html