SONAR-9845 Display type of disk in System Info on Linu

2 files changed, 20 insertions, 5 deletions

diff --git a/server/sonar-process/src/main/java/org/sonar/process/systeminfo/SystemInfoUtils.java b/server/sonar-process/src/main/java/org/sonar/process/systeminfo/SystemInfoUtils.java
index 0126bbe2ae7..58cb458bdeb 100644
--- a/server/sonar-process/src/main/java/org/sonar/process/systeminfo/SystemInfoUtils.java
+++ b/server/sonar-process/src/main/java/org/sonar/process/systeminfo/SystemInfoUtils.java
@@ -55,11 +55,13 @@ public class SystemInfoUtils {
     }
   }
 
-  public static void setAttribute(Section.Builder section, String key, boolean value) {
-    section.addAttributesBuilder()
-      .setKey(key)
-      .setBooleanValue(value)
-      .build();
+  public static void setAttribute(Section.Builder section, String key, @Nullable Boolean value) {
+    if (value != null) {
+      section.addAttributesBuilder()
+        .setKey(key)
+        .setBooleanValue(value)
+        .build();
+    }
   }
 
   public static void setAttribute(Section.Builder section, String key, long value) {
diff --git a/server/sonar-process/src/test/java/org/sonar/process/systeminfo/SystemInfoUtilsTest.java b/server/sonar-process/src/test/java/org/sonar/process/systeminfo/SystemInfoUtilsTest.java
index 03410f34ea9..f7e3c09ca72 100644
--- a/server/sonar-process/src/test/java/org/sonar/process/systeminfo/SystemInfoUtilsTest.java
+++ b/server/sonar-process/src/test/java/org/sonar/process/systeminfo/SystemInfoUtilsTest.java
@@ -31,6 +31,19 @@ import static org.assertj.core.api.Assertions.assertThat;
 public class SystemInfoUtilsTest {
 
   @Test
+  public void test_setAttribute_with_boolean_parameter() {
+    Section.Builder builder = Section.newBuilder();
+    SystemInfoUtils.setAttribute(builder, "isNull", (Boolean)null);
+    SystemInfoUtils.setAttribute(builder, "isTrue", true);
+    SystemInfoUtils.setAttribute(builder, "isFalse", false);
+
+    Section section = builder.build();
+    assertThat(SystemInfoUtils.attribute(section, "isNull")).isNull();
+    assertThat(SystemInfoUtils.attribute(section, "isTrue").getBooleanValue()).isTrue();
+    assertThat(SystemInfoUtils.attribute(section, "isFalse").getBooleanValue()).isFalse();
+  }
+
+  @Test
   public void test_order() {
     Collection<Section> sections = asList(
       newSection("end2"),