Upgrade TestNG 6.8.8, get rid of transitive JUnit

Author: Stephane Landelle

pom.xml

@@ -126,9 +126,8 @@
         <dependency>
             <groupId>org.testng</groupId>
             <artifactId>testng</artifactId>
-            <version>5.8</version>
+            <version>6.8.8</version>
             <scope>test</scope>
-            <classifier>jdk15</classifier>
         </dependency>
         <dependency>
             <groupId>org.eclipse.jetty</groupId>

src/test/java/com/ning/http/client/async/SimpleAsyncHttpClientTest.java

@@ -14,9 +14,7 @@
 
 import static java.nio.charset.StandardCharsets.*;
 
-import static junit.framework.Assert.assertTrue;
-import static org.testng.Assert.assertEquals;
-import static org.testng.Assert.fail;
+import static org.testng.Assert.*;
 import static org.testng.AssertJUnit.assertNotNull;
 import static org.testng.AssertJUnit.assertNotSame;
 
@@ -220,11 +218,12 @@ public void onBytesReceived(Uri uri, long amount, long current, long total) {
 
     @Test(groups = { "standalone", "default_provider" })
     public void testNullUrl() throws Exception {
-        SimpleAsyncHttpClient client = new SimpleAsyncHttpClient.Builder().setProviderClass(getProviderClass()).build().derive().build();
+        SimpleAsyncHttpClient client = null;
         try {
-            assertTrue(true);
+            client = new SimpleAsyncHttpClient.Builder().setProviderClass(getProviderClass()).build().derive().build();
         } finally {
-            client.close();
+            if (client != null)
+                client.close();
         }
     }