🎨 Fix formatting

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

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/DeletePostPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/DeletePostPageTest.kt
index 152e70d..9464920 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/DeletePostPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/DeletePostPageTest.kt
@@ -1,32 +1,23 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.data.Post
-import net.pterodactylus.sone.data.Sone
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.DeletePostPage
-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.Before
-import org.junit.Test
-import org.mockito.Mockito.never
-import org.mockito.Mockito.verify
+import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.sone.web.page.*
+import net.pterodactylus.util.web.Method.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
+import org.mockito.Mockito.*
 
 /**
  * Unit test for [DeletePostPage].
  */
-class DeletePostPageTest : WebPageTest() {
-
-       private val page = DeletePostPage(template, webInterface)
+class DeletePostPageTest : WebPageTest(::DeletePostPage) {
 
        private val post = mock<Post>()
        private val sone = mock<Sone>()
 
-       override fun getPage() = page
-
        @Before
        fun setupPost() {
                whenever(post.sone).thenReturn(sone)
@@ -35,23 +26,21 @@ class DeletePostPageTest : WebPageTest() {
 
        @Test
        fun `page returns correct path`() {
-           assertThat(page.path, equalTo("deletePost.html"))
+               assertThat(page.path, equalTo("deletePost.html"))
        }
 
        @Test
        fun `page requires login`() {
-           assertThat(page.requiresLogin(), equalTo(true))
+               assertThat(page.requiresLogin(), equalTo(true))
        }
 
        @Test
        fun `get request with invalid post redirects to no permission page`() {
-               request("", GET)
                verifyRedirect("noPermission.html")
        }
 
        @Test
        fun `get request with valid post sets post and return page in template context`() {
-               request("", GET)
                addPost("post-id", post)
                addHttpRequestParameter("post", "post-id")
                addHttpRequestParameter("returnPage", "return.html")
@@ -62,27 +51,27 @@ class DeletePostPageTest : WebPageTest() {
 
        @Test
        fun `post request with invalid post redirects to no permission page`() {
-               request("", POST)
+               setMethod(POST)
                verifyRedirect("noPermission.html")
        }
 
        @Test
        fun `post request with post from non-local sone redirects to no permission page`() {
-               request("", POST)
+               setMethod(POST)
                whenever(sone.isLocal).thenReturn(false)
                addPost("post-id", post)
-               addHttpRequestParameter("post", "post-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("post", "post-id")
+               addHttpRequestPart("returnPage", "return.html")
                verifyRedirect("noPermission.html")
        }
 
        @Test
        fun `post request with confirmation deletes post and redirects to return page`() {
-               request("", POST)
+               setMethod(POST)
                addPost("post-id", post)
-               addHttpRequestParameter("post", "post-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("confirmDelete", "true")
+               addHttpRequestPart("post", "post-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("confirmDelete", "true")
                verifyRedirect("return.html") {
                        verify(core).deletePost(post)
                }
@@ -90,11 +79,11 @@ class DeletePostPageTest : WebPageTest() {
 
        @Test
        fun `post request with abort delete does not delete post and redirects to return page`() {
-               request("", POST)
+               setMethod(POST)
                addPost("post-id", post)
-               addHttpRequestParameter("post", "post-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("abortDelete", "true")
+               addHttpRequestPart("post", "post-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("abortDelete", "true")
                verifyRedirect("return.html") {
                        verify(core, never()).deletePost(post)
                }
@@ -102,13 +91,23 @@ class DeletePostPageTest : WebPageTest() {
 
        @Test
        fun `post request without delete or abort sets post in template context`() {
-               request("", POST)
+               setMethod(POST)
                addPost("post-id", post)
-               addHttpRequestParameter("post", "post-id")
-               addHttpRequestParameter("returnPage", "return.html")
+               addHttpRequestPart("post", "post-id")
+               addHttpRequestPart("returnPage", "return.html")
                page.processTemplate(freenetRequest, templateContext)
                assertThat(templateContext["post"], equalTo<Any>(post))
                assertThat(templateContext["returnPage"], equalTo<Any>("return.html"))
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+               assertThat(baseInjector.getInstance<DeletePostPage>(), notNullValue())
+       }
+
+       @Test
+       fun `page is annotated with correct template path`() {
+               assertThat(page.templatePath, equalTo("/templates/deletePost.html"))
+       }
+
 }