WBRI-369

git-svn-id: http://anonsvn.jboss.org/repos/weld/ri/trunk@3549 1c488680-804c-0410-94cd-c6b725194a0e
weld · Aug 18, 2009 · e8e70ae · e8e70ae
1 parent 5075948
commit e8e70ae
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 7 deletions.
diff --git a/impl/src/main/java/org/jboss/webbeans/introspector/jlr/WBClassImpl.java b/impl/src/main/java/org/jboss/webbeans/introspector/jlr/WBClassImpl.java
@@ -19,6 +19,7 @@
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Field;
+import java.lang.reflect.Member;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
 import java.lang.reflect.Type;
@@ -201,7 +202,7 @@ protected WBClassImpl(Class<T> rawType, Type type, AnnotationStore annotationSto
             {
                field.setAccessible(true);
             }
-            WBField<?, ?> annotatedField = WBFieldImpl.of(field, this, classTransformer);
+            WBField<?, ?> annotatedField = WBFieldImpl.of(field, getDeclaringWBClass(field, classTransformer), classTransformer);
             this.fields.add(annotatedField);
             if (c == rawType)
             {
@@ -249,11 +250,10 @@ public Set<WBConstructor<?>> get()
 
       });
       this.declaredConstructorsBySignature = new HashMap<ConstructorSignature, WBConstructor<?>>();
-      for (Constructor<?> constructor : rawType.getDeclaredConstructors())
+      for (Constructor<Object> constructor : rawType.getDeclaredConstructors())
       {
-         @SuppressWarnings("unchecked")
-         Constructor<T> c = (Constructor<T>) constructor;
-         WBConstructor<T> annotatedConstructor = WBConstructorImpl.of(c, this, classTransformer);
+         // TODO Fix this cast
+         WBConstructor<T> annotatedConstructor = (WBConstructor<T>) WBConstructorImpl.of(constructor, getDeclaringWBClass(constructor, classTransformer), classTransformer);
          if (!constructor.isAccessible())
          {
             constructor.setAccessible(true);
@@ -330,7 +330,7 @@ public Set<WBConstructor<?>> get()
                method.setAccessible(true);
             }
 
-            WBMethod<?, ?> annotatedMethod = WBMethodImpl.of(method, this, classTransformer);
+            WBMethod<?, ?> annotatedMethod = WBMethodImpl.of(method, getDeclaringWBClass(method, classTransformer), classTransformer);
             this.methods.add(annotatedMethod);
             this.methodsBySignature.put(annotatedMethod.getSignature(), annotatedMethod);
             if (c == rawType)
@@ -360,6 +360,20 @@ public Set<WBConstructor<?>> get()
          }
       }
    }
+
+   @SuppressWarnings("unchecked")
+   private <X> WBClass<X> getDeclaringWBClass(Member member, ClassTransformer transformer)
+   {
+      if (member.getDeclaringClass().equals(getJavaClass()))
+      {
+         return (WBClass<X>) this;
+      }
+      else
+      {
+         WBClass<X> loadClass = transformer.loadClass(member.getDeclaringClass());
+         return loadClass;
+      }
+   }
 
    /**
     * Gets the implementing class

diff --git a/...est/java/org/jboss/webbeans/test/unit/implementation/annotatedItem/DeclaringTypeTest.java b/...est/java/org/jboss/webbeans/test/unit/implementation/annotatedItem/DeclaringTypeTest.java
@@ -33,7 +33,7 @@
 public class DeclaringTypeTest extends AbstractWebBeansTest
 {
 
-   @Test(groups="broken")
+   @Test
    public void testInheritance()
    {
       AnnotatedType<Child> type = getCurrentManager().createAnnotatedType(Child.class);