Jonathan, Thats not a problem at all. I'm glad someone is taking care of it. I just wanted to make sure it didn't get forgotten and since I didn't hear any response in about a week, I thought that it had been.
Thanks, Justin On Feb 19, 2008 5:12 PM, Jonathan Pobst <[EMAIL PROTECTED]> wrote: > It is not ok to commit. There are several issues I am working through > on it currently. I will fix it up and commit it. > > - It will not compile on 1.1 (string.IsNullOrEmpty) > - It returns incorrect ProductName, ProductVersion, and CompanyName when > the attributes are not present. > > Sorry I did not get to this sooner, we had last week off for Hack Week. > > Jonathan > > > > Justin Cherniak wrote: > > I'm not sure how this works, but since I haven't heard back from anyone, > > can I assume this patch is OK and I can commit it? > > > > Thanks, > > Justin Cherniak > > > > On Feb 14, 2008 12:27 AM, Justin Cherniak <[EMAIL PROTECTED] > > <mailto:[EMAIL PROTECTED]>> wrote: > > > > Also note, I filed this bug (and my fix and testcase) as bug 361709 > > <https://bugzilla.novell.com/show_bug.cgi?id=361709>. > > > > Thanks, > > Justin Cherniak > > > > > > On Wed, Feb 13, 2008 at 8:00 PM, Justin Cherniak > > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> > wrote: > > > > 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] > > <mailto:[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]> > > [mailto:[EMAIL PROTECTED] > > <mailto:[EMAIL PROTECTED]>] *On > > Behalf Of *Justin Cherniak > > *Sent:* zaterdag 3 november 2007 22:17 > > *To:* [email protected] > > <mailto:[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] <mailto:[EMAIL PROTECTED]>> wrote: > > > > On 10/31/07, Justin Cherniak < [EMAIL PROTECTED] > > <mailto:[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 > > > > > > > > > > > > > > > > ------------------------------------------------------------------------ > > > > _______________________________________________ > > Mono-winforms-list maillist - [email protected] > > http://lists.ximian.com/mailman/listinfo/mono-winforms-list > >
_______________________________________________ Mono-winforms-list maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-winforms-list
