Actually differentiate between GET and POST parameters

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

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
index f595b4a..a95e34c 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
@@ -2,10 +2,11 @@ package net.pterodactylus.sone.web.pages
 
 import net.pterodactylus.sone.data.Sone
 import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.web.pages.FollowSonePage
-import net.pterodactylus.sone.web.pages.WebPageTest
+import net.pterodactylus.sone.test.whenever
 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.Test
 import org.mockito.ArgumentMatchers
 import org.mockito.ArgumentMatchers.any
@@ -23,9 +24,25 @@ class FollowSonePageTest : WebPageTest() {
        override fun getPage() = page
 
        @Test
+       fun `page returns correct path`() {
+           assertThat(page.path, equalTo("followSone.html"))
+       }
+
+       @Test
+       fun `page requires login`() {
+           assertThat(page.requiresLogin(), equalTo(true))
+       }
+
+       @Test
+       fun `page returns correct title`() {
+           whenever(l10n.getString("Page.FollowSone.Title")).thenReturn("follow sone page title")
+               assertThat(page.getPageTitle(freenetRequest), equalTo("follow sone page title"))
+       }
+
+       @Test
        fun `get request does not redirect`() {
                request("", GET)
-               page.handleRequest(freenetRequest, templateContext)
+               page.processTemplate(freenetRequest, templateContext)
        }
 
        @Test
@@ -33,8 +50,8 @@ class FollowSonePageTest : WebPageTest() {
                request("", POST)
                val sone = mock<Sone>()
                addSone("sone-id", sone)
-               addHttpRequestParameter("sone", "sone-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("sone", "sone-id")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html") {
                        verify(core).followSone(currentSone, "sone-id")
                        verify(core).markSoneKnown(sone)
@@ -48,8 +65,8 @@ class FollowSonePageTest : WebPageTest() {
                addSone("sone-id1", firstSone)
                val secondSone = mock<Sone>()
                addSone("sone-id2", secondSone)
-               addHttpRequestParameter("sone", "sone-id1,sone-id2")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("sone", "sone-id1,sone-id2")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html") {
                        verify(core).followSone(currentSone, "sone-id1")
                        verify(core).followSone(currentSone, "sone-id2")
@@ -61,8 +78,8 @@ class FollowSonePageTest : WebPageTest() {
        @Test
        fun `a non-existing sone is not followed`() {
                request("", POST)
-               addHttpRequestParameter("sone", "sone-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("sone", "sone-id")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("return.html") {
                        verify(core, never()).followSone(ArgumentMatchers.eq(currentSone), anyString())
                        verify(core, never()).markSoneKnown(any<Sone>())