Looks good.  Thanks Adam.  I ran it successfully and committed the changes
to revision 12203.

On Nov 20, 2007 4:05 PM, Adam Monsen <[EMAIL PROTECTED]> wrote:

> Changeset 12200 (on the trunk) includes a fix for two unit tests
> (TestFinancialActionConstants and StringUtilsTest) wherein the wrong class
> was passed to a constructor for JUnit4TestAdapter. This is an easy mistake
> to make, and I thought of a way to test to make sure it doesn't happen
> again.
>
> My methods are passable at best. I'd rather use something more
> programmatically sound like reflection or introspection or whatever, but
> hopefully this is better than nothing. Suggestions/improvements are welcome.
>
>
> This test takes a few seconds to run on my box (the first time).
> Subsequent runs are about one second. Hopefully this is fast enough to
> justify inclusion into FastTests.
>
> Hope this helps!
> -Adam
>
> --
> Adam Monsen
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2005.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

Reply via email to