Gert, Seemingly this passed under my radar and I forgot to submit my changes. Attached is the updated patch with the modified test cases. If there is any way this could be committed to 1.9 (since in some cases its a showstopper bug), it would be much appreciated.
Thanks, Justin Cherniak On Nov 3, 2007 8:22 PM, Gert Driesen <[EMAIL PROTECTED]> wrote: > Justin, > > A few "small" remarks: > > 1) Your test should first verify that these directories do not exist > before the test is run, or perhaps remove them if they do exist. > 2) Your should add a TearDown in which you remove the directories that > were created by the unit test. > 3) Use tabs instead of spaces. > 4) It would also be nice if you could add tests that verify whether the > registry keys are in fact created (same principle here, ensure they do not > exist before your run the test to avoid false positives). > > I'd be happy to take care of these if you want. > > Gert > > ------------------------------ > *From:* [EMAIL PROTECTED] [mailto: > [EMAIL PROTECTED] *On Behalf Of *Justin > Cherniak > *Sent:* zaterdag 3 november 2007 22:17 > *To:* [email protected] > *Subject:* Re: [Mono-winforms-list] [PATCH] Patch > forSystem.Windows.Forms.Application to match MSDN specificationsfor > Application Data > > OK, Attached is the updates patch with new tests and ChangeLog entries. > Can I commit? > > Thanks, > Justin > > On Oct 31, 2007 9:33 AM, Andreia Gaita < [EMAIL PROTECTED]> wrote: > > > On 10/31/07, Justin Cherniak < [EMAIL PROTECTED]> wrote: > > > Let me ask, is there any documentation on writing tests, I'm a little > > > unclear on how/where to do this? > > > > Take a look at the Managed.Windows.Forms/Test/System.Windows.Forms > > directory, it has all the unit tests for mwf. You basically set > > properties and Assert() them, look at ButtonTest, for instance, to see > > extensive property testing. ApplicationTest has the Application type > > tests, so your tests should go in there. Just create a new public void > > Method() marked with [Test], and add your tests. You can then run them > > with make run-test on MWF. > > > > andreia gaita > > > >
ApplicationData.patch
Description: Binary data
_______________________________________________ Mono-winforms-list maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-winforms-list
