Set URI and request method separately in tests

[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..5e32d84 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnlockSonePageTest.kt
@@ -4,6 +4,7 @@ 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.junit.Test
 import org.mockito.ArgumentMatchers.any
 import org.mockito.Mockito.never
@@ -19,26 +20,29 @@ class UnlockSonePageTest : WebPageTest() {
        override fun getPage() = page
 
        @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 +50,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") {