Improve names of startup-related tests
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Mon, 27 Sep 2021 14:08:29 +0000 (16:08 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Mon, 27 Sep 2021 14:10:58 +0000 (16:10 +0200)
Because honestly, what does "new config" even mean? Nobody knows unless
they also know the implementation (which ties a boolean value to an
annotation named "NewConfig") so stating clearly in which scenarios the
tests are running is preferrable. Like, by A LOT.

src/test/kotlin/net/pterodactylus/sone/main/SonePluginTest.kt

index d1e2cec..a656702 100644 (file)
@@ -147,7 +147,7 @@ class SonePluginTest {
        }
 
        @Test
-       fun `config-not-read event is sent to event bus when new config is true`() {
+       fun `config-not-read event is sent to event bus when config file is invalid`() {
                File("sone.properties").deleteAfter {
                        writeText("Invalid")
                        val configNotReadReceived = AtomicBoolean()
@@ -160,7 +160,7 @@ class SonePluginTest {
        }
 
        @Test
-       fun `config-not-read event is not sent to event bus when first start is true`() {
+       fun `config-not-read event is not sent to event bus when config file does not exist`() {
                File("sone.properties").delete()
                val configNotReadReceived = AtomicBoolean()
                runSonePluginWithRealInjector {
@@ -171,7 +171,7 @@ class SonePluginTest {
        }
 
        @Test
-       fun `config-not-read event is not sent to event bus when new config is false`() {
+       fun `config-not-read event is not sent to event bus when config file is valid`() {
                File("sone.properties").deleteAfter {
                        writeText("# comment")
                        val configNotReadReceived = AtomicBoolean()