🎨 Fix formatting
[Sone.git] / src / test / kotlin / net / pterodactylus / sone / web / pages / DismissNotificationPageTest.kt
index b6524a8..e8583df 100644 (file)
@@ -1,27 +1,21 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.DismissNotificationPage
-import net.pterodactylus.util.notify.Notification
-import net.pterodactylus.util.web.Method.GET
-import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers.equalTo
-import org.junit.Test
-import org.mockito.Mockito.never
-import org.mockito.Mockito.verify
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.util.notify.*
+import net.pterodactylus.util.web.Method.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
+import org.mockito.Mockito.*
 
 /**
  * Unit test for [DismissNotificationPage].
  */
-class DismissNotificationPageTest: WebPageTest() {
+class DismissNotificationPageTest : WebPageTest(::DismissNotificationPage) {
 
-       private val page = DismissNotificationPage(template, webInterface)
        private val notification = mock<Notification>()
 
-       override fun getPage() = page
-
        @Test
        fun `page returns correct path`() {
                assertThat(page.path, equalTo("dismissNotification.html"))
@@ -35,37 +29,42 @@ class DismissNotificationPageTest: WebPageTest() {
        @Test
        fun `page returns correct title`() {
                whenever(l10n.getString("Page.DismissNotification.Title")).thenReturn("dismiss notification page")
-               assertThat(page.getPageTitle(freenetRequest), equalTo("dismiss notification page"))
+               assertThat(page.getPageTitle(soneRequest), equalTo("dismiss notification page"))
        }
 
        @Test
        fun `get request with invalid notification ID redirects to return page`() {
-               request("", GET)
-               addHttpRequestParameter("returnPage", "return.html")
+               setMethod(POST)
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html")
        }
 
        @Test
        fun `get request with non-dismissible notification never dismisses the notification but redirects to return page`() {
-               request("", GET)
+               setMethod(POST)
                addNotification("notification-id", notification)
-               addHttpRequestParameter("notification", "notification-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("notification", "notification-id")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html") {
                        verify(notification, never()).dismiss()
                }
        }
 
        @Test
-       fun `get request with dismissible notification dismisses the notification and redirects to return page`() {
-               request("", GET)
+       fun `post request with dismissible notification dismisses the notification and redirects to return page`() {
+               setMethod(POST)
                whenever(notification.isDismissable).thenReturn(true)
                addNotification("notification-id", notification)
-               addHttpRequestParameter("notification", "notification-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("notification", "notification-id")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html") {
                        verify(notification).dismiss()
                }
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+               assertThat(baseInjector.getInstance<DismissNotificationPage>(), notNullValue())
+       }
+
 }