fix: Improve handling of runtime errors

This includes for example TLS-related errors, as
reported in #194 - hence the test case.

src/main/java/it/mulders/mcs/common/Result.java

@@ -20,6 +20,11 @@ public void ifPresent(final Consumer<T> consumer) {
             consumer.accept(this.value);
         }
 
+        @Override
+        public void ifPresentOrElse(Consumer<T> successConsumer, Consumer<Throwable> failureConsumer) {
+            successConsumer.accept(value);
+        }
+
         @Override
         public Throwable cause() {
             throw new NoSuchElementException("success: " + this.value);
@@ -36,6 +41,11 @@ public <U> Result<U> map(final Function<T, U> mapping) {
         public void ifPresent(final Consumer<T> consumer) {
         }
 
+        @Override
+        public void ifPresentOrElse(Consumer<T> successConsumer, Consumer<Throwable> failureConsumer) {
+            failureConsumer.accept(cause);
+        }
+
         @Override
         public T value() {
             throw new NoSuchElementException("failure: " + this.cause.getLocalizedMessage());
@@ -46,6 +56,8 @@ public T value() {
 
     void ifPresent(final Consumer<T> consumer);
 
+    void ifPresentOrElse(final Consumer<T> successConsumer, final Consumer<Throwable> failureConsumer);
+
     T value();
 
     Throwable cause();

src/main/java/it/mulders/mcs/search/SearchCommandHandler.java

@@ -27,7 +27,10 @@ public SearchCommandHandler(final OutputPrinter coordinateOutput) {
     public void search(final SearchQuery query) {
         performSearch(query)
                 .map(response -> performAdditionalSearch(query, response))
-                .ifPresent(response -> printResponse(query, response));
+                .ifPresentOrElse(
+                        response -> printResponse(query, response),
+                        failure -> { throw new RuntimeException(failure); }
+                );
     }
 
     private SearchResponse.Response performAdditionalSearch(final SearchQuery query,

src/test/java/it/mulders/mcs/search/SearchCommandHandlerTest.java

@@ -9,6 +9,8 @@
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
 
+import javax.net.ssl.SSLHandshakeException;
+
 import static org.mockito.Mockito.any;
 import static org.mockito.Mockito.eq;
 import static org.mockito.Mockito.mock;
@@ -35,6 +37,9 @@ class SearchCommandHandlerTest implements WithAssertions {
     private final SearchClient searchClient = new SearchClient() {
         @Override
         public Result<SearchResponse> search(final SearchQuery query) {
+            if (query.toSolrQuery().contains("tls-error")) {
+                return new Result.Failure<>(new SSLHandshakeException("PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target"));
+            }
             if (query instanceof WildcardSearchQuery) {
                 return new Result.Success<>(new SearchResponse(null, wildcardResponse));
             } else if (query instanceof CoordinateQuery cq && cq.version().isBlank()) {
@@ -55,6 +60,12 @@ void should_invoke_search_client() {
             handler.search(SearchQuery.search("plexus-utils").build());
             verify(outputPrinter).print(any(WildcardSearchQuery.class), eq(wildcardResponse), any());
         }
+
+        @Test
+        void should_propagate_tls_exception_to_runtime_exception() {
+            assertThatThrownBy(() -> handler.search(SearchQuery.search("tls-error").build()))
+                    .isInstanceOf(RuntimeException.class);
+        }
     }
 
     @Nested
@@ -77,5 +88,11 @@ void should_invoke_search_client_with_groupId_and_artifactId_and_version() {
             handler.search(SearchQuery.search("org.codehaus.plexus:plexus-utils:3.4.1").build());
             verify(outputPrinter).print(any(CoordinateQuery.class), eq(threePartCoordinateResponse), any());
         }
+
+        @Test
+        void should_propagate_tls_exception_to_runtime_exception() {
+            assertThatThrownBy(() -> handler.search(SearchQuery.search("org.codehaus.plexus:tls-error:3.4.1").build()))
+                    .isInstanceOf(RuntimeException.class);
+        }
     }
 }