{WIP} fix jetcd-all which was broken under OSGi (etcd-io#382)

TODO With the ClientServiceTest actually "doing something" now (before
it dit not really), it only works if you have an etcd locally running
("sudo systemctl start etcd") ... we would need to actually start a
server, like the tests in jetcd-core do, for the ClientServiceTest to
pass on CI.

jetcd-all/pom.xml

@@ -127,6 +127,7 @@
                         </Export-Package>
                         <Import-Package>
                             org.slf4j;version="[1.7,2)",
+                            javax.security.cert,
                             javax.net.ssl;resolution:=optional
                         </Import-Package>
                     </instructions>

jetcd-osgi/jetcd-karaf/src/test/java/io/etcd/jetcd/osgi/ClientServiceTest.java

@@ -30,8 +30,11 @@
 import static org.ops4j.pax.exam.karaf.options.KarafDistributionOption.logLevel;
 
 import io.etcd.jetcd.Client;
+import io.etcd.jetcd.data.ByteSequence;
 import io.etcd.jetcd.resolver.URIResolver;
 import java.io.File;
+import java.nio.charset.Charset;
+import java.util.concurrent.TimeUnit;
 import javax.inject.Inject;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -52,6 +55,7 @@
 @RunWith(PaxExam.class)
 @ExamReactorStrategy(PerClass.class)
 public class ClientServiceTest extends TestSupport {
+
   @Inject
   protected BundleContext bundleContext;
 
@@ -109,7 +113,7 @@ public Option[] config() {
             .artifactId("assertj-core")
             .versionAsInProject()
             .start(),
-        editConfigurationFilePut("etc/io.etcd.jetcd.cfg", "endpoints", "localhost:2379"),
+        editConfigurationFilePut("etc/io.etcd.jetcd.cfg", "endpoints", "http://localhost:2379"),
         editConfigurationFilePut("etc/io.etcd.jetcd.resolver.dnssrv.cfg", "foo", "bar"),
         keepRuntimeFolder(),
         cleanCaches(),
@@ -118,10 +122,25 @@ public Option[] config() {
   }
 
   @Test
-  public void testServiceAvailability() {
+  public void testServiceAvailability() throws Exception {
     assertThat(bundleContext).isNotNull();
     assertThat(client).isNotNull();
     assertThat(uriResolver).isNotNull();
+
+    try {
+        client.getKVClient().get(ByteSequence.from("non-existing", Charset.forName("UTF-8"))).get(13, TimeUnit.SECONDS);
+    } catch (Throwable t) {
+        // Pax Exam's WrappedTestContainerException unfortunately only includes the message, not the cause,
+        // so the real reason for failures needs to be searched for in target/exam/*/data/log/karaf.log ...
+        // Just for convenience in local debugging, and to eastily understand failures on CI, we dump
+        // the failure's stack trace to STDOUT; like that it's easy to see in maven-surefire-plugin,
+        // or when running in the IDE, like for any other non-OSGi test failure.
+        t.printStackTrace();
+        throw t;
+    }
+
+    // NB: Any NoClassDefFoundError/ClassNotFoundException in the log "because the bundle wiring for io.etcd.jetcd-all is no longer valid"
+    // can be safely ignored - that's just jetcd/gRPC/Netty not being cleany shut down and still running.. doesn't really matter much.
   }
 
 }