Remove unnecessary type parameters

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

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..0fff35c 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
@@ -1,9 +1,14 @@
 package net.pterodactylus.sone.web.pages
 
 import net.pterodactylus.sone.data.Sone
+import net.pterodactylus.sone.test.getInstance
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.UnlockSonePage
+import net.pterodactylus.sone.web.baseInjector
+import net.pterodactylus.util.web.Method.POST
+import org.hamcrest.MatcherAssert.assertThat
+import org.hamcrest.Matchers.equalTo
+import org.hamcrest.Matchers.notNullValue
 import org.junit.Test
 import org.mockito.ArgumentMatchers.any
 import org.mockito.Mockito.never
@@ -12,43 +17,59 @@ 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"))
+       }
+
+       @Test
+       fun `page does not require login`() {
+           assertThat(page.requiresLogin(), equalTo(false))
+       }
 
-       override fun getPage() = page
+       @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")
-               addSone("remote-sone", mock<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())
                verifyRedirect("return.html") {
                        verify(core, never()).unlockSone(any())
                }
        }
 
        @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") {
@@ -56,4 +77,9 @@ class UnlockSonePageTest : WebPageTest() {
                }
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+           assertThat(baseInjector.getInstance<UnlockSonePage>(), notNullValue())
+       }
+
 }