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

_______________________________________________
rules-dev mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/rules-dev

Reply via email to