aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTravis Collins <travistx@gmail.com>2025-05-21 23:21:10 -0600
committersonartech <sonartech@sonarsource.com>2025-05-22 20:03:29 +0000
commit1daea343a92b371d720e6d1503c09a282d2553f0 (patch)
tree937faaccdc57ce05f28cce1654d5bdba74fcacb6
parent8834c23310e34ed2f66bf2b464779f2a1ec96038 (diff)
downloadsonarqube-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.java5
-rw-r--r--sonar-scanner-engine/src/test/java/org/sonar/scanner/sca/CliCacheServiceTest.java2
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");
}