Merge pull request #41 from VariantSync/benjamin/assertions

Always check assertions

Author: pmbittner

src/main/java/org/variantsync/diffdetective/diff/difftree/DiffNode.java

@@ -363,12 +363,12 @@ public void drop() {
     }
 
     private void dropBeforeChild(final DiffNode child) {
-        assert child.beforeParent == this;
+        Assert.assertTrue(child.beforeParent == this);
         child.beforeParent = null;
     }
 
     private void dropAfterChild(final DiffNode child) {
-        assert child.afterParent == this;
+        Assert.assertTrue(child.afterParent == this);
         child.afterParent = null;
     }
 

src/main/java/org/variantsync/diffdetective/diff/difftree/transform/CollapseNestedNonEditedMacros.java

@@ -7,6 +7,7 @@
 import org.variantsync.diffdetective.diff.difftree.DiffTree;
 import org.variantsync.diffdetective.diff.difftree.DiffType;
 import org.variantsync.diffdetective.diff.difftree.traverse.DiffTreeTraversal;
+import org.variantsync.diffdetective.util.Assert;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -45,7 +46,7 @@ public void transform(final DiffTree diffTree) {
 
         // All found chains should at least have size 2.
         for (final Stack<DiffNode> chain : chains) {
-            assert chain.size() >= 2;
+            Assert.assertTrue(chain.size() >= 2);
         }
 
 //        System.out.println(StringUtils.prettyPrintNestedCollections(chains));
@@ -97,7 +98,7 @@ private static void collapseChain(Stack<DiffNode> chain) {
         final ArrayList<Node> featureMappings = new ArrayList<>(chain.size());
 
         DiffNode lastPopped = null;
-        assert !chain.isEmpty();
+        Assert.assertTrue(!chain.isEmpty());
         while (!chain.isEmpty()) {
             lastPopped = chain.pop();
 
@@ -118,7 +119,7 @@ private static void collapseChain(Stack<DiffNode> chain) {
             }
         }
 
-        assert head == lastPopped;
+        Assert.assertTrue(head == lastPopped);
 
         final DiffNode beforeParent = head.getBeforeParent();
         final DiffNode afterParent = head.getAfterParent();

src/main/java/org/variantsync/diffdetective/internal/SimpleRenderer.java

@@ -14,6 +14,7 @@
 import org.variantsync.diffdetective.mining.DiffTreeMiner;
 import org.variantsync.diffdetective.mining.RWCompositePatternNodeFormat;
 import org.variantsync.diffdetective.mining.RWCompositePatternTreeFormat;
+import org.variantsync.diffdetective.util.Assert;
 import org.variantsync.diffdetective.util.FileUtils;
 
 import java.io.IOException;
@@ -156,7 +157,7 @@ public static void main(String[] args) throws IOException {
 
             final List<DiffTreeTransformer> transform = DiffTreeMiner.Postprocessing(repository);
             final PatchDiff patch = DiffTreeParser.parsePatch(repository, file, commit);
-            assert patch != null;
+            Assert.assertNotNull(patch != null);
             DiffTreeTransformer.apply(transform, patch.getDiffTree());
             renderer.render(patch, Path.of("render", repoName), RENDER_OPTIONS_TO_USE);
         }