added tests for modifiable integers and respective modifcations

src/main/java/de/rub/nds/modifiablevariable/biginteger/BigIntegerExplicitValueModification.java

@@ -38,6 +38,9 @@
 
     @Override
     protected BigInteger modifyImplementationHook(BigInteger input) {
+        if (input == null) {
+            throw new NullPointerException("original value must not be null");
+        }
         return explicitValue;
     }
 

src/main/java/de/rub/nds/modifiablevariable/bool/BooleanExplicitValueModification.java

@@ -36,6 +36,9 @@
 
     @Override
     protected Boolean modifyImplementationHook(Boolean input) {
+        if (input == null) {
+            throw new NullPointerException("original value must not be null");
+        }
         return explicitValue;
     }
 

src/main/java/de/rub/nds/modifiablevariable/bool/BooleanToggleModification.java

@@ -28,9 +28,6 @@ public BooleanToggleModification createCopy() {
 
     @Override
     protected Boolean modifyImplementationHook(Boolean input) {
-        if (input == null) {
-            input = Boolean.FALSE;
-        }
         return !input;
     }
 

src/main/java/de/rub/nds/modifiablevariable/bytearray/ByteArrayAppendValueModification.java

@@ -41,10 +41,6 @@ public ByteArrayAppendValueModification createCopy() {
 
     @Override
     protected byte[] modifyImplementationHook(byte[] input) {
-        if (input == null) {
-            input = new byte[0];
-        }
-
         return ArrayConverter.concatenate(input, bytesToAppend);
     }
 

src/main/java/de/rub/nds/modifiablevariable/bytearray/ByteArrayDeleteModification.java

@@ -42,9 +42,6 @@ public ByteArrayDeleteModification createCopy() {
 
     @Override
     protected byte[] modifyImplementationHook(byte[] input) {
-        if (input == null) {
-            input = new byte[0];
-        }
         if (input.length == 0) {
             return input;
         }

src/main/java/de/rub/nds/modifiablevariable/bytearray/ByteArrayExplicitValueModification.java

@@ -41,6 +41,9 @@
 
     @Override
     protected byte[] modifyImplementationHook(byte[] input) {
+        if (input == null) {
+            throw new NullPointerException("original value must not be null");
+        }
         return explicitValue.clone();
     }
 

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerAddModification.java

@@ -37,7 +37,7 @@ public IntegerAddModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? summand : input + summand;
+        return input + summand;
     }
 
     public Integer getSummand() {

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerExplicitValueModification.java

@@ -37,6 +37,9 @@ public IntegerExplicitValueModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
+        if (input == null) {
+            throw new NullPointerException("original value must not be null");
+        }
         return explicitValue;
     }
 

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerMultiplyModification.java

@@ -37,7 +37,7 @@ public IntegerMultiplyModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? 0 : input * factor;
+        return input * factor;
     }
 
     public Integer getFactor() {

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerShiftLeftModification.java

@@ -38,7 +38,7 @@ public IntegerShiftLeftModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? 0 : input << shift % MAX_SHIFT_MODIFIER;
+        return input << shift % MAX_SHIFT_MODIFIER;
     }
 
     public int getShift() {

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerShiftRightModification.java

@@ -38,7 +38,7 @@ public IntegerShiftRightModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? 0 : input >> shift % MAX_SHIFT_MODIFIER;
+        return input >> shift % MAX_SHIFT_MODIFIER;
     }
 
     public int getShift() {

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerSubtractModification.java

@@ -37,7 +37,7 @@ public IntegerSubtractModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? -subtrahend : input - subtrahend;
+        return input - subtrahend;
     }
 
     public Integer getSubtrahend() {

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerSwapEndianModification.java

@@ -28,9 +28,6 @@ public IntegerSwapEndianModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        if (input == null) {
-            return null;
-        }
         return Integer.reverseBytes(input);
     }
 

src/main/java/de/rub/nds/modifiablevariable/integer/IntegerXorModification.java

@@ -37,7 +37,7 @@ public IntegerXorModification createCopy() {
 
     @Override
     protected Integer modifyImplementationHook(Integer input) {
-        return input == null ? xor : input ^ xor;
+        return input ^ xor;
     }
 
     public Integer getXor() {

src/main/java/de/rub/nds/modifiablevariable/integer/ModifiableInteger.java

@@ -45,7 +45,11 @@ public void setAssertEquals(Integer assertEquals) {
 
     @Override
     public boolean isOriginalValueModified() {
-        return getOriginalValue() != null && getOriginalValue().compareTo(getValue()) != 0;
+        if (getOriginalValue() == null) {
+            throw new IllegalStateException("Original value must not be null");
+        } else {
+            return getOriginalValue().compareTo(getValue()) != 0;
+        }
     }
 
     public byte[] getByteArray(int size) {