diff options
author | Travis Collins <travistx@gmail.com> | 2025-05-21 23:21:10 -0600 |
---|---|---|
committer | sonartech <sonartech@sonarsource.com> | 2025-05-22 20:03:29 +0000 |
commit | 1daea343a92b371d720e6d1503c09a282d2553f0 (patch) | |
tree | 937faaccdc57ce05f28cce1654d5bdba74fcacb6 | |
parent | 8834c23310e34ed2f66bf2b464779f2a1ec96038 (diff) | |
download | sonarqube-1daea343a92b371d720e6d1503c09a282d2553f0.tar.gz sonarqube-1daea343a92b371d720e6d1503c09a282d2553f0.zip |
SCA-432 SCA CLI client error handling
-rw-r--r-- | sonar-scanner-engine/src/main/java/org/sonar/scanner/sca/CliCacheService.java | 5 | ||||
-rw-r--r-- | sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/sonar-scanner-engine/src/main/java/org/sonar/scanner/sca/CliCacheService.java b/sonar-scanner-engine/src/main/java/org/sonar/scanner/sca/CliCacheService.java index 305bc224349..24db0ddec64 100644 --- a/sonar-scanner-engine/src/main/java/org/sonar/scanner/sca/CliCacheService.java +++ b/sonar-scanner-engine/src/main/java/org/sonar/scanner/sca/CliCacheService.java @@ -25,6 +25,7 @@ import java.io.File; import java.io.IOException; import java.io.InputStream; import java.io.Reader; +import java.io.UncheckedIOException; import java.lang.reflect.Type; import java.nio.file.Files; import java.nio.file.Path; @@ -175,8 +176,8 @@ public class CliCacheService { }.getType(); return new Gson().fromJson(reader, listOfMetadata); } - } catch (Exception e) { - throw new IllegalStateException("Unable to load CLI metadata", e); + } catch (IOException e) { + throw new UncheckedIOException(e); } } diff --git a/sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java b/sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java index 9193157e213..17214a13cfc 100644 --- a/sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java +++ b/sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java @@ -161,7 +161,7 @@ class CliCacheServiceTest { WsTestUtil.mockException(scannerWsClient, e); assertThatThrownBy(underTest::cacheCli).isInstanceOf(IllegalStateException.class) - .hasMessageContaining("Unable to load CLI metadata"); + .hasMessageContaining("http error"); verify(telemetryCache).put("scanner.sca.get.cli.success", "false"); } |