X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FCreatePostPageTest.kt;h=6212c90abf2cfb3161d0d42b43c5d0d49707ac17;hp=45a2451ba2ac5a03f5e8146088755f323c775969;hb=04709e23f38e9d447337682ba27201da5dc19bd9;hpb=de7568a82eb4150bf6d2b0553841b7b69f84c968

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/CreatePostPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/CreatePostPageTest.kt
index 45a2451..6212c90 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/CreatePostPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/CreatePostPageTest.kt
@@ -1,25 +1,21 @@
 package net.pterodactylus.sone.web.pages
 
-import com.google.common.base.Optional.absent
-import net.pterodactylus.sone.data.Sone
-import net.pterodactylus.sone.test.asOptional
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.CreatePostPage
-import net.pterodactylus.util.web.Method.POST
-import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers.equalTo
-import org.junit.Test
-import org.mockito.Mockito.verify
+import com.google.common.base.Optional.*
+import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.utils.*
+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 [CreatePostPage].
  */
-class CreatePostPageTest: WebPageTest() {
-
-	private val page = CreatePostPage(template, webInterface)
-
-	override fun getPage() = page
+class CreatePostPageTest : WebPageTest(::CreatePostPage) {
 
 	@Test
 	fun `page returns correct path`() {
@@ -33,65 +29,75 @@ class CreatePostPageTest: WebPageTest() {
 
 	@Test
 	fun `return page is set in template context`() {
-		addHttpRequestParameter("returnPage", "return.html")
+		addHttpRequestPart("returnPage", "return.html")
 		page.processTemplate(freenetRequest, templateContext)
 		assertThat(templateContext["returnPage"], equalTo<Any>("return.html"))
 	}
 
 	@Test
 	fun `post is created correctly`() {
-		request("", POST)
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("text", "post text")
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("text", "post text")
 		verifyRedirect("return.html") {
-			verify(core).createPost(currentSone, absent(), "post text")
+			verify(core).createPost(currentSone, null, "post text")
 		}
 	}
 
 	@Test
 	fun `creating an empty post is denied`() {
-		request("", POST)
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("text", "  ")
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("text", "  ")
 		page.processTemplate(freenetRequest, templateContext)
 		assertThat(templateContext["errorTextEmpty"], equalTo<Any>(true))
 	}
 
 	@Test
 	fun `a sender can be selected`() {
-		request("", POST)
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("text", "post text")
-		addHttpRequestParameter("sender", "sender-id")
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("text", "post text")
+		addHttpRequestPart("sender", "sender-id")
 		val sender = mock<Sone>()
 		addLocalSone("sender-id", sender)
 		verifyRedirect("return.html") {
-			verify(core).createPost(sender, absent(), "post text")
+			verify(core).createPost(sender, null, "post text")
 		}
 	}
 
 	@Test
 	fun `a recipient can be selected`() {
-		request("", POST)
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("text", "post text")
-		addHttpRequestParameter("recipient", "recipient-id")
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("text", "post text")
+		addHttpRequestPart("recipient", "recipient-id")
 		val recipient = mock<Sone>()
 		addSone("recipient-id", recipient)
 		verifyRedirect("return.html") {
-			verify(core).createPost(currentSone, recipient.asOptional(), "post text")
+			verify(core).createPost(currentSone, recipient, "post text")
 		}
 	}
 
 	@Test
 	fun `text is filtered correctly`() {
-		request("", POST)
-		addHttpRequestParameter("returnPage", "return.html")
-		addHttpRequestParameter("text", "post http://localhost:12345/KSK@foo text")
+		setMethod(POST)
+		addHttpRequestPart("returnPage", "return.html")
+		addHttpRequestPart("text", "post http://localhost:12345/KSK@foo text")
 		addHttpRequestHeader("Host", "localhost:12345")
 		verifyRedirect("return.html") {
-			verify(core).createPost(currentSone, absent(), "post KSK@foo text")
+			verify(core).createPost(currentSone, null, "post KSK@foo text")
 		}
 	}
 
+	@Test
+	fun `page can be created by dependency injection`() {
+		assertThat(baseInjector.getInstance<CreatePostPage>(), notNullValue())
+	}
+
+	@Test
+	fun `page is annotated with correct template path`() {
+		assertThat(page.templatePath, equalTo("/templates/createPost.html"))
+	}
+
 }