X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FRescuePageTest.kt;h=1f62a92ef0cace81999964ac74a05e1aa42d9b79;hb=e7319b53b914961d59b8fce999da75adfab54009;hp=cbc8968c73b9e969ebfb47a46830b4488606bfd6;hpb=05fb821e72072bde52f383bdc5a988da67f66d0c;p=Sone.git

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..1f62a92 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/RescuePageTest.kt
@@ -3,8 +3,6 @@ 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
@@ -17,7 +15,7 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [RescuePage].
  */
-class RescuePageTest : WebPageTest() {
+class RescuePageTest: WebPageTest() {
 
 	private val page = RescuePage(template, webInterface)
 
@@ -31,21 +29,37 @@ class RescuePageTest : WebPageTest() {
 	}
 
 	@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(freenetRequest), 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 +69,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 +78,16 @@ 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()
+		}
+	}
+
+
 }