Clean up some trailing parentheses

Author: code-asher

src/test/kotlin/com/coder/gateway/cli/CoderCLIManagerTest.kt

@@ -120,8 +120,7 @@ internal class CoderCLIManagerTest {
 
         val (srv, url) = mockServer()
         val ccm = CoderCLIManager(url, CoderSettings(CoderSettingsState(
-            dataDirectory = tmpdir.resolve("cli-dir-fail-to-write").toString()))
-        )
+            dataDirectory = tmpdir.resolve("cli-dir-fail-to-write").toString())))
 
         ccm.localBinaryPath.parent.toFile().mkdirs()
         ccm.localBinaryPath.parent.toFile().setWritable(false)
@@ -146,8 +145,7 @@ internal class CoderCLIManagerTest {
         }
 
         val ccm = CoderCLIManager(url.toURL(), CoderSettings(CoderSettingsState(
-            dataDirectory = tmpdir.resolve("real-cli").toString()))
-        )
+            dataDirectory = tmpdir.resolve("real-cli").toString())))
 
         assertTrue(ccm.download())
         assertDoesNotThrow { ccm.version() }
@@ -165,11 +163,10 @@ internal class CoderCLIManagerTest {
     fun testDownloadMockCLI() {
         val (srv, url) = mockServer()
         var ccm = CoderCLIManager(url, CoderSettings(CoderSettingsState(
-            dataDirectory = tmpdir.resolve("mock-cli").toString()))
-        )
+            dataDirectory = tmpdir.resolve("mock-cli").toString()),
+            binaryName = "coder.bat"))
 
         assertEquals(true, ccm.download())
-
         assertEquals(SemVer(url.port.toLong(), 0, 0), ccm.version())
 
         // It should skip the second attempt.
@@ -189,8 +186,7 @@ internal class CoderCLIManagerTest {
     @Test
     fun testRunNonExistentBinary() {
         val ccm = CoderCLIManager(URL("https://foo"), CoderSettings(CoderSettingsState(
-            dataDirectory = tmpdir.resolve("does-not-exist").toString()))
-        )
+            dataDirectory = tmpdir.resolve("does-not-exist").toString())))
 
         assertFailsWith(
             exceptionClass = ProcessInitException::class,
@@ -201,8 +197,7 @@ internal class CoderCLIManagerTest {
     fun testOverwitesWrongVersion() {
         val (srv, url) = mockServer()
         val ccm = CoderCLIManager(url, CoderSettings(CoderSettingsState(
-            dataDirectory = tmpdir.resolve("overwrite-cli").toString()))
-        )
+            dataDirectory = tmpdir.resolve("overwrite-cli").toString())))
 
         ccm.localBinaryPath.parent.toFile().mkdirs()
         ccm.localBinaryPath.toFile().writeText("cli")
@@ -329,9 +324,7 @@ internal class CoderCLIManagerTest {
                 sshConfigPath = tmpdir.resolve("configured$it.conf"))
             settings.sshConfigPath.parent.toFile().mkdirs()
             Path.of("src/test/fixtures/inputs").resolve("$it.conf").toFile().copyTo(
-                settings.sshConfigPath.toFile(),
-                true,
-            )
+                settings.sshConfigPath.toFile(), true)
 
             val ccm = CoderCLIManager(URL("https://test.coder.invalid"), settings)
 
@@ -349,8 +342,7 @@ internal class CoderCLIManagerTest {
 
         tests.forEach {
             val ccm = CoderCLIManager(URL("https://test.coder.invalid"), CoderSettings(CoderSettingsState(
-                headerCommand = it))
-            )
+                headerCommand = it)))
 
             assertFailsWith(
                 exceptionClass = Exception::class,
@@ -587,8 +579,7 @@ internal class CoderCLIManagerTest {
         tests.forEach {
             val (srv, url) = mockServer(version = it.first)
             val ccm = CoderCLIManager(url, CoderSettings(CoderSettingsState(
-                dataDirectory = tmpdir.resolve("features").toString()))
-            )
+                dataDirectory = tmpdir.resolve("features").toString())))
             assertEquals(true, ccm.download())
             assertEquals(it.second, ccm.features, "version: ${it.first}")
 

src/test/kotlin/com/coder/gateway/settings/CoderSettingsTest.kt

