Remove deprecated non-filtered ingress and egress connect point APIs
Change-Id: Ie360dfc2288b27587ffcdb23bc29f266df1a8d73
diff --git a/core/api/src/main/java/org/onosproject/net/intent/LinkCollectionIntent.java b/core/api/src/main/java/org/onosproject/net/intent/LinkCollectionIntent.java
index 91db96d..40f13d0 100644
--- a/core/api/src/main/java/org/onosproject/net/intent/LinkCollectionIntent.java
+++ b/core/api/src/main/java/org/onosproject/net/intent/LinkCollectionIntent.java
@@ -172,44 +172,6 @@
}
/**
- * Sets the ingress point of the single point to multi point intent
- * that will be built.
- *
- * @param ingressPoints ingress connect points
- * @return this builder
- */
- @Deprecated
- public Builder ingressPoints(Set<ConnectPoint> ingressPoints) {
- if (this.ingressPoints != null) {
- log.warn("Ingress points are already set, " +
- "this will override original ingress points.");
- }
- this.ingressPoints = ingressPoints.stream()
- .map(FilteredConnectPoint::new)
- .collect(Collectors.toSet());
- return this;
- }
-
- /**
- * Sets the egress points of the single point to multi point intent
- * that will be built.
- *
- * @param egressPoints egress connect points
- * @return this builder
- */
- @Deprecated
- public Builder egressPoints(Set<ConnectPoint> egressPoints) {
- if (this.egressPoints != null) {
- log.warn("Egress points are already set, " +
- "this will override original egress points.");
- }
- this.egressPoints = egressPoints.stream()
- .map(FilteredConnectPoint::new)
- .collect(Collectors.toSet());
- return this;
- }
-
- /**
* Sets the filtered ingress point of the single point to multi point intent
* that will be built.
*
diff --git a/core/api/src/test/java/org/onosproject/net/intent/LinkCollectionIntentTest.java b/core/api/src/test/java/org/onosproject/net/intent/LinkCollectionIntentTest.java
index b79328b..99b5e69 100644
--- a/core/api/src/test/java/org/onosproject/net/intent/LinkCollectionIntentTest.java
+++ b/core/api/src/test/java/org/onosproject/net/intent/LinkCollectionIntentTest.java
@@ -78,8 +78,8 @@
.selector(selector)
.treatment(treatment)
.links(links1)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.resourceGroup(resourceGroup)
.build();
@@ -91,8 +91,8 @@
.selector(selector)
.treatment(treatment)
.links(links2)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.resourceGroup(resourceGroup)
.build();
@@ -115,8 +115,8 @@
.selector(selector)
.treatment(treatment)
.links(links1)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.resourceGroup(resourceGroup)
.build();
@@ -149,8 +149,8 @@
.selector(selector)
.treatment(treatment)
.links(links1)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.constraints(constraints)
.priority(8888)
.build();
@@ -231,8 +231,8 @@
.selector(selector)
.treatment(treatment)
.links(links1)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.build();
}
@@ -245,8 +245,8 @@
.selector(selector)
.treatment(treatment)
.links(links2)
- .ingressPoints(ImmutableSet.of(ingress))
- .egressPoints(ImmutableSet.of(egress))
+ .filteredIngressPoints(ImmutableSet.of(new FilteredConnectPoint(ingress)))
+ .filteredEgressPoints(ImmutableSet.of(new FilteredConnectPoint(egress)))
.build();
}