From f2204dfd4d091d868e9435e672187ced4c58b56e Mon Sep 17 00:00:00 2001
From: Paul Schaub <vanitasvitae@fsfe.org>
Date: Wed, 15 Nov 2023 18:06:35 +0100
Subject: [PATCH] Kotlin conversion: ListProfilesExternal

---
 .../operation/ListProfilesExternal.java       | 50 -------------------
 .../operation/ListProfilesExternal.kt         | 36 +++++++++++++
 2 files changed, 36 insertions(+), 50 deletions(-)
 delete mode 100644 external-sop/src/main/java/sop/external/operation/ListProfilesExternal.java
 create mode 100644 external-sop/src/main/kotlin/sop/external/operation/ListProfilesExternal.kt

diff --git a/external-sop/src/main/java/sop/external/operation/ListProfilesExternal.java b/external-sop/src/main/java/sop/external/operation/ListProfilesExternal.java
deleted file mode 100644
index 21d5e13..0000000
--- a/external-sop/src/main/java/sop/external/operation/ListProfilesExternal.java
+++ /dev/null
@@ -1,50 +0,0 @@
-// SPDX-FileCopyrightText: 2023 Paul Schaub <vanitasvitae@fsfe.org>
-//
-// SPDX-License-Identifier: Apache-2.0
-
-package sop.external.operation;
-
-import sop.Profile;
-import sop.external.ExternalSOP;
-import sop.operation.ListProfiles;
-
-import javax.annotation.Nonnull;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Properties;
-
-public class ListProfilesExternal implements ListProfiles {
-
-    private final List<String> commandList = new ArrayList<>();
-    private final List<String> envList;
-
-    public ListProfilesExternal(String binary, Properties properties) {
-        this.commandList.add(binary);
-        this.commandList.add("list-profiles");
-        this.envList = ExternalSOP.propertiesToEnv(properties);
-    }
-
-    @Override
-    @Nonnull
-    public List<Profile> subcommand(@Nonnull String command) {
-        commandList.add(command);
-        try {
-            String output = new String(ExternalSOP.executeProducingOperation(Runtime.getRuntime(), commandList, envList).getBytes());
-            return toProfiles(output);
-        } catch (IOException e) {
-            throw new RuntimeException(e);
-        }
-    }
-
-    private static List<Profile> toProfiles(String output) {
-        List<Profile> profiles = new ArrayList<>();
-        for (String line : output.split("\n")) {
-            if (line.trim().isEmpty()) {
-                continue;
-            }
-            profiles.add(Profile.parse(line));
-        }
-        return profiles;
-    }
-}
diff --git a/external-sop/src/main/kotlin/sop/external/operation/ListProfilesExternal.kt b/external-sop/src/main/kotlin/sop/external/operation/ListProfilesExternal.kt
new file mode 100644
index 0000000..5e8ff89
--- /dev/null
+++ b/external-sop/src/main/kotlin/sop/external/operation/ListProfilesExternal.kt
@@ -0,0 +1,36 @@
+// SPDX-FileCopyrightText: 2023 Paul Schaub <vanitasvitae@fsfe.org>
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package sop.external.operation
+
+import java.io.IOException
+import java.util.Properties
+import sop.Profile
+import sop.external.ExternalSOP
+import sop.operation.ListProfiles
+
+/** Implementation of the [ListProfiles] operation using an external SOP binary. */
+class ListProfilesExternal(binary: String, environment: Properties) : ListProfiles {
+
+    private val commandList = mutableListOf(binary, "list-profiles")
+    private val envList = ExternalSOP.propertiesToEnv(environment)
+
+    override fun subcommand(command: String): List<Profile> {
+        return try {
+            String(
+                    ExternalSOP.executeProducingOperation(
+                            Runtime.getRuntime(), commandList.plus(command), envList)
+                        .bytes)
+                .let { toProfiles(it) }
+        } catch (e: IOException) {
+            throw RuntimeException(e)
+        }
+    }
+
+    companion object {
+        @JvmStatic
+        private fun toProfiles(output: String): List<Profile> =
+            output.split("\n").filter { it.isNotBlank() }.map { Profile.parse(it) }
+    }
+}