@@ -36,8 +36,7 @@ internal class CoderSettingsTest {
             env = Environment(mapOf(
                 "LOCALAPPDATA"     to "/tmp/coder-gateway-test/localappdata",
                 "HOME"             to "/tmp/coder-gateway-test/home",
-                "XDG_DATA_HOME"    to "/tmp/coder-gateway-test/xdg-data"))
-        )
+                "XDG_DATA_HOME"    to "/tmp/coder-gateway-test/xdg-data")))
         var expected = when(getOS()) {
             OS.WINDOWS -> "/tmp/coder-gateway-test/localappdata/coder-gateway/localhost"
             OS.MAC -> "/tmp/coder-gateway-test/home/Library/Application Support/coder-gateway/localhost"
@@ -52,8 +51,7 @@ internal class CoderSettingsTest {
             settings = CoderSettings(state,
                 env = Environment(mapOf(
                     "XDG_DATA_HOME" to "",
-                    "HOME" to "/tmp/coder-gateway-test/home"))
-            )
+                    "HOME" to "/tmp/coder-gateway-test/home")))
             expected = "/tmp/coder-gateway-test/home/.local/share/coder-gateway/localhost"
 
             assertEquals(Path.of(expected).toAbsolutePath(), settings.dataDir(url))
@@ -66,8 +64,7 @@ internal class CoderSettingsTest {
             env = Environment(mapOf(
                 "LOCALAPPDATA"     to "/ignore",
                 "HOME"             to "/ignore",
-                "XDG_DATA_HOME"    to "/ignore"))
-        )
+                "XDG_DATA_HOME"    to "/ignore")))
         expected = "/tmp/coder-gateway-test/data-dir/localhost"
         assertEquals(Path.of(expected).toAbsolutePath(), settings.dataDir(url))
         assertEquals(Path.of(expected).toAbsolutePath(), settings.binPath(url).parent)
@@ -110,11 +107,10 @@ internal class CoderSettingsTest {
     fun testCoderConfigDir() {
         val state = CoderSettingsState()
         var settings = CoderSettings(state,
-            env = Environment(
-                mapOf("APPDATA"          to "/tmp/coder-gateway-test/cli-appdata",
-                    "HOME"             to "/tmp/coder-gateway-test/cli-home",
-                    "XDG_CONFIG_HOME"  to "/tmp/coder-gateway-test/cli-xdg-config"))
-        )
+            env = Environment(mapOf(
+                "APPDATA"        to "/tmp/coder-gateway-test/cli-appdata",
+                "HOME"             to "/tmp/coder-gateway-test/cli-home",
+                "XDG_CONFIG_HOME"  to "/tmp/coder-gateway-test/cli-xdg-config")))
         var expected = when(getOS()) {
             OS.WINDOWS -> "/tmp/coder-gateway-test/cli-appdata/coderv2"
             OS.MAC -> "/tmp/coder-gateway-test/cli-home/Library/Application Support/coderv2"
@@ -125,25 +121,20 @@ internal class CoderSettingsTest {
         // Fall back to HOME on Linux.
         if (getOS() == OS.LINUX) {
             settings = CoderSettings(state,
-                env = Environment(
-                    mapOf("XDG_CONFIG_HOME" to "",
-                        "HOME" to "/tmp/coder-gateway-test/cli-home",
-                    ))
-            )
+                env = Environment(mapOf(
+                    "XDG_CONFIG_HOME" to "",
+                    "HOME" to "/tmp/coder-gateway-test/cli-home")))
             expected = "/tmp/coder-gateway-test/cli-home/.config/coderv2"
             assertEquals(Path.of(expected), settings.coderConfigDir)
         }
 
         // Read CODER_CONFIG_DIR.
         settings = CoderSettings(state,
-            env = Environment(
-                mapOf(
-                    "CODER_CONFIG_DIR" to "/tmp/coder-gateway-test/coder-config-dir",
-                    "APPDATA"          to "/ignore",
-                    "HOME"             to "/ignore",
-                    "XDG_CONFIG_HOME"  to "/ignore",
-                ))
-        )
+            env = Environment(mapOf(
+                "CODER_CONFIG_DIR" to "/tmp/coder-gateway-test/coder-config-dir",
+                "APPDATA"          to "/ignore",
+                "HOME"             to "/ignore",
+                "XDG_CONFIG_HOME"  to "/ignore")))
         expected = "/tmp/coder-gateway-test/coder-config-dir"
         assertEquals(Path.of(expected), settings.coderConfigDir)
     }