commit 64a0369dfb0efd482a90c54bdd64469039683b01
author Brian O'Connor <bocon@onlab.us> Fri Feb 20 22:02:59 2015 -0800
committer Brian O'Connor <bocon@onlab.us> Fri Feb 20 22:15:46 2015 -0800
tree 9eba68d6d5720e816de56906494fe109425df23b
parent 1ee563a50312465e85693510ade84b01cb145422

IntentInstaller: changing from Set to Collection to improve perf

Change-Id: Ia7f0f7d893645c7528ac9f51acff133f6d82383d

core/net/src/test/java/org/onosproject/net/intent/impl/IntentManagerTest.java

diff --git a/core/net/src/test/java/org/onosproject/net/intent/impl/IntentManagerTest.java b/core/net/src/test/java/org/onosproject/net/intent/impl/IntentManagerTest.java
index f55b575..642a66f 100644
--- a/core/net/src/test/java/org/onosproject/net/intent/impl/IntentManagerTest.java
+++ b/core/net/src/test/java/org/onosproject/net/intent/impl/IntentManagerTest.java
@@ -194,7 +194,7 @@
 
     private static class TestIntentInstaller implements IntentInstaller<MockInstallableIntent> {
         @Override
-        public List<Set<org.onosproject.net.flow.FlowRuleOperation>> install(MockInstallableIntent intent) {
+        public List<Collection<org.onosproject.net.flow.FlowRuleOperation>> install(MockInstallableIntent intent) {
             FlowRule fr = new IntentTestsMocks.MockFlowRule(intent.number().intValue());
             Set<FlowRuleOperation> rules = ImmutableSet.of(
                     new FlowRuleOperation(fr, FlowRuleOperation.Type.ADD));
@@ -202,7 +202,7 @@
         }
 
         @Override
-        public List<Set<FlowRuleOperation>> uninstall(MockInstallableIntent intent) {
+        public List<Collection<FlowRuleOperation>> uninstall(MockInstallableIntent intent) {
             FlowRule fr = new IntentTestsMocks.MockFlowRule(intent.number().intValue());
             Set<FlowRuleOperation> rules = ImmutableSet.of(
                     new FlowRuleOperation(fr, FlowRuleOperation.Type.REMOVE));
@@ -210,7 +210,8 @@
         }
 
         @Override
-        public List<Set<FlowRuleOperation>> replace(MockInstallableIntent oldIntent, MockInstallableIntent newIntent) {
+        public List<Collection<FlowRuleOperation>> replace(MockInstallableIntent oldIntent,
+                                                           MockInstallableIntent newIntent) {
             FlowRule fr = new IntentTestsMocks.MockFlowRule(oldIntent.number().intValue());
             FlowRule fr2 = new IntentTestsMocks.MockFlowRule(newIntent.number().intValue());
             Set<FlowRuleOperation> rules = ImmutableSet.of(
@@ -222,17 +223,18 @@
 
     private static class TestIntentErrorInstaller implements IntentInstaller<MockInstallableIntent> {
         @Override
-        public List<Set<FlowRuleOperation>> install(MockInstallableIntent intent) {
+        public List<Collection<FlowRuleOperation>> install(MockInstallableIntent intent) {
             throw new IntentInstallationException("install() always fails");
         }
 
         @Override
-        public List<Set<FlowRuleOperation>> uninstall(MockInstallableIntent intent) {
+        public List<Collection<FlowRuleOperation>> uninstall(MockInstallableIntent intent) {
             throw new IntentRemovalException("uninstall() always fails");
         }
 
         @Override
-        public List<Set<FlowRuleOperation>> replace(MockInstallableIntent oldIntent, MockInstallableIntent newIntent) {
+        public List<Collection<FlowRuleOperation>> replace(MockInstallableIntent oldIntent,
+                                                           MockInstallableIntent newIntent) {
             throw new IntentInstallationException("replace() always fails");
         }
     }