Limit KeePassXC installation checks to Linux

Author: purejava

src/main/java/org/keepassxc/KindOfKeePassXC.java

@@ -1,14 +1,22 @@
 package org.keepassxc;
 
+import org.apache.commons.lang3.SystemUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import java.util.Optional;
 
+/**
+ * Utility class to determine the kind of the KeePassXC installation on Linux
+ */
 public class KindOfKeePassXC {
     private static final Logger LOG = LoggerFactory.getLogger(KindOfKeePassXC.class);
 
     public static Optional<KeePassXCType> determineType() {
+        if (!SystemUtils.IS_OS_LINUX) {
+            return Optional.empty();
+        }
+
         ///home/<user>/Downloads/KeePassXC-2.7.4-x86_64.AppImage
         var processHandle = ProcessHandle.allProcesses()
                 .filter(ph -> ph.info().command().isPresent() && ph.info().command().get().contains("KeePassXC")

src/test/java/org/purejava/AppImageTest.java

@@ -3,6 +3,8 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.EnabledOnOs;
+import org.junit.jupiter.api.condition.OS;
 import org.keepassxc.KeePassXCType;
 import org.keepassxc.KindOfKeePassXC;
 
@@ -15,6 +17,7 @@ public class AppImageTest {
     @Test
     @Order(7)
     @DisplayName("Testing KeePassXC installation type")
+    @EnabledOnOs(OS.LINUX)
     public void shouldHaveNoErrors() {
         var type = KindOfKeePassXC.determineType();
         assertTrue(type.isPresent() && type.get().equals(KeePassXCType.AppImage));

src/test/java/org/purejava/FlatpakTest.java

@@ -3,6 +3,8 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.EnabledOnOs;
+import org.junit.jupiter.api.condition.OS;
 import org.keepassxc.KeePassXCType;
 import org.keepassxc.KindOfKeePassXC;
 
@@ -15,6 +17,7 @@ public class FlatpakTest {
     @Test
     @Order(4)
     @DisplayName("Testing KeePassXC installation type")
+    @EnabledOnOs(OS.LINUX)
     public void shouldHaveNoErrors() {
         var type = KindOfKeePassXC.determineType();
         assertTrue(type.isPresent() && type.get().equals(KeePassXCType.Flatpak));

src/test/java/org/purejava/RepoTest.java

@@ -3,6 +3,8 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.EnabledOnOs;
+import org.junit.jupiter.api.condition.OS;
 import org.keepassxc.KeePassXCType;
 import org.keepassxc.KindOfKeePassXC;
 
@@ -15,6 +17,7 @@ public class RepoTest {
     @Test
     @Order(6)
     @DisplayName("Testing KeePassXC installation type")
+    @EnabledOnOs(OS.LINUX)
     public void shouldHaveNoErrors() {
         var type = KindOfKeePassXC.determineType();
         assertTrue(type.isPresent() && type.get().equals(KeePassXCType.Repo));

src/test/java/org/purejava/SnapTest.java

@@ -3,6 +3,8 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.EnabledOnOs;
+import org.junit.jupiter.api.condition.OS;
 import org.keepassxc.KeePassXCType;
 import org.keepassxc.KindOfKeePassXC;
 
@@ -15,6 +17,7 @@ public class SnapTest {
     @Test
     @Order(5)
     @DisplayName("Testing KeePassXC installation type")
+    @EnabledOnOs(OS.LINUX)
     public void shouldHaveNoErrors() {
         var type = KindOfKeePassXC.determineType();
         assertTrue(type.isPresent() && type.get().equals(KeePassXCType.Snap));