fix: allow using configure commands in DMs (for admins)

DarkAtra · Aug 14, 2024 · adda4ec · adda4ec
1 parent 160a6f7
commit adda4ec
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 3 deletions.
diff --git a/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigurePlayerActivityFeedCommand.kt b/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigurePlayerActivityFeedCommand.kt
@@ -37,7 +37,7 @@ class ConfigurePlayerActivityFeedCommand(
             description = description
         ) {
 
-            dmPermission = false
+            dmPermission = true
             disableCommandInGuilds()
 
             addServerIdParameter()

diff --git a/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigurePvpKillFeedCommand.kt b/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigurePvpKillFeedCommand.kt
@@ -37,7 +37,7 @@ class ConfigurePvpKillFeedCommand(
             description = description
         ) {
 
-            dmPermission = false
+            dmPermission = true
             disableCommandInGuilds()
 
             addServerIdParameter()

diff --git a/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigureStatusMonitorCommand.kt b/src/main/kotlin/de/darkatra/vrising/discord/commands/ConfigureStatusMonitorCommand.kt
@@ -41,7 +41,7 @@ class ConfigureStatusMonitorCommand(
             description = description
         ) {
 
-            dmPermission = false
+            dmPermission = true
             disableCommandInGuilds()
 
             addServerIdParameter()

diff --git a/src/test/kotlin/de/darkatra/vrising/discord/clients/botcompanion/BotCompanionClientTest.kt b/src/test/kotlin/de/darkatra/vrising/discord/clients/botcompanion/BotCompanionClientTest.kt
@@ -25,6 +25,29 @@ class BotCompanionClientTest {
         }
     )
 
+    @Test
+    fun `should handle timeouts correctly`(wireMockRuntimeInfo: WireMockRuntimeInfo) {
+
+        wireMockRuntimeInfo.wireMock.register(
+            WireMock.get("/v-rising-discord-bot/characters")
+                .withHeader(HttpHeaders.ACCEPT, equalTo(MediaType.APPLICATION_JSON_VALUE))
+                .withHeader(HttpHeaders.USER_AGENT, equalTo("test"))
+                .willReturn(
+                    WireMock.aResponse()
+                        .withFixedDelay(10_000)
+                        .withStatus(HttpStatus.OK.value())
+                )
+        )
+
+        val charactersResult = runBlocking {
+            botCompanionClient.getCharacters("localhost", wireMockRuntimeInfo.httpPort)
+        }
+        assertThat(charactersResult.isFailure).isTrue()
+
+        val exception = charactersResult.exceptionOrNull()
+        assertThat(exception).hasMessageContaining("Unexpected exception performing")
+    }
+
     @Test
     fun `should get characters`(wireMockRuntimeInfo: WireMockRuntimeInfo) {
 

diff --git a/src/test/kotlin/de/darkatra/vrising/discord/persistence/ServerRepositoryTest.kt b/src/test/kotlin/de/darkatra/vrising/discord/persistence/ServerRepositoryTest.kt
@@ -96,6 +96,8 @@ class ServerRepositoryTest {
 
         serverRepository.updateServer(update1)
 
+        assertThat(serverRepository.getServer(server.id, server.discordServerId)!!.hostname).isEqualTo("test-1")
+
         val e = assertThrows<OutdatedServerException> {
             serverRepository.updateServer(update2)
         }