You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by ml...@apache.org on 2006/08/22 13:44:06 UTC

svn commit: r433605 - in /incubator/harmony/enhanced/classlib/trunk/modules/beans/src: main/java/java/beans/IndexedPropertyDescriptor.java test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java

Author: mloenko
Date: Tue Aug 22 04:44:06 2006
New Revision: 433605

URL: http://svn.apache.org/viewvc?rev=433605&view=rev
Log:
applied patch for HARMONY-1236
[classlib][beans] no expected IntrospectionException for ctor IndexedPropertyDescriptor(String, Class)

Modified:
    incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/IndexedPropertyDescriptor.java
    incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java

Modified: incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/IndexedPropertyDescriptor.java
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/IndexedPropertyDescriptor.java?rev=433605&r1=433604&r2=433605&view=diff
==============================================================================
--- incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/IndexedPropertyDescriptor.java (original)
+++ incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/IndexedPropertyDescriptor.java Tue Aug 22 04:44:06 2006
@@ -72,11 +72,15 @@
         String indexedGetterName = createDefaultMethodName(propertyName, "get");
         if(hasMethod(beanClass, indexedGetterName)) {
             setIndexedReadMethod(beanClass, indexedGetterName);
+        } else {
+            throw new IntrospectionException("Method not found: get" + propertyName);
         }
         
         String indexedSetterName = createDefaultMethodName(propertyName, "set");
         if(hasMethod(beanClass, indexedSetterName)) {
             setIndexedWriteMethod(beanClass, indexedSetterName);
+        } else {
+            throw new IntrospectionException("Method not found: set" + propertyName);
         }
     }
 

Modified: incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java?rev=433605&r1=433604&r2=433605&view=diff
==============================================================================
--- incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java (original)
+++ incubator/harmony/enhanced/classlib/trunk/modules/beans/src/test/java/org/apache/harmony/beans/tests/java/beans/IndexedPropertyDescriptorTest.java Tue Aug 22 04:44:06 2006
@@ -348,6 +348,14 @@
         assertFalse(ipd.isExpert());
         assertFalse(ipd.isHidden());
         assertFalse(ipd.isPreferred());
+
+        //Regression for HARMONY-1236
+        try {
+            new IndexedPropertyDescriptor("0xDFRF", Float.TYPE);
+            fail("IntrospectionException expected");
+        } catch (IntrospectionException e) {
+            //expected
+        }
     }
 
     public void testIndexedPropertyDescriptorStringClass_PropertyNameNull()