summaryrefslogtreecommitdiffstats
path: root/sca-java-2.x/trunk
diff options
context:
space:
mode:
Diffstat (limited to 'sca-java-2.x/trunk')
-rw-r--r--sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/Bean1.java6
-rw-r--r--sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/JavaIntrospectionHelperTestCase.java6
-rw-r--r--sca-java-2.x/trunk/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/introspect/JavaIntrospectionHelper.java9
-rw-r--r--sca-java-2.x/trunk/testing/itest/properties/src/main/resources/CTCalcTest.composite4
-rw-r--r--sca-java-2.x/trunk/testing/itest/properties/src/test/java/org/apache/tuscany/sca/itest/ctcalc/CTCalcTestCase.java3
5 files changed, 10 insertions, 18 deletions
diff --git a/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/Bean1.java b/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/Bean1.java
index a289a58091..e993cdd382 100644
--- a/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/Bean1.java
+++ b/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/Bean1.java
@@ -25,9 +25,9 @@ package org.apache.tuscany.sca.implementation.java.util;
*/
public class Bean1 extends SuperBean {
- public static final int ALL_BEAN1_FIELDS = 6; //TUSCANY-3924 + ALL_SUPER_FIELDS;
- public static final int ALL_BEAN1_PUBLIC_PROTECTED_FIELDS = 5; //TUSCANY-3924 + ALL_SUPER_PUBLIC_PROTECTED_FIELDS;
- public static final int ALL_BEAN1_METHODS = 4; //TUSCANY-3924 + ALL_SUPER_METHODS - 1;
+ public static final int ALL_BEAN1_FIELDS = 6 + ALL_SUPER_FIELDS;
+ public static final int ALL_BEAN1_PUBLIC_PROTECTED_FIELDS = 5 + ALL_SUPER_PUBLIC_PROTECTED_FIELDS;
+ public static final int ALL_BEAN1_METHODS = 4 + ALL_SUPER_METHODS - 1;
public String field3;
protected String field2;
private String field1;
diff --git a/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/JavaIntrospectionHelperTestCase.java b/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/JavaIntrospectionHelperTestCase.java
index dfb3c77966..9624865dcc 100644
--- a/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/JavaIntrospectionHelperTestCase.java
+++ b/sca-java-2.x/trunk/modules/implementation-java-runtime/src/test/java/org/apache/tuscany/sca/implementation/java/util/JavaIntrospectionHelperTestCase.java
@@ -29,7 +29,6 @@ import java.util.Map;
import java.util.Set;
import org.apache.tuscany.sca.implementation.java.introspect.JavaIntrospectionHelper;
-import org.junit.Ignore;
import org.junit.Test;
/**
@@ -47,7 +46,7 @@ public class JavaIntrospectionHelperTestCase {
@Test
public void testBean1AllPublicProtectedFields() throws Exception {
Set<Field> beanFields = JavaIntrospectionHelper.getAllPublicAndProtectedFields(Bean1.class, true);
- assertEquals(2, beanFields.size()); //Bean1.ALL_BEAN1_PUBLIC_PROTECTED_FIELDS
+ assertEquals(4, beanFields.size()); //Bean1.ALL_BEAN1_PUBLIC_PROTECTED_FIELDS
}
@Test
@@ -76,8 +75,7 @@ public class JavaIntrospectionHelperTestCase {
throw new Exception("Override never invoked");
}
}
-
- @Ignore("TUSCANY-3924")
+
@Test
public void testNoOverrideMethod() throws Exception {
Set<Method> beanFields = JavaIntrospectionHelper.getAllUniquePublicProtectedMethods(Bean1.class, true);
diff --git a/sca-java-2.x/trunk/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/introspect/JavaIntrospectionHelper.java b/sca-java-2.x/trunk/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/introspect/JavaIntrospectionHelper.java
index f67ebe793c..a04ad3721a 100644
--- a/sca-java-2.x/trunk/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/introspect/JavaIntrospectionHelper.java
+++ b/sca-java-2.x/trunk/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/introspect/JavaIntrospectionHelper.java
@@ -83,8 +83,7 @@ public final class JavaIntrospectionHelper {
if (clazz == null || clazz.isArray() || Object.class.equals(clazz)) {
return fields;
}
- // TUSCANY-3924 - don't consider inherited fields
- //fields = getAllPublicAndProtectedFields(clazz.getSuperclass(), fields, validating);
+ fields = getAllPublicAndProtectedFields(clazz.getSuperclass(), fields, validating);
Field[] declaredFields = null;
@@ -137,8 +136,7 @@ public final class JavaIntrospectionHelper {
return fields;
}
- // TUSCANY-3924 - don't consider inherited fields
- //fields = getInjectableFields(clazz.getSuperclass(), fields, validating);
+ fields = getInjectableFields(clazz.getSuperclass(), fields, validating);
Field[] declaredFields = null;
@@ -237,8 +235,7 @@ public final class JavaIntrospectionHelper {
}
// evaluate class hierarchy - this is done last to track inherited
// methods
- // TUSCANY-3924 - don't consider inherited fields
- //methods = getAllUniqueMethods(pClass.getSuperclass(), methods, validating);
+ methods = getAllUniqueMethods(pClass.getSuperclass(), methods, validating);
return methods;
}
diff --git a/sca-java-2.x/trunk/testing/itest/properties/src/main/resources/CTCalcTest.composite b/sca-java-2.x/trunk/testing/itest/properties/src/main/resources/CTCalcTest.composite
index e562568492..db7b3be2d2 100644
--- a/sca-java-2.x/trunk/testing/itest/properties/src/main/resources/CTCalcTest.composite
+++ b/sca-java-2.x/trunk/testing/itest/properties/src/main/resources/CTCalcTest.composite
@@ -35,10 +35,8 @@
<component name="Annotated2Component">
<implementation.java class="org.apache.tuscany.sca.itest.ctcalc.Annotated2CTCalcComponentImpl" />
- <property name="aPropertyAnnotated" value="xyz"/>
-<!-- TUSCANY-3924 playing with whether we should take note of inherited annotations as per JSR-250
+ <property name="aPropertyAnnotated" value="xyz"/>
<property name="aPropertyPublic" value="abc"/>
- -->
</component>
</composite>
diff --git a/sca-java-2.x/trunk/testing/itest/properties/src/test/java/org/apache/tuscany/sca/itest/ctcalc/CTCalcTestCase.java b/sca-java-2.x/trunk/testing/itest/properties/src/test/java/org/apache/tuscany/sca/itest/ctcalc/CTCalcTestCase.java
index c77dbb1232..b0bd9ec8fe 100644
--- a/sca-java-2.x/trunk/testing/itest/properties/src/test/java/org/apache/tuscany/sca/itest/ctcalc/CTCalcTestCase.java
+++ b/sca-java-2.x/trunk/testing/itest/properties/src/test/java/org/apache/tuscany/sca/itest/ctcalc/CTCalcTestCase.java
@@ -84,7 +84,7 @@ public class CTCalcTestCase {
@Test
public void testUnannotatedExtension() {
assertEquals("Unannotated", unannotatedComponent.test());
- assertEquals(2, ((NodeImpl)node).getDomainComposite().getComponents().get(1).getProperties().size());
+ assertEquals(4, ((NodeImpl)node).getDomainComposite().getComponents().get(1).getProperties().size());
System.out.println(((NodeImpl)node).getDomainComposite().getComponents().get(1).getProperties());
}
@@ -95,7 +95,6 @@ public class CTCalcTestCase {
System.out.println(((NodeImpl)node).getDomainComposite().getComponents().get(2).getProperties());
}
- @Ignore("TUSCANY-3924 playing with whether we should take note of inherited annotations as per JSR-250")
@Test
public void testAnnotated2Extension() {
assertEquals("Annotated", annotatedComponent.test());