X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FUnlockSonePageTest.kt;h=6d46922a1b9494e4897acd5e31dd1860a6392c07;hp=390c1fc392d51aad7c30f52c3ceb008c27813476;hb=2dd40fba7031cffb35a5156435547a5d964535c9;hpb=de7568a82eb4150bf6d2b0553841b7b69f84c968

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
index 390c1fc..6d46922 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
@@ -3,7 +3,9 @@ package net.pterodactylus.sone.web.pages
 import net.pterodactylus.sone.data.Sone
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.UnlockSonePage
+import net.pterodactylus.util.web.Method.POST
+import org.hamcrest.MatcherAssert.assertThat
+import org.hamcrest.Matchers.equalTo
 import org.junit.Test
 import org.mockito.ArgumentMatchers.any
 import org.mockito.Mockito.never
@@ -12,33 +14,48 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [UnlockSonePage].
  */
-class UnlockSonePageTest : WebPageTest() {
+class UnlockSonePageTest: WebPageTest(::UnlockSonePage) {
 
-	private val page = UnlockSonePage(template, webInterface)
+	@Test
+	fun `page returns correct path`() {
+	    assertThat(page.path, equalTo("unlockSone.html"))
+	}
 
-	override fun getPage() = page
+	@Test
+	fun `page does not require login`() {
+	    assertThat(page.requiresLogin(), equalTo(false))
+	}
+
+	@Test
+	fun `page returns correct title`() {
+		addTranslation("Page.UnlockSone.Title", "unlock page title")
+	    assertThat(page.getPageTitle(freenetRequest), equalTo("unlock page title"))
+	}
 
 	@Test
-	fun `get request without sone redirects to return page`() {
-		addHttpRequestParameter("returnPage", "return.html")
+	fun `post request without sone redirects to return page`() {
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
 		verifyRedirect("return.html") {
 			verify(core, never()).unlockSone(any())
 		}
 	}
 
 	@Test
-	fun `get request without invalid local sone does not unlock any sone and redirects to return page`() {
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("sone", "invalid-sone")
+	fun `post request without invalid local sone does not unlock any sone and redirects to return page`() {
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("sone", "invalid-sone")
 		verifyRedirect("return.html") {
 			verify(core, never()).unlockSone(any())
 		}
 	}
 
 	@Test
-	fun `get request without remote sone does not unlock any sone and redirects to return page`() {
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("sone", "remote-sone")
+	fun `post request without remote sone does not unlock any sone and redirects to return page`() {
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("sone", "remote-sone")
 		addSone("remote-sone", mock<Sone>())
 		verifyRedirect("return.html") {
 			verify(core, never()).unlockSone(any())
@@ -46,9 +63,10 @@ class UnlockSonePageTest : WebPageTest() {
 	}
 
 	@Test
-	fun `get request with local sone unlocks sone and redirects to return page`() {
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("sone", "local-sone")
+	fun `post request with local sone unlocks sone and redirects to return page`() {
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("sone", "local-sone")
 		val sone = mock<Sone>().apply { whenever(isLocal).thenReturn(true) }
 		addLocalSone("local-sone", sone)
 		verifyRedirect("return.html") {