[aloubyansky]

This does fix the test

Index: src/main/org/jboss/reflect/plugins/MethodInfoImpl.java
===================================================================
--- src/main/org/jboss/reflect/plugins/MethodInfoImpl.java (revision 66504)
+++ src/main/org/jboss/reflect/plugins/MethodInfoImpl.java (working copy)
@@ -221,6 +221,10 @@

 if (name.equals(other.getName()) == false)
 return false;
+
+ if(!returnType.equals(other.getReturnType()))
+ return false;
+
 return Arrays.equals(parameterTypes, other.getParameterTypes());
 }

But I can't run the whole testsuite. 'mvn test' doesn't run any test in the container. Is it just me?

[aloubyansky]

It doesn't seem to be a fix actually but rather a coincidence that the test passes after this patch.
In AbstractBeanInfoFactory.getBeanProperties(methods) the getter that returns String happens to follow the getter that returns Object.
This discussion should move to the MC forum.

[alesj]

You should still take covariant return values into an account:
- http://fisheye.jboss.com/browse/JBossAS/projects/microcontainer/trunk/container/src/main/org/jboss/reflect/plugins/MethodInfoImpl.java

But, as you said, lets move this to MC forum.
And I'll see what can be done. :-)