Use golden files for SSH config tests

Also add trailing newline when appending to file.  Just a bit nicer, for
example when you cat the file.

If we are replacing a block at the end and there is no newline then we
will preserve that; no newlines are added when replacing.

Author: code-asher

src/main/kotlin/com/coder/gateway/sdk/CoderCLIManager.kt

@@ -196,12 +196,11 @@ class CoderCLIManager @JvmOverloads constructor(private val deploymentURL: URL,
                 logger.info("Leaving $sshConfigPath alone since there are no workspaces and no config to remove")
             } else if (start == null && end == null) {
                 logger.info("Appending config to $sshConfigPath")
-                sshConfigPath.toFile().writeText(
-                    if (contents.isEmpty()) blockContent else listOf(
-                        contents,
-                        blockContent
-                    ).joinToString(System.lineSeparator())
-                )
+                val toAppend = if (contents.isEmpty()) blockContent else listOf(
+                    contents,
+                    blockContent
+                ).joinToString(System.lineSeparator())
+                sshConfigPath.toFile().writeText(toAppend + System.lineSeparator())
             } else if (start == null) {
                 throw SSHConfigFormatException("End block exists but no start block")
             } else if (end == null) {
@@ -234,7 +233,7 @@ class CoderCLIManager @JvmOverloads constructor(private val deploymentURL: URL,
             }
         } catch (e: FileNotFoundException) {
             logger.info("Writing config to $sshConfigPath")
-            sshConfigPath.toFile().writeText(blockContent)
+            sshConfigPath.toFile().writeText(blockContent + System.lineSeparator())
         }
     }
 

src/test/fixtures/inputs/blank-newlines.conf

@@ -0,0 +1,3 @@
+
+
+

src/test/fixtures/inputs/blank.conf

@@ -0,0 +1,5 @@
+Host test
+  Port 80
+Host test2
+  Port 443 # --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config # --- END CODER JETBRAINS test.coder.invalid

src/test/fixtures/inputs/existing-end-no-newline.conf

@@ -0,0 +1,7 @@
+Host test
+  Port 80
+Host test2
+  Port 443
+# --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid

src/test/fixtures/inputs/existing-end.conf

@@ -0,0 +1,13 @@
+Host test
+  Port 80
+# ------------START-CODER-----------
+some coder config
+# ------------END-CODER------------
+# --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid
+Host test2
+  Port 443
+# --- START CODER JETBRAINS test.coder.unrelated
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.unrelated

src/test/fixtures/inputs/existing-middle-and-unrelated.conf

@@ -0,0 +1,7 @@
+Host test
+  Port 80
+# --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid
+Host test2
+  Port 443

src/test/fixtures/inputs/existing-middle.conf

@@ -0,0 +1,3 @@
+# --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid

src/test/fixtures/inputs/existing-only.conf

@@ -0,0 +1,7 @@
+# --- START CODER JETBRAINS test.coder.invalid
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid
+Host test
+  Port 80
+Host test2
+  Port 443

src/test/fixtures/inputs/existing-start.conf

@@ -0,0 +1,3 @@
+# --- START CODER JETBRAINS test.coder.something-else
+some jetbrains config
+# --- END CODER JETBRAINS test.coder.invalid