Author: hasan
Date: Wed Jan 27 11:22:13 2010
New Revision: 903610

URL: http://svn.apache.org/viewvc?rev=903610&view=rev
Log:
CLEREZZA-70: used java annotation for scr Reference in Editor.java and improved 
coding in test

Modified:
    
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.platform.content/src/main/java/org/apache/clerezza/platform/content/Editor.java
    
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.rdf.utils/src/test/java/org/apache/clerezza/rdf/utils/MGraphUtilsTest.java

Modified: 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.platform.content/src/main/java/org/apache/clerezza/platform/content/Editor.java
URL: 
http://svn.apache.org/viewvc/incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.platform.content/src/main/java/org/apache/clerezza/platform/content/Editor.java?rev=903610&r1=903609&r2=903610&view=diff
==============================================================================
--- 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.platform.content/src/main/java/org/apache/clerezza/platform/content/Editor.java
 (original)
+++ 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.platform.content/src/main/java/org/apache/clerezza/platform/content/Editor.java
 Wed Jan 27 11:22:13 2010
@@ -41,6 +41,7 @@
 import org.osgi.service.component.ComponentContext;
 import org.apache.felix.scr.annotations.Component;
 import org.apache.felix.scr.annotations.Property;
+import org.apache.felix.scr.annotations.Reference;
 import org.apache.felix.scr.annotations.Service;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -68,15 +69,11 @@
 @Property(name="javax.ws.rs", boolValue=true)
 @Path("tools/editor")
 public class Editor {
-       
-       /**
-        * @scr.reference
-        */
+
+       @Reference
        private ContentGraphProvider cgProvider;
 
-       /**
-        * @scr.reference
-        */
+       @Reference
        private TcManager tcManager;
 
        

Modified: 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.rdf.utils/src/test/java/org/apache/clerezza/rdf/utils/MGraphUtilsTest.java
URL: 
http://svn.apache.org/viewvc/incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.rdf.utils/src/test/java/org/apache/clerezza/rdf/utils/MGraphUtilsTest.java?rev=903610&r1=903609&r2=903610&view=diff
==============================================================================
--- 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.rdf.utils/src/test/java/org/apache/clerezza/rdf/utils/MGraphUtilsTest.java
 (original)
+++ 
incubator/clerezza/issues/CLEREZZA-70/org.apache.clerezza.rdf.utils/src/test/java/org/apache/clerezza/rdf/utils/MGraphUtilsTest.java
 Wed Jan 27 11:22:13 2010
@@ -40,15 +40,7 @@
 
        @Test
        public void removeSubGraph() throws NoSuchSubGraphException {
-               MGraph baseGraph = new SimpleMGraph();
-               {
-                       BNode bNode1 = new BNode();
-                       BNode bNode2 = new BNode();
-                       baseGraph.add(new TripleImpl(u1, u2, bNode2));
-                       baseGraph.add(new TripleImpl(bNode2, u2, bNode2));
-                       baseGraph.add(new TripleImpl(bNode2, u2, bNode1));
-                       baseGraph.add(new TripleImpl(u3, u2, u1));
-               }
+               MGraph baseGraph = createBaseGraph();
 
                MGraph subGraph = new SimpleMGraph();
                {
@@ -61,13 +53,8 @@
                MGraphUtils.removeSubGraph(baseGraph, subGraph);
                Assert.assertEquals(1, baseGraph.size());
        }
-       
-       /** It is required that the subgraph comprises the whole context of the 
Bnodes it ioncludes
-        * 
-        * @throws 
org.apache.clerezza.rdf.utils.MGraphUtils.NoSuchSubGraphException
-        */
-       @Test(expected=NoSuchSubGraphException.class)
-       public void removeIncompleteSubGraph() throws NoSuchSubGraphException {
+
+       private MGraph createBaseGraph() {
                MGraph baseGraph = new SimpleMGraph();
                {
                        BNode bNode1 = new BNode();
@@ -77,6 +64,16 @@
                        baseGraph.add(new TripleImpl(bNode2, u2, bNode1));
                        baseGraph.add(new TripleImpl(u3, u2, u1));
                }
+               return baseGraph;
+       }
+       
+       /** It is required that the subgraph comprises the whole context of the 
Bnodes it ioncludes
+        * 
+        * @throws 
org.apache.clerezza.rdf.utils.MGraphUtils.NoSuchSubGraphException
+        */
+       @Test(expected=NoSuchSubGraphException.class)
+       public void removeIncompleteSubGraph() throws NoSuchSubGraphException {
+               MGraph baseGraph = createBaseGraph();
 
                MGraph subGraph = new SimpleMGraph();
                {
@@ -90,15 +87,7 @@
 
        @Test(expected=NoSuchSubGraphException.class)
        public void removeInvalidSubGraph() throws NoSuchSubGraphException {
-               MGraph baseGraph = new SimpleMGraph();
-               {
-                       BNode bNode1 = new BNode();
-                       BNode bNode2 = new BNode();
-                       baseGraph.add(new TripleImpl(u1, u2, bNode2));
-                       baseGraph.add(new TripleImpl(bNode2, u2, bNode2));
-                       baseGraph.add(new TripleImpl(bNode2, u2, bNode1));
-                       baseGraph.add(new TripleImpl(u3, u2, u1));
-               }
+               MGraph baseGraph = createBaseGraph();
 
                MGraph subGraph = new SimpleMGraph();
                {
@@ -111,5 +100,5 @@
                }
                MGraphUtils.removeSubGraph(baseGraph, subGraph);
        }
-
 }
+


Reply via email to