🔥 Remove unnecessary not-null checks

[Sone.git] / src / test / kotlin / net / pterodactylus / sone / web / notification / NotificationHandlerModuleTest.kt

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/notification/NotificationHandlerModuleTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/notification/NotificationHandlerModuleTest.kt
index 76926fb..11e728d 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/notification/NotificationHandlerModuleTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/notification/NotificationHandlerModuleTest.kt
@@ -23,6 +23,7 @@ import com.google.inject.name.Names.*
 import net.pterodactylus.sone.core.*
 import net.pterodactylus.sone.core.event.*
 import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.data.Post.*
 import net.pterodactylus.sone.data.impl.*
 import net.pterodactylus.sone.main.*
 import net.pterodactylus.sone.notify.*
@@ -51,18 +52,13 @@ class NotificationHandlerModuleTest {
        )
 
        @Test
-       fun `module can create notification handler`() {
-               assertThat(injector.getInstance<NotificationHandler>(), notNullValue())
-       }
-
-       @Test
        fun `notification handler is created as singleton`() {
                injector.verifySingletonInstance<NotificationHandler>()
        }
 
        @Test
-       fun `module can create mark-post-known-during-first-start handler`() {
-               assertThat(injector.getInstance<MarkPostKnownDuringFirstStartHandler>(), notNullValue())
+       fun `mark-post-known-during-first-start handler is created as singleton`() {
+               injector.verifySingletonInstance<MarkPostKnownDuringFirstStartHandler>()
        }
 
        @Test
@@ -77,11 +73,6 @@ class NotificationHandlerModuleTest {
        }
 
        @Test
-       fun `module can create sone-locked-on-startup handler`() {
-               assertThat(injector.getInstance<SoneLockedOnStartupHandler>(), notNullValue())
-       }
-
-       @Test
        fun `sone-locked-on-startup handler is created as singleton`() {
                injector.verifySingletonInstance<SoneLockedOnStartupHandler>()
        }
@@ -114,11 +105,6 @@ class NotificationHandlerModuleTest {
        }
 
        @Test
-       fun `new-sone handler can be created`() {
-               assertThat(injector.getInstance<NewSoneHandler>(), notNullValue())
-       }
-
-       @Test
        fun `new-sone handler is created as singleton`() {
                injector.verifySingletonInstance<NewSoneHandler>()
        }
@@ -142,4 +128,118 @@ class NotificationHandlerModuleTest {
                assertThat(injector.getInstance<ListNotification<Sone>>(named("newSone")).isDismissable, equalTo(false))
        }
 
+       @Test
+       fun `new-remote-post handler is created as singleton`() {
+               injector.verifySingletonInstance<NewRemotePostHandler>()
+       }
+
+       @Test
+       fun `new-remote-post notification is created as singleton`() {
+               injector.verifySingletonInstance<ListNotification<Post>>(named("newRemotePost"))
+       }
+
+       @Test
+       fun `new-remote-post notification has correct ID`() {
+               assertThat(injector.getInstance<ListNotification<Post>>(named("newRemotePost")).id, equalTo("new-post-notification"))
+       }
+
+       @Test
+       fun `new-remote-post notification is not dismissable`() {
+               assertThat(injector.getInstance<ListNotification<Post>>(named("newRemotePost")).isDismissable, equalTo(false))
+       }
+
+       @Test
+       fun `new-remote-post notification has correct key and template`() {
+               loaders.templates += "/templates/notify/newPostNotification.html" to "<% posts>".asTemplate()
+               val notification = injector.getInstance<ListNotification<Post>>(named("newRemotePost"))
+               val posts = listOf(EmptyPost("post1"), EmptyPost("post2"))
+               posts.forEach(notification::add)
+               assertThat(notification.render(), equalTo(posts.toString()))
+       }
+
+       @Test
+       fun `sone-locked notification is created as singleton`() {
+               injector.verifySingletonInstance<ListNotification<Sone>>(named("soneLocked"))
+       }
+
+       @Test
+       fun `sone-locked notification is dismissable`() {
+               assertThat(injector.getInstance<ListNotification<Sone>>(named("soneLocked")).isDismissable, equalTo(true))
+       }
+
+       @Test
+       fun `sone-locked notification has correct ID`() {
+               assertThat(injector.getInstance<ListNotification<Sone>>(named("soneLocked")).id, equalTo("sones-locked-notification"))
+       }
+
+       @Test
+       fun `sone-locked notification has correct key and template`() {
+               loaders.templates += "/templates/notify/lockedSonesNotification.html" to "<% sones>".asTemplate()
+               val notification = injector.getInstance<ListNotification<Sone>>(named("soneLocked"))
+               val sones = listOf(IdOnlySone("sone1"), IdOnlySone("sone2"))
+               sones.forEach(notification::add)
+               assertThat(notification.render(), equalTo(sones.toString()))
+       }
+
+       @Test
+       fun `sone-locked handler is created as singleton`() {
+               injector.verifySingletonInstance<SoneLockedHandler>()
+       }
+
+       @Test
+       fun `local-post notification is not dismissable`() {
+               assertThat(injector.getInstance<ListNotification<Post>>(named("localPost")).isDismissable, equalTo(false))
+       }
+
+       @Test
+       fun `local-post notification has correct ID`() {
+               assertThat(injector.getInstance<ListNotification<Post>>(named("localPost")).id, equalTo("local-post-notification"))
+       }
+
+       @Test
+       fun `local-post notification has correct key and template`() {
+               loaders.templates += "/templates/notify/newPostNotification.html" to "<% posts>".asTemplate()
+               val notification = injector.getInstance<ListNotification<Post>>(named("localPost"))
+               val posts = listOf(EmptyPost("post1"), EmptyPost("post2"))
+               posts.forEach(notification::add)
+               assertThat(notification.render(), equalTo(posts.toString()))
+       }
+
+       @Test
+       fun `local-post notification is created as singleton`() {
+               injector.verifySingletonInstance<ListNotification<Post>>(named("localPost"))
+       }
+
+       @Test
+       fun `local-post handler is created as singleton`() {
+               injector.verifySingletonInstance<LocalPostHandler>()
+       }
+
+       @Test
+       fun `new-version notification is created as singleton`() {
+               injector.verifySingletonInstance<TemplateNotification>(named("newVersion"))
+       }
+
+       @Test
+       fun `new-version notification has correct ID`() {
+               assertThat(injector.getInstance<TemplateNotification>(named("newVersion")).id, equalTo("new-version-notification"))
+       }
+
+       @Test
+       fun `new-version notification is dismissable`() {
+               assertThat(injector.getInstance<TemplateNotification>(named("newVersion")).isDismissable, equalTo(true))
+       }
+
+       @Test
+       fun `new-version notification loads correct template`() {
+               loaders.templates += "/templates/notify/newVersionNotification.html" to "1".asTemplate()
+               val notification = injector.getInstance<TemplateNotification>(named("newVersion"))
+               assertThat(notification.render(), equalTo("1"))
+       }
+
+       @Test
+       fun `new-version handler is created as singleton`() {
+               injector.verifySingletonInstance<NewVersionHandler>()
+       }
+
 }