From 18f1df454ea6f5d1ee46d51c5c620cbce0d43474 Mon Sep 17 00:00:00 2001
From: Cedric Beust <cedric.beust@le.com>
Date: Tue, 4 Apr 2017 10:50:02 -0700
Subject: [PATCH] Fix DependencyManagerTest.

---
 .../com/beust/kobalt/maven/DependencyManagerTest.kt    | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt b/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt
index a6f0e9f3..278350dd 100644
--- a/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt
+++ b/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt
@@ -16,7 +16,7 @@ import org.testng.annotations.Test
 
 @Guice(modules = arrayOf(TestModule::class))
 class DependencyManagerTest @Inject constructor(val dependencyManager: DependencyManager,
-        compilerFactory: BuildFileCompiler.IFactory) : BaseTest(compilerFactory) {
+        compilerFactory: BuildFileCompiler.IFactory) : BaseTest() {
 
     private fun assertContains(dependencies: List<IClasspathDependency>, vararg ids: String) {
         ids.forEach { id ->
@@ -55,10 +55,12 @@ class DependencyManagerTest @Inject constructor(val dependencyManager: Dependenc
         // Should resolve to TestNG and its dependencies
         dependencyManager.transitiveClosure(testDeps).let { dependencies ->
             assertContains(dependencies, ":jcommander:")
-            assertContains(dependencies, ":bsh:")
-            assertContains(dependencies, ":ant:")
-            assertContains(dependencies, ":ant-launcher:")
             assertContains(dependencies, ":testng:")
+
+            // Optional dependencies
+            assertDoesNotContain(dependencies, ":bsh:")
+            assertDoesNotContain(dependencies, ":ant:")
+            assertDoesNotContain(dependencies, ":ant-launcher:")
         }
     }