🎨 Clean up imports

[Sone.git] / src / test / kotlin / net / pterodactylus / sone / web / pages / RescuePageTest.kt

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
index cbc8968..dabcbbe 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
@@ -1,15 +1,13 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.core.SoneRescuer
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.RescuePage
-import net.pterodactylus.util.web.Method.GET
-import net.pterodactylus.util.web.Method.POST
-import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers.equalTo
-import org.junit.Before
-import org.junit.Test
+import net.pterodactylus.sone.core.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.sone.web.page.*
+import net.pterodactylus.util.web.Method.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
 import org.mockito.ArgumentMatchers.anyLong
 import org.mockito.Mockito.never
 import org.mockito.Mockito.verify
@@ -17,35 +15,47 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [RescuePage].
  */
-class RescuePageTest : WebPageTest() {
-
-       private val page = RescuePage(template, webInterface)
+class RescuePageTest: WebPageTest(::RescuePage) {
 
        private val soneRescuer = mock<SoneRescuer>()
 
-       override fun getPage() = page
-
        @Before
        fun setupSoneRescuer() {
                whenever(core.getSoneRescuer(currentSone)).thenReturn(soneRescuer)
        }
 
        @Test
+       fun `page returns correct path`() {
+               assertThat(page.path, equalTo("rescue.html"))
+       }
+
+       @Test
+       fun `page requires login`() {
+               assertThat(page.requiresLogin(), equalTo(true))
+       }
+
+       @Test
+       fun `page returns correct title`() {
+               addTranslation("Page.Rescue.Title", "rescue page title")
+               assertThat(page.getPageTitle(soneRequest), equalTo("rescue page title"))
+       }
+
+       @Test
        fun `get request sets rescuer in template context`() {
-               request("", GET)
-               page.handleRequest(freenetRequest, templateContext)
-               assertThat(templateContext["soneRescuer"], equalTo<Any>(soneRescuer))
+               verifyNoRedirect {
+                       assertThat(templateContext["soneRescuer"], equalTo<Any>(soneRescuer))
+               }
        }
 
        @Test
        fun `post request redirects to rescue page`() {
-               request("", POST)
+               setMethod(POST)
                verifyRedirect("rescue.html")
        }
 
        @Test
        fun `post request with fetch and invalid edition starts next fetch`() {
-               request("", POST)
+               setMethod(POST)
                addHttpRequestPart("fetch", "true")
                verifyRedirect("rescue.html") {
                        verify(soneRescuer, never()).setEdition(anyLong())
@@ -55,7 +65,7 @@ class RescuePageTest : WebPageTest() {
 
        @Test
        fun `post request with fetch and valid edition sets edition and starts next fetch`() {
-               request("", POST)
+               setMethod(POST)
                addHttpRequestPart("fetch", "true")
                addHttpRequestPart("edition", "123")
                verifyRedirect("rescue.html") {
@@ -64,4 +74,30 @@ class RescuePageTest : WebPageTest() {
                }
        }
 
+       @Test
+       fun `post request with negative edition will not set edition`() {
+               setMethod(POST)
+               addHttpRequestPart("fetch", "true")
+               addHttpRequestPart("edition", "-123")
+               verifyRedirect("rescue.html") {
+                       verify(soneRescuer, never()).setEdition(anyLong())
+                       verify(soneRescuer).startNextFetch()
+               }
+       }
+
+       @Test
+       fun `page can be created by dependency injection`() {
+           assertThat(baseInjector.getInstance<RescuePage>(), notNullValue())
+       }
+
+       @Test
+       fun `page is annotated with correct menuname`() {
+           assertThat(page.menuName, equalTo("Rescue"))
+       }
+
+       @Test
+       fun `page is annotated with correct template path`() {
+           assertThat(page.templatePath, equalTo("/templates/rescue.html"))
+       }
+
 }