X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FRescuePageTest.kt;h=dabcbbee11bd01eeca3bf8b83f5844242b8d45a1;hp=547c580947744b4ff05330e18013b591b5880103;hb=a76956e389fcfe6282ad4ca7156bbf76327bb0c0;hpb=5ba707d40f9d2a20094aaabc21647aeec1feed46

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 547c580..dabcbbe 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
@@ -1,13 +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.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
@@ -15,23 +15,36 @@ 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`() {
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["soneRescuer"], equalTo<Any>(soneRescuer))
+		verifyNoRedirect {
+			assertThat(templateContext["soneRescuer"], equalTo<Any>(soneRescuer))
+		}
 	}
 
 	@Test
@@ -61,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"))
+	}
+
 }