X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;ds=sidebyside;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FDeleteReplyPageTest.kt;h=d2c9844e995c3c39bf39a6ce4d3c1a665c6bfc68;hb=a76956e389fcfe6282ad4ca7156bbf76327bb0c0;hp=7944321ac1770bdd91ddc2d0c3aad179eec1d780;hpb=05fb821e72072bde52f383bdc5a988da67f66d0c;p=Sone.git

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPageTest.kt
index 7944321..d2c9844 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteReplyPageTest.kt
@@ -1,32 +1,23 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.data.PostReply
-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.DeleteReplyPage
-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 [DeleteReplyPage].
  */
-class DeleteReplyPageTest : WebPageTest() {
-
-	private val page = DeleteReplyPage(template, webInterface)
+class DeleteReplyPageTest: WebPageTest(::DeleteReplyPage) {
 
 	private val sone = mock<Sone>()
 	private val reply = mock<PostReply>()
 
-	override fun getPage() = page
-
 	@Before
 	fun setupReply() {
 		whenever(sone.isLocal).thenReturn(true)
@@ -45,7 +36,6 @@ class DeleteReplyPageTest : WebPageTest() {
 
 	@Test
 	fun `get request sets reply ID and return page in template context`() {
-		request("", GET)
 		addHttpRequestParameter("reply", "reply-id")
 		addHttpRequestParameter("returnPage", "return.html")
 		page.processTemplate(freenetRequest, templateContext)
@@ -55,7 +45,7 @@ class DeleteReplyPageTest : WebPageTest() {
 
 	@Test
 	fun `post request without any action sets reply ID and return page in template context`() {
-		request("", POST)
+		setMethod(POST)
 		addPostReply("reply-id", reply)
 		addHttpRequestPart("reply", "reply-id")
 		addHttpRequestPart("returnPage", "return.html")
@@ -66,13 +56,13 @@ class DeleteReplyPageTest : WebPageTest() {
 
 	@Test
 	fun `trying to delete a reply with an invalid ID results in no permission page`() {
-		request("", POST)
+		setMethod(POST)
 		verifyRedirect("noPermission.html")
 	}
 
 	@Test
 	fun `trying to delete a reply from a non-local sone results in no permission page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("reply", "reply-id")
 		whenever(sone.isLocal).thenReturn(false)
 		addPostReply("reply-id", reply)
@@ -81,7 +71,7 @@ class DeleteReplyPageTest : WebPageTest() {
 
 	@Test
 	fun `confirming deletion of reply deletes the reply and redirects to return page`() {
-		request("", POST)
+		setMethod(POST)
 		addPostReply("reply-id", reply)
 		addHttpRequestPart("reply", "reply-id")
 		addHttpRequestPart("returnPage", "return.html")
@@ -93,7 +83,7 @@ class DeleteReplyPageTest : WebPageTest() {
 
 	@Test
 	fun `aborting deletion of reply redirects to return page`() {
-		request("", POST)
+		setMethod(POST)
 		addPostReply("reply-id", reply)
 		addHttpRequestPart("reply", "reply-id")
 		addHttpRequestPart("returnPage", "return.html")
@@ -103,4 +93,14 @@ class DeleteReplyPageTest : WebPageTest() {
 		}
 	}
 
+	@Test
+	fun `page can be created by dependency injection`() {
+		assertThat(baseInjector.getInstance<DeleteReplyPage>(), notNullValue())
+	}
+
+	@Test
+	fun `page is annotated with correct template path`() {
+	    assertThat(page.templatePath, equalTo("/templates/deleteReply.html"))
+	}
+
 }