commit 44f3761e4cc4304d5d56b7611cc6c1f69e5b2032
author Sho SHIMIZU <sshimizu@us.fujitsu.com> Wed Nov 25 16:23:22 2015 -0800
committer Sho Shimizu <sho.shimizu@gmail.com> Wed Dec 02 17:41:04 2015 +0000
tree cfc042b5554a815c585d545d2aa5d9e8ed6e9748
parent e3af26514753915decfd0e5afb3e7ba347f03e9c

Abondon the concept of resources under a link

Resources under a link are tied to resources under both ends of the link,
and resources under a port are thought to be first-class objects compared
to concept of link resources. We will deal with only device related
resources from now on.

Change-Id: I6aa418d1bf64b28374f325db0bc7e393f770dcdd

core/api/src/test/java/org/onosproject/net/newresource/ResourcePathTest.java

diff --git a/core/api/src/test/java/org/onosproject/net/newresource/ResourcePathTest.java b/core/api/src/test/java/org/onosproject/net/newresource/ResourcePathTest.java
index 35dcf1e..15b457c 100644
--- a/core/api/src/test/java/org/onosproject/net/newresource/ResourcePathTest.java
+++ b/core/api/src/test/java/org/onosproject/net/newresource/ResourcePathTest.java
@@ -19,9 +19,7 @@
 import org.junit.Test;
 import org.onlab.packet.VlanId;
 import org.onlab.util.Bandwidth;
-import org.onosproject.net.ConnectPoint;
 import org.onosproject.net.DeviceId;
-import org.onosproject.net.LinkKey;
 import org.onosproject.net.PortNumber;
 
 import java.util.Optional;
@@ -35,19 +33,17 @@
     private static final DeviceId D1 = DeviceId.deviceId("of:001");
     private static final DeviceId D2 = DeviceId.deviceId("of:002");
     private static final PortNumber P1 = PortNumber.portNumber(1);
-    private static final ConnectPoint CP1_1 = new ConnectPoint(D1, P1);
-    private static final ConnectPoint CP2_1 = new ConnectPoint(D2, P1);
     private static final VlanId VLAN1 = VlanId.vlanId((short) 100);
     private static final Bandwidth BW1 = Bandwidth.gbps(2);
     private static final Bandwidth BW2 = Bandwidth.gbps(1);
 
     @Test
     public void testEquals() {
-        ResourcePath resource1 = ResourcePath.discrete(LinkKey.linkKey(CP1_1, CP2_1), VLAN1);
-        ResourcePath sameAsResource1 = ResourcePath.discrete(LinkKey.linkKey(CP1_1, CP2_1), VLAN1);
-        ResourcePath resource2 = ResourcePath.discrete(LinkKey.linkKey(CP2_1, CP1_1), VLAN1);
-        ResourcePath resource3 = ResourcePath.continuous(BW1.bps(), LinkKey.linkKey(CP1_1, CP2_1), BW1);
-        ResourcePath sameAsResource3 = ResourcePath.continuous(BW2.bps(), LinkKey.linkKey(CP1_1, CP2_1), BW1);
+        ResourcePath resource1 = ResourcePath.discrete(D1, P1, VLAN1);
+        ResourcePath sameAsResource1 = ResourcePath.discrete(D1, P1, VLAN1);
+        ResourcePath resource2 = ResourcePath.discrete(D2, P1, VLAN1);
+        ResourcePath resource3 = ResourcePath.continuous(BW1.bps(), D1, P1, BW1);
+        ResourcePath sameAsResource3 = ResourcePath.continuous(BW2.bps(), D1, P1, BW1);
 
         new EqualsTester()
                 .addEqualityGroup(resource1, sameAsResource1)
@@ -72,25 +68,24 @@
 
     @Test
     public void testThereIsParent() {
-        ResourcePath path = ResourcePath.discrete(LinkKey.linkKey(CP1_1, CP2_1), VLAN1);
-        ResourcePath parent = ResourcePath.discrete(LinkKey.linkKey(CP1_1, CP2_1));
+        ResourcePath path = ResourcePath.discrete(D1, P1, VLAN1);
+        ResourcePath parent = ResourcePath.discrete(D1, P1);
 
         assertThat(path.parent(), is(Optional.of(parent)));
     }
 
     @Test
     public void testNoParent() {
-        ResourcePath path = ResourcePath.discrete(LinkKey.linkKey(CP1_1, CP2_1));
+        ResourcePath path = ResourcePath.discrete(D1);
 
         assertThat(path.parent(), is(Optional.of(ResourcePath.ROOT)));
     }
 
     @Test
     public void testBase() {
-        LinkKey linkKey = LinkKey.linkKey(CP1_1, CP2_1);
-        ResourcePath path = ResourcePath.discrete(linkKey);
+        ResourcePath path = ResourcePath.discrete(D1);
 
-        LinkKey child = (LinkKey) path.last();
-        assertThat(child, is(linkKey));
+        DeviceId child = (DeviceId) path.last();
+        assertThat(child, is(D1));
     }
 }