Add logger implementation to netconf-impl test scope

Also rethrow exceptions on failures in ConcurrentClientsTest to
make issues more apparent.

Change-Id: I3e907568c6a384f9512a81c05ae798e11972f607
Signed-off-by: Tomas Cere <tcere@cisco.com>

diff --git a/netconf/netconf-impl/pom.xml b/netconf/netconf-impl/pom.xml
index 2f18c3e9d17323866b5cb37cc17ea562a9e9c9f3..00f9712956fe99c057e8d0b818fc5ca84bcc86f6 100644 (file)
--- a/netconf/netconf-impl/pom.xml
+++ b/netconf/netconf-impl/pom.xml
@@ -92,6 +92,11 @@
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.slf4j</groupId>
+      <artifactId>slf4j-simple</artifactId>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>${project.groupId}</groupId>
       <artifactId>netconf-client</artifactId>

diff --git a/netconf/netconf-impl/src/test/java/org/opendaylight/netconf/impl/ConcurrentClientsTest.java b/netconf/netconf-impl/src/test/java/org/opendaylight/netconf/impl/ConcurrentClientsTest.java
index 420a8f5d59ec0a6eca3dd299f737cd22ac44af25..089fd7875108ae5fc6e3a62845680d670c0825ce 100644 (file)
--- a/netconf/netconf-impl/src/test/java/org/opendaylight/netconf/impl/ConcurrentClientsTest.java
+++ b/netconf/netconf-impl/src/test/java/org/opendaylight/netconf/impl/ConcurrentClientsTest.java
@@ -210,7 +210,7 @@ public class ConcurrentClientsTest {
                 throw new IllegalStateException(e);
             } catch (ExecutionException e) {
                 LOG.error("Thread for testing client failed", e);
-                fail("Client failed: " + e.getMessage());
+                throw e;
             }
         }