You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by lr...@apache.org on 2009/09/26 01:33:43 UTC

svn commit: r819063 - in /tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl: PropertyProcessorTestCase.java ReferenceProcessorTestCase.java

Author: lresende
Date: Fri Sep 25 23:33:43 2009
New Revision: 819063

URL: http://svn.apache.org/viewvc?rev=819063&view=rev
Log:
Minor cleanup

Modified:
    tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/PropertyProcessorTestCase.java
    tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/ReferenceProcessorTestCase.java

Modified: tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/PropertyProcessorTestCase.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/PropertyProcessorTestCase.java?rev=819063&r1=819062&r2=819063&view=diff
==============================================================================
--- tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/PropertyProcessorTestCase.java (original)
+++ tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/PropertyProcessorTestCase.java Fri Sep 25 23:33:43 2009
@@ -168,7 +168,7 @@
     		fail("Processor should not accept a static field with Property annotation");
     	}
     	catch (IllegalPropertyException e) {
-			// System.out.println("Caught expected exception");
+			// expected
 		}
     	catch (Exception e) {
 			fail("Wrong exception detected");
@@ -182,7 +182,7 @@
     		fail("Processor should not accept a static method with Property annotation");
     	}
     	catch (IllegalPropertyException e) {
-			// System.out.println("Caught expected exception");
+			// expected
 		}
     	catch (Exception e) {
 			fail("Wrong exception detected");
@@ -200,8 +200,7 @@
         	
             fail("Method with @Property annotated args should be rejected");
         } catch (IllegalPropertyException e) {
-//        	e.printStackTrace();
-//        	System.out.println("Exception successfully received");
+        	// expected
         }
         catch (Exception e) {
 			fail("Wrong exception received");

Modified: tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/ReferenceProcessorTestCase.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/ReferenceProcessorTestCase.java?rev=819063&r1=819062&r2=819063&view=diff
==============================================================================
--- tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/ReferenceProcessorTestCase.java (original)
+++ tuscany/java/sca/modules/implementation-java/src/test/java/org/apache/tuscany/sca/implementation/java/introspect/impl/ReferenceProcessorTestCase.java Fri Sep 25 23:33:43 2009
@@ -140,8 +140,7 @@
         	
             fail("reference annotation on ordinary method arg should be rejected");
         } catch (IllegalReferenceException e) {
-//        	e.printStackTrace();
-//        	System.out.println("Exception successfully received");
+        	// expected
         }
     	catch (Exception e) {
 			fail("Wrong exception detected");
@@ -156,7 +155,6 @@
         assertNotNull(ref);
         assertSame(Ref.class, ((JavaInterface)ref.getInterfaceContract().getInterface()).getJavaClass());
         assertEquals(Multiplicity.ONE_N, ref.getMultiplicity());
-        // assertEquals(Multiplicity.ONE_ONE, ref.getMultiplicity());
     }
 
     @Test
@@ -166,7 +164,6 @@
         assertNotNull(ref);
         assertSame(Ref.class, ((JavaInterface)ref.getInterfaceContract().getInterface()).getJavaClass());
         assertEquals(Multiplicity.ZERO_N, ref.getMultiplicity());
-        // assertFalse(ref.isMustSupply());
     }
 
     @Test
@@ -176,7 +173,6 @@
         assertNotNull(ref);
         assertSame(Ref.class, ((JavaInterface)ref.getInterfaceContract().getInterface()).getJavaClass());
         assertEquals(Multiplicity.ONE_N, ref.getMultiplicity());
-        // assertEquals(Multiplicity.ONE_ONE, ref.getMultiplicity());
     }
 
     @Test
@@ -186,7 +182,6 @@
         assertNotNull(ref);
         assertSame(Ref.class, ((JavaInterface)ref.getInterfaceContract().getInterface()).getJavaClass());
         assertEquals(Multiplicity.ZERO_N, ref.getMultiplicity());
-        // assertFalse(ref.isMustSupply());
     }
     
     @Test
@@ -196,7 +191,7 @@
     		fail("Processor should not accept a static field with Property annotation");
     	}
     	catch (IllegalReferenceException e) {
-			// System.out.println("Caught expected exception");
+			// expected
 		}
     	catch (Exception e) {
 			fail("Wrong exception detected");
@@ -211,7 +206,7 @@
     		fail("Processor should not accept a static method with Property annotation");
     	}
     	catch (IllegalPropertyException e) {
-			// System.out.println("Caught expected exception");
+			// expected
 		}
     	catch (Exception e) {
 			fail("Wrong exception detected");