🎨 Clean up imports

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

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt
index 810e149..7e26e9a 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt
@@ -1,35 +1,26 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.data.Album
-import net.pterodactylus.sone.data.Image
-import net.pterodactylus.sone.data.Image.Modifier
-import net.pterodactylus.sone.data.Sone
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.mockBuilder
-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.Before
-import org.junit.Test
-import org.mockito.Mockito.never
-import org.mockito.Mockito.verify
+import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.data.Image.*
+import net.pterodactylus.sone.data.Image.Modifier.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.util.web.Method.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
+import org.mockito.Mockito.*
 
 /**
  * Unit test for [EditImagePage].
  */
-class EditImagePageTest : WebPageTest() {
-
-       private val page = EditImagePage(template, webInterface)
+class EditImagePageTest: WebPageTest(::EditImagePage) {
 
        private val image = mock<Image>()
        private val modifier = mockBuilder<Modifier>()
        private val sone = mock<Sone>()
        private val album = mock<Album>()
 
-       override fun getPage() = page
-
        @Before
        fun setupImage() {
                whenever(sone.isLocal).thenReturn(true)
@@ -53,37 +44,36 @@ class EditImagePageTest : WebPageTest() {
        @Test
        fun `page returns correct title`() {
                whenever(l10n.getString("Page.EditImage.Title")).thenReturn("edit image page title")
-           assertThat(page.getPageTitle(freenetRequest), equalTo("edit image page title"))
+           assertThat(page.getPageTitle(soneRequest), equalTo("edit image page title"))
        }
 
        @Test
        fun `get request does not redirect`() {
-               request("", GET)
-               page.handleRequest(freenetRequest, templateContext)
+               page.processTemplate(freenetRequest, templateContext)
        }
 
        @Test
        fun `post request with invalid image redirects to invalid page`() {
-               request("", POST)
+               setMethod(POST)
                verifyRedirect("invalid.html")
        }
 
        @Test
        fun `post request with valid image from non-local sone redirects to no permission page`() {
-               request("", POST)
+               setMethod(POST)
                whenever(sone.isLocal).thenReturn(false)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
+               addHttpRequestPart("image", "image-id")
                verifyRedirect("noPermission.html")
        }
 
        @Test
        fun `post request with valid image and move left requested moves image left and redirects to return page`() {
-               request("", POST)
+               setMethod(POST)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("moveLeft", "true")
+               addHttpRequestPart("image", "image-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("moveLeft", "true")
                verifyRedirect("return.html") {
                        verify(album).moveImageUp(image)
                        verify(core).touchConfiguration()
@@ -92,11 +82,11 @@ class EditImagePageTest : WebPageTest() {
 
        @Test
        fun `post request with valid image and move right requested moves image right and redirects to return page`() {
-               request("", POST)
+               setMethod(POST)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("moveRight", "true")
+               addHttpRequestPart("image", "image-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("moveRight", "true")
                verifyRedirect("return.html") {
                        verify(album).moveImageDown(image)
                        verify(core).touchConfiguration()
@@ -105,11 +95,12 @@ class EditImagePageTest : WebPageTest() {
 
        @Test
        fun `post request with valid image but only whitespace in the title redirects to empty image title page`() {
-               request("", POST)
+               setMethod(POST)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("title", "   ")
+               addHttpRequestPart("image", "image-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("title", "   ")
+               whenever(modifier.update()).doThrow<ImageTitleMustNotBeEmpty>()
                verifyRedirect("emptyImageTitle.html") {
                        verify(core, never()).touchConfiguration()
                }
@@ -117,12 +108,12 @@ class EditImagePageTest : WebPageTest() {
 
        @Test
        fun `post request with valid image title and description modifies image and redirects to reutrn page`() {
-               request("", POST)
+               setMethod(POST)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("title", "Title")
-               addHttpRequestParameter("description", "Description")
+               addHttpRequestPart("image", "image-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("title", "Title")
+               addHttpRequestPart("description", "Description")
                verifyRedirect("return.html") {
                        verify(modifier).setTitle("Title")
                        verify(modifier).setDescription("Description")
@@ -132,14 +123,14 @@ class EditImagePageTest : WebPageTest() {
        }
 
        @Test
-       fun `post request with image title and description modifies image with filtered description and redirects to reutrn page`() {
-               request("", POST)
+       fun `post request with image title and description modifies image with filtered description and redirects to return page`() {
+               setMethod(POST)
                addImage("image-id", image)
-               addHttpRequestParameter("image", "image-id")
-               addHttpRequestParameter("returnPage", "return.html")
-               addHttpRequestParameter("title", "Title")
+               addHttpRequestPart("image", "image-id")
+               addHttpRequestPart("returnPage", "return.html")
+               addHttpRequestPart("title", "Title")
                addHttpRequestHeader("Host", "www.te.st")
-               addHttpRequestParameter("description", "Get http://www.te.st/KSK@GPL.txt")
+               addHttpRequestPart("description", "Get http://www.te.st/KSK@GPL.txt")
                verifyRedirect("return.html") {
                        verify(modifier).setTitle("Title")
                        verify(modifier).setDescription("Get KSK@GPL.txt")
@@ -148,4 +139,9 @@ class EditImagePageTest : WebPageTest() {
                }
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+               assertThat(baseInjector.getInstance<EditImagePage>(), notNullValue())
+       }
+
 }