Remove methods deprecated in Drake from Instructions API
Change-Id: I87a20cb9e8abbf4b27bbb4760a62947169866ea6
diff --git a/core/api/src/main/java/org/onosproject/net/flow/instructions/Instructions.java b/core/api/src/main/java/org/onosproject/net/flow/instructions/Instructions.java
index 878d654..aef755f 100644
--- a/core/api/src/main/java/org/onosproject/net/flow/instructions/Instructions.java
+++ b/core/api/src/main/java/org/onosproject/net/flow/instructions/Instructions.java
@@ -72,17 +72,6 @@
}
/**
- * Creates a drop instruction.
- *
- * @return drop instruction
- * @deprecated 1.4.0 Emu Release
- */
- @Deprecated
- public static DropInstruction createDrop() {
- return new DropInstruction();
- }
-
- /**
* Creates a no action instruction.
*
* @return no action instruction
@@ -484,42 +473,6 @@
}
/**
- * Drop instruction.
- * @deprecated 1.4.0 Emu Release
- */
- @Deprecated
- public static final class DropInstruction implements Instruction {
-
- private DropInstruction() {}
-
- @Override
- public Type type() {
- return Type.DROP;
- }
-
- @Override
- public String toString() {
- return type().toString();
- }
-
- @Override
- public int hashCode() {
- return type().ordinal();
- }
-
- @Override
- public boolean equals(Object obj) {
- if (this == obj) {
- return true;
- }
- if (obj instanceof DropInstruction) {
- return true;
- }
- return false;
- }
- }
-
- /**
* No Action instruction.
*/
public static final class NoActionInstruction implements Instruction {
diff --git a/core/api/src/test/java/org/onosproject/net/flow/instructions/InstructionsTest.java b/core/api/src/test/java/org/onosproject/net/flow/instructions/InstructionsTest.java
index ee51c33..28b015c 100644
--- a/core/api/src/test/java/org/onosproject/net/flow/instructions/InstructionsTest.java
+++ b/core/api/src/test/java/org/onosproject/net/flow/instructions/InstructionsTest.java
@@ -101,7 +101,6 @@
*/
@Test
public void testImmutabilityOfInstructions() {
- assertThatClassIsImmutable(Instructions.DropInstruction.class);
assertThatClassIsImmutable(Instructions.OutputInstruction.class);
assertThatClassIsImmutable(L0ModificationInstruction.ModLambdaInstruction.class);
assertThatClassIsImmutable(L0ModificationInstruction.ModOchSignalInstruction.class);
@@ -155,40 +154,6 @@
assertThat(noAction1.hashCode(), is(equalTo(noAction2.hashCode())));
}
- // DropInstruction
-
- private final Instructions.DropInstruction drop1 = Instructions.createDrop();
- private final Instructions.DropInstruction drop2 = Instructions.createDrop();
-
- /**
- * Test the createDrop method.
- */
- @Test
- public void testCreateDropMethod() {
- Instructions.DropInstruction instruction = Instructions.createDrop();
- checkAndConvert(instruction,
- Instruction.Type.DROP,
- Instructions.DropInstruction.class);
- }
-
- /**
- * Test the equals() method of the DropInstruction class.
- */
-
- @Test
- public void testDropInstructionEquals() throws Exception {
- assertThat(drop1, is(equalTo(drop2)));
- }
-
- /**
- * Test the hashCode() method of the DropInstruction class.
- */
-
- @Test
- public void testDropInstructionHashCode() {
- assertThat(drop1.hashCode(), is(equalTo(drop2.hashCode())));
- }
-
// OutputInstruction
private final PortNumber port1 = portNumber(1);
diff --git a/core/common/src/main/java/org/onosproject/codec/impl/DecodeInstructionCodecHelper.java b/core/common/src/main/java/org/onosproject/codec/impl/DecodeInstructionCodecHelper.java
index 4878c17..fbb8c57 100644
--- a/core/common/src/main/java/org/onosproject/codec/impl/DecodeInstructionCodecHelper.java
+++ b/core/common/src/main/java/org/onosproject/codec/impl/DecodeInstructionCodecHelper.java
@@ -257,8 +257,8 @@
+ " is not supported");
}
return Instructions.createOutput(portNumber);
- } else if (type.equals(Instruction.Type.DROP.name())) {
- return Instructions.createDrop();
+ } else if (type.equals(Instruction.Type.NOACTION.name())) {
+ return Instructions.createNoAction();
} else if (type.equals(Instruction.Type.L0MODIFICATION.name())) {
return decodeL0();
} else if (type.equals(Instruction.Type.L1MODIFICATION.name())) {
diff --git a/core/common/src/test/java/org/onosproject/codec/impl/InstructionCodecTest.java b/core/common/src/test/java/org/onosproject/codec/impl/InstructionCodecTest.java
index 8d45f4c..1b7a501 100644
--- a/core/common/src/test/java/org/onosproject/codec/impl/InstructionCodecTest.java
+++ b/core/common/src/test/java/org/onosproject/codec/impl/InstructionCodecTest.java
@@ -73,18 +73,6 @@
}
/**
- * Tests the encoding of drop instructions.
- */
- @Test
- public void dropInstructionTest() {
- final Instructions.DropInstruction instruction =
- Instructions.createDrop();
- final ObjectNode instructionJson =
- instructionCodec.encode(instruction, context);
- assertThat(instructionJson, matchesInstruction(instruction));
- }
-
- /**
* Tests the encoding of output instructions.
*/
@Test
diff --git a/core/common/src/test/java/org/onosproject/codec/impl/InstructionJsonMatcher.java b/core/common/src/test/java/org/onosproject/codec/impl/InstructionJsonMatcher.java
index 3f65e8c..9c0a318 100644
--- a/core/common/src/test/java/org/onosproject/codec/impl/InstructionJsonMatcher.java
+++ b/core/common/src/test/java/org/onosproject/codec/impl/InstructionJsonMatcher.java
@@ -20,7 +20,6 @@
import org.onlab.util.HexString;
import org.onosproject.net.OduSignalId;
import org.onosproject.net.flow.instructions.Instruction;
-import org.onosproject.net.flow.instructions.Instructions.DropInstruction;
import org.onosproject.net.flow.instructions.Instructions.NoActionInstruction;
import org.onosproject.net.flow.instructions.Instructions.OutputInstruction;
import org.onosproject.net.flow.instructions.L0ModificationInstruction.ModLambdaInstruction;
@@ -452,8 +451,6 @@
if (instruction instanceof PushHeaderInstructions) {
return matchPushHeaderInstruction(jsonInstruction, description);
- } else if (instruction instanceof DropInstruction) {
- return true;
} else if (instruction instanceof OutputInstruction) {
return matchOutputInstruction(jsonInstruction, description);
} else if (instruction instanceof ModLambdaInstruction) {
diff --git a/core/common/src/test/resources/org/onosproject/codec/impl/FilteringObjective.json b/core/common/src/test/resources/org/onosproject/codec/impl/FilteringObjective.json
index 228e7b0..414e3d6 100644
--- a/core/common/src/test/resources/org/onosproject/codec/impl/FilteringObjective.json
+++ b/core/common/src/test/resources/org/onosproject/codec/impl/FilteringObjective.json
@@ -17,9 +17,9 @@
"port": -3
},
{
- "type": "DROP"
+ "type": "NOACTION"
}
],
"deferred": []
}
-}
\ No newline at end of file
+}
diff --git a/core/common/src/test/resources/org/onosproject/codec/impl/ForwardingObjective.json b/core/common/src/test/resources/org/onosproject/codec/impl/ForwardingObjective.json
index f3f53d4..e979626 100644
--- a/core/common/src/test/resources/org/onosproject/codec/impl/ForwardingObjective.json
+++ b/core/common/src/test/resources/org/onosproject/codec/impl/ForwardingObjective.json
@@ -17,8 +17,8 @@
"instructions":
[
{"type":"OUTPUT","port":-3},
- {"type":"DROP"}
+ {"type":"NOACTION"}
],
"deferred":[]
}
-}
\ No newline at end of file
+}
diff --git a/core/common/src/test/resources/org/onosproject/codec/impl/NextObjective.json b/core/common/src/test/resources/org/onosproject/codec/impl/NextObjective.json
index dad9b03..6f52f11 100644
--- a/core/common/src/test/resources/org/onosproject/codec/impl/NextObjective.json
+++ b/core/common/src/test/resources/org/onosproject/codec/impl/NextObjective.json
@@ -10,7 +10,7 @@
"port": -3
},
{
- "type": "DROP"
+ "type": "NOACTION"
}
],
"deferred": []
@@ -24,4 +24,4 @@
}
]
}
-}
\ No newline at end of file
+}
diff --git a/core/common/src/test/resources/org/onosproject/codec/impl/instructions-flow.json b/core/common/src/test/resources/org/onosproject/codec/impl/instructions-flow.json
index 2a184b3..fb3c03b 100644
--- a/core/common/src/test/resources/org/onosproject/codec/impl/instructions-flow.json
+++ b/core/common/src/test/resources/org/onosproject/codec/impl/instructions-flow.json
@@ -8,7 +8,7 @@
"instructions":
[
{"type":"OUTPUT","port":-3},
- {"type":"DROP"},
+ {"type":"NOACTION"},
{"type":"L2MODIFICATION","subtype":"ETH_SRC","mac":"12:34:56:78:90:12"},
{"type":"L2MODIFICATION","subtype":"ETH_DST","mac":"98:76:54:32:01:00"},
{"type":"L2MODIFICATION","subtype":"VLAN_ID","vlanId":22},
diff --git a/core/store/serializers/src/main/java/org/onosproject/store/serializers/KryoNamespaces.java b/core/store/serializers/src/main/java/org/onosproject/store/serializers/KryoNamespaces.java
index 624bcd1..ae27fc2 100644
--- a/core/store/serializers/src/main/java/org/onosproject/store/serializers/KryoNamespaces.java
+++ b/core/store/serializers/src/main/java/org/onosproject/store/serializers/KryoNamespaces.java
@@ -382,7 +382,6 @@
Criterion.class,
Criterion.Type.class,
DefaultTrafficTreatment.class,
- Instructions.DropInstruction.class,
Instructions.NoActionInstruction.class,
Instructions.OutputInstruction.class,
Instructions.GroupInstruction.class,
diff --git a/drivers/default/src/main/java/org/onosproject/driver/pipeline/DefaultSingleTablePipeline.java b/drivers/default/src/main/java/org/onosproject/driver/pipeline/DefaultSingleTablePipeline.java
index baa08b8..2792791 100644
--- a/drivers/default/src/main/java/org/onosproject/driver/pipeline/DefaultSingleTablePipeline.java
+++ b/drivers/default/src/main/java/org/onosproject/driver/pipeline/DefaultSingleTablePipeline.java
@@ -118,7 +118,7 @@
(fwd.treatment().tableTransition() == null) &&
(!fwd.treatment().clearedDeferred())) {
TrafficTreatment.Builder flowTreatment = DefaultTrafficTreatment.builder();
- flowTreatment.add(Instructions.createDrop());
+ flowTreatment.add(Instructions.createNoAction());
treatment = flowTreatment.build();
}
diff --git a/drivers/default/src/main/java/org/onosproject/driver/pipeline/OpenVSwitchPipeline.java b/drivers/default/src/main/java/org/onosproject/driver/pipeline/OpenVSwitchPipeline.java
index 1a7f576..df910dd 100644
--- a/drivers/default/src/main/java/org/onosproject/driver/pipeline/OpenVSwitchPipeline.java
+++ b/drivers/default/src/main/java/org/onosproject/driver/pipeline/OpenVSwitchPipeline.java
@@ -293,7 +293,7 @@
// MAC table flow rules
if ((selector.getCriterion(Type.TUNNEL_ID) != null && selector
.getCriterion(Type.ETH_DST) != null)
- || tb.allInstructions().contains(Instructions.createDrop())) {
+ || tb.allInstructions().contains(Instructions.createNoAction())) {
forTable = MAC_TABLE;
return reassemblyFlowRule(ruleBuilder, tb, transition, forTable);
}
diff --git a/web/api/src/test/resources/org/onosproject/rest/post-filter-objective.json b/web/api/src/test/resources/org/onosproject/rest/post-filter-objective.json
index 665539d..e4a6128 100644
--- a/web/api/src/test/resources/org/onosproject/rest/post-filter-objective.json
+++ b/web/api/src/test/resources/org/onosproject/rest/post-filter-objective.json
@@ -18,8 +18,8 @@
"port": -3
},
{
- "type": "DROP"
+ "type": "NOACTION"
}
]
}
-}
\ No newline at end of file
+}
diff --git a/web/api/src/test/resources/org/onosproject/rest/post-forward-objective.json b/web/api/src/test/resources/org/onosproject/rest/post-forward-objective.json
index 634424d..8d660fc 100644
--- a/web/api/src/test/resources/org/onosproject/rest/post-forward-objective.json
+++ b/web/api/src/test/resources/org/onosproject/rest/post-forward-objective.json
@@ -17,7 +17,7 @@
"instructions":
[
{"type":"OUTPUT","port":-3},
- {"type":"DROP"}
+ {"type":"NOACTION"}
]
}
-}
\ No newline at end of file
+}
diff --git a/web/api/src/test/resources/org/onosproject/rest/post-next-objective.json b/web/api/src/test/resources/org/onosproject/rest/post-next-objective.json
index 1b19216..17fc203 100644
--- a/web/api/src/test/resources/org/onosproject/rest/post-next-objective.json
+++ b/web/api/src/test/resources/org/onosproject/rest/post-next-objective.json
@@ -10,7 +10,7 @@
"port": -3
},
{
- "type": "DROP"
+ "type": "NOACTION"
}
]
}
@@ -23,4 +23,4 @@
}
]
}
-}
\ No newline at end of file
+}