Mark, This is the test from the second patch, it is not quite clear which test from the first patch do you mean. Am I missing something?
============================================= Author: mloenko Date: Thu Apr 20 23:23:20 2006 New Revision: 395794 URL: http://svn.apache.org/viewcvs?rev=395794&view=rev Log: fixes for HARMONY-237 PropertyDescriptor constructors throws incorrect (or no) exception Modified: incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/ja va/beans/PropertyDescriptor.java incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/te sts/api/java/beans/PropertyDescriptorTest.java ... + try { + PropertyDescriptor pd = new PropertyDescriptor("",null,null); + fail("Constructor PropertyDescriptor(\"\",null,null) should "+ + "throw IntrospectionException"); + } catch (IntrospectionException e) { + assertEquals("bad property name", e.getMessage()); + } Thanks, Mikhail >-----Original Message----- >From: Mark Hindess (JIRA) [mailto:[EMAIL PROTECTED] >Sent: Friday, April 21, 2006 2:17 PM >To: [EMAIL PROTECTED] >Subject: [jira] Commented: (HARMONY-237) PropertyDescriptor constructors throws incorrect (or no) >exception > > [ http://issues.apache.org/jira/browse/HARMONY-237?page=comments#action_12 375504 ] > >Mark Hindess commented on HARMONY-237: >-------------------------------------- > >This fixes the behaviour but it looks like the tests from the first patch for the three argument >case and from the second patch are still missing. > > >> PropertyDescriptor constructors throws incorrect (or no) exception >> ------------------------------------------------------------------ >> >> Key: HARMONY-237 >> URL: http://issues.apache.org/jira/browse/HARMONY-237 >> Project: Harmony >> Type: Bug > >> Components: Classlib >> Reporter: Mark Hindess >> Assignee: Mikhail Loenko >> Priority: Minor >> Attachments: 02.propertydescriptor.empty.propery.name.diff, propertydescriptor.npe.diff >> >> Passing all null arguments to these constructors gives incorrect behaviour. >> j.beans.PropertyDescriptor(j.l.String,j.l.Class): >> RI throws j.beans.IntrospectionException but >> Harmony throws j.l.NullPointerException >> j.beans.PropertyDescriptor(j.l.String,j.l.Class,j.l.String,j.l.String): >> RI throws j.beans.IntrospectionException but >> Harmony throws j.l.NullPointerException >> j.beans.PropertyDescriptor(j.l.String,j.l.reflect.Method,j.l.reflect.Met hod): >> RI throws j.beans.IntrospectionException but >> Harmony doesn't throw an exception >> Will attach a patch. > >-- >This message is automatically generated by JIRA. >- >If you think it was sent incorrectly contact one of the administrators: > http://issues.apache.org/jira/secure/Administrators.jspa >- >For more information on JIRA, see: > http://www.atlassian.com/software/jira --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
