Fixing the setting of the DSCP_TOSOverwrite value with the DSCPTOS object to write...

The setter that uses the byte value was correct and set index 1 where the other setter was setting the byte value to index 2.
Gate COPS data will need additional unit testing and really should be refactored for normalization and semantics.

Change-Id: I719f186d661b3749ed7d9f9f3bef20a7323d4910
Signed-off-by: Steven Pisarski <s.pisarski@cablelabs.com>

diff --git a/packetcable-driver/src/main/java/org/pcmm/gates/impl/GateSpec.java b/packetcable-driver/src/main/java/org/pcmm/gates/impl/GateSpec.java
index 914855852403819e6b32cf194e6f62d2eeb5c97c..c802d32e954dd1e50788be17fe06c0d362d7e4a1 100644 (file)
--- a/packetcable-driver/src/main/java/org/pcmm/gates/impl/GateSpec.java
+++ b/packetcable-driver/src/main/java/org/pcmm/gates/impl/GateSpec.java
@@ -90,13 +90,13 @@ public class GateSpec extends PCMMBaseObject implements IGateSpec {
     @Override
     public void setDSCP_TOSOverwrite(DSCPTOS dscpTos) {
         // OR in the DSCPTOS enable flag with the Direction flag
-        flags |= dscpTos.getValue();
-        setByte(flags, (short) 0);
+        setDSCP_TOSOverwrite(dscpTos.getValue());
     }
 
     @Override
     public void setDSCP_TOSOverwrite(byte dscpTos) {
-        setByte(dscpTos, (short) 1);
+        flags |= dscpTos;
+        setByte(flags, (short) 1);
     }