Improve testCloseOnUndeploy

Sometimes we can see a stack trace in build logs.

It happens because the test completes before the verticle is fully undeployed.
So when the tearDown method closes vertx, another attempt to deploy the verticle is made, which prints a failure to the console, because the original verticle undeployment has been completed concurrently.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

Author: tsegismont

vertx-sql-client/src/test/java/io/vertx/sqlclient/tck/ConnectionTestBase.java

@@ -105,19 +105,19 @@ public void start() throws Exception {
 
   @Test
   public void testCloseOnUndeploy(TestContext ctx) {
-    Async done = ctx.async();
+    Async done = ctx.async(2);
     vertx.deployVerticle(new AbstractVerticle() {
       @Override
       public void start(Promise<Void> startPromise) throws Exception {
         connect(ctx.asyncAssertSuccess(conn -> {
           conn.closeHandler(v -> {
-            done.complete();
+            done.countDown();
           });
           startPromise.complete();
         }));
       }
     }).onComplete(ctx.asyncAssertSuccess(id -> {
-      vertx.undeploy(id);
+      vertx.undeploy(id).onComplete(ctx.asyncAssertSuccess(v -> done.countDown()));
     }));
   }