wealthfront#10 Fix test to account for random ordering of classes

kevinconaway · Jan 29, 2016 · 11d445d · 11d445d
1 parent 1536a9a
commit 11d445d
Showing 1 changed file with 27 additions and 4 deletions.
diff --git a/kawala-testing/src/test/java/com/kaching/platform/testing/DependencyTestRunnerTest.java b/kawala-testing/src/test/java/com/kaching/platform/testing/DependencyTestRunnerTest.java
@@ -13,6 +13,7 @@
 import static com.google.common.collect.Lists.newArrayList;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.fail;
 import jdepend.framework.JavaClass;
 import jdepend.framework.JavaPackage;
@@ -183,11 +184,33 @@ public void violationToString() {
     classB.addImportedPackage(violation.efferent);
     violation.javaPackage.addClass(classB);
 
-    violation.javaPackage.addClass(new JavaClass("a.BigC"));
+    JavaClass classC = new JavaClass("a.BigC");
+    violation.javaPackage.addClass(classC);
+
+    String message = violation.toString();
+
+    String packageName = violation.javaPackage.getName();
+
+    assertTrue(message, message.contains("package a cannot depend on package b"));
+    assertTrue(
+        message,
+        message.contains(
+            classA.getName().substring(packageName.length() + 1)
+        )
+    );
+    assertTrue(
+        message,
+        message.contains(
+            classB.getName().substring(packageName.length() + 1)
+        )
+    );
+    assertFalse(
+        message,
+        message.contains(
+            classC.getName().substring(packageName.length() + 1)
+        )
+    );
 
-    assertEquals(
-        "package a cannot depend on package b (classes BigA, BigB)",
-        violation.toString());
   }
 
   @Test