X-Git-Url: http://wamblee.org/gitweb/?a=blobdiff_plain;f=system%2Fgeneral%2Fsrc%2Ftest%2Fjava%2Forg%2Fwamblee%2Fsystem%2Fgraph%2FCompositeEdgeFilterTest.java;h=042b8edfa677a0a4c59a9490b8a2946425e7d8f7;hb=49ce7cb8387601982d5e6ef186ce206d38f6e3d7;hp=55cae8964983a8e12efd2be53a7910651772a700;hpb=8557fbe8c7ea4e1cbcbf10d3c4e8c60c9c1e312b;p=utils

diff --git a/system/general/src/test/java/org/wamblee/system/graph/CompositeEdgeFilterTest.java b/system/general/src/test/java/org/wamblee/system/graph/CompositeEdgeFilterTest.java
index 55cae896..042b8edf 100644
--- a/system/general/src/test/java/org/wamblee/system/graph/CompositeEdgeFilterTest.java
+++ b/system/general/src/test/java/org/wamblee/system/graph/CompositeEdgeFilterTest.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2008 the original author or authors.
+ * Copyright 2005-2010 the original author or authors.
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -12,14 +12,12 @@
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
  * limitations under the License.
- */ 
+ */
 package org.wamblee.system.graph;
 
-import java.util.Arrays;
-
 import junit.framework.TestCase;
-
-import static org.mockito.Mockito.*; 
+import static org.mockito.Matchers.*;
+import static org.mockito.Mockito.*;
 
 import org.wamblee.system.container.Application;
 import org.wamblee.system.core.Component;
@@ -29,75 +27,83 @@ import org.wamblee.system.core.RequiredInterface;
 import org.wamblee.system.graph.component.ProvidedInterfaceNode;
 import org.wamblee.system.graph.component.RequiredInterfaceNode;
 
+/**
+ * 
+ * @author $author$
+ * @version $Revision$
+ */
 public class CompositeEdgeFilterTest extends TestCase {
-    private Application _app = new Application(); 
-    private Environment _env = new Environment();
-    
-    private Edge createEdge(Component aClient, RequiredInterface aRequired, 
-            Component aServer, ProvidedInterface aProvided) { 
+    private Application app = new Application();
+
+    private Environment env = new Environment();
+
+    private Edge createEdge(Component aClient, RequiredInterface aRequired,
+        Component aServer, ProvidedInterface aProvided) {
         Node from = new RequiredInterfaceNode(aClient, aRequired);
         Node to = new ProvidedInterfaceNode(aServer, aProvided);
+
         return new DefaultEdge(from, to);
     }
 
-    public void testEmpty() { 
-        EdgeFilter restriction = new CompositeEdgeFilter(); 
-        assertFalse(restriction.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
+    public void testEmpty() {
+        EdgeFilter restriction = new CompositeEdgeFilter();
+        assertFalse(restriction.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
     }
-    
-    private void configureRestriction(EdgeFilter base, boolean aResult) {
-    	stub(base.isViolated((Edge)anyObject())).toReturn(aResult);
+
+    private void configureRestriction(EdgeFilter aBase, boolean aResult) {
+        stub(aBase.isViolated((Edge) anyObject())).toReturn(aResult);
     }
-    
-    public void testOneRestriction() { 
+
+    public void testOneRestriction() {
         EdgeFilter base = mock(EdgeFilter.class);
         CompositeEdgeFilter composite = new CompositeEdgeFilter();
         composite.add(base);
-        
-        // First let the base return false and verify the result. 
-        
+
+        // First let the base return false and verify the result.
         configureRestriction(base, false);
-        
-        assertFalse(composite.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
-        
+
+        assertFalse(composite.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
+
         // Second let the base return true and verify the result.
         configureRestriction(base, true);
-        
-        assertTrue(composite.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
+
+        assertTrue(composite.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
     }
 
-   
-    
-    public void testTwoRestrictions() { 
+    public void testTwoRestrictions() {
         EdgeFilter base1 = mock(EdgeFilter.class);
         CompositeEdgeFilter composite = new CompositeEdgeFilter();
         composite.add(base1);
+
         EdgeFilter base2 = mock(EdgeFilter.class);
         composite.add(base2);
-        
-        // 1. base1 not violated and base 2 not violated -> not violated. 
-        
+
+        // 1. base1 not violated and base 2 not violated -> not violated.
         configureRestriction(base1, false);
         configureRestriction(base2, false);
-        assertFalse(composite.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
-        
+        assertFalse(composite.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
+
         // 2. base 1 not violated but base 2 violated -> violated
         configureRestriction(base1, false);
         configureRestriction(base2, true);
-       
-        assertTrue(composite.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
-     
-        // 3. base 1 violated -> violated and base 2 not called. 
+
+        assertTrue(composite.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
+
+        // 3. base 1 violated -> violated and base 2 not called.
         configureRestriction(base1, true);
         // base 2 should not be called.
-      
-        assertTrue(composite.isViolated(createEdge(_app, _app.getRequiredInterfaces().get(0), 
-                _env, _env.getProvidedInterfaces().get(0))));
+        assertTrue(composite.isViolated(createEdge(app, app
+            .getRequiredInterfaces().get(0), env, env.getProvidedInterfaces()
+            .get(0))));
     }
-    
 }