X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FUploadImagePageTest.kt;h=58e63a2fa3f9fada5bfba7fcb411a5750fd64ac7;hb=ffd92ca2374c0b2218e583d02e0bdd24b8c110ae;hp=232601b6a71117043fd876d19af6efc1635e49e5;hpb=de7568a82eb4150bf6d2b0553841b7b69f84c968;p=Sone.git

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt
index 232601b..58e63a2 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt
@@ -8,7 +8,6 @@ import net.pterodactylus.sone.data.TemporaryImage
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.mockBuilder
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.UploadImagePage
 import net.pterodactylus.util.web.Method.POST
 import org.hamcrest.MatcherAssert.assertThat
 import org.hamcrest.Matchers.equalTo
@@ -21,32 +20,47 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [UploadImagePage].
  */
-class UploadImagePageTest : WebPageTest() {
+class UploadImagePageTest: WebPageTest(::UploadImagePage) {
 
 	private val parentAlbum = mock<Album>().apply {
 		whenever(id).thenReturn("parent-id")
 		whenever(sone).thenReturn(currentSone)
 	}
 
-	override fun getPage() = UploadImagePage(template, webInterface)
+	@Test
+	fun `page returns correct path`() {
+	    assertThat(page.path, equalTo("uploadImage.html"))
+	}
+
+	@Test
+	fun `page requires login`() {
+	    assertThat(page.requiresLogin(), equalTo(true))
+	}
+
+	@Test
+	fun `page returns correct title`() {
+	    addTranslation("Page.UploadImage.Title", "upload image page title")
+		assertThat(page.getPageTitle(freenetRequest), equalTo("upload image page title"))
+	}
 
 	@Test
 	fun `get request does not redirect or upload anything`() {
-		page.handleRequest(freenetRequest, templateContext)
-		verify(core, never()).createTemporaryImage(any(), any())
-		verify(core, never()).createImage(any(), any(), any())
+		verifyNoRedirect {
+			verify(core, never()).createTemporaryImage(any(), any())
+			verify(core, never()).createImage(any(), any(), any())
+		}
 	}
 
 	@Test
 	fun `post request without parent results in no permission error page`() {
-		request("", POST)
+		setMethod(POST)
 		verifyRedirect("noPermission.html")
 	}
 
 	@Test
 	fun `post request with parent that is not the current sone results in no permission error page`() {
-		request("", POST)
-		addHttpRequestParameter("parent", "parent-id")
+		setMethod(POST)
+		addHttpRequestPart("parent", "parent-id")
 		whenever(parentAlbum.sone).thenReturn(mock<Sone>())
 		addAlbum("parent-id", parentAlbum)
 		verifyRedirect("noPermission.html")
@@ -54,33 +68,36 @@ class UploadImagePageTest : WebPageTest() {
 
 	@Test
 	fun `post request with empty name redirects to error page`() {
-		request("", POST)
+		setMethod(POST)
 		addAlbum("parent-id", parentAlbum)
-		addHttpRequestParameter("parent", "parent-id")
-		addHttpRequestParameter("title", " ")
+		addHttpRequestPart("parent", "parent-id")
+		addHttpRequestPart("title", " ")
 		verifyRedirect("emptyImageTitle.html")
 	}
 
 	@Test
 	fun `uploading an invalid image results in no redirect and message set in template context`() {
-		request("", POST)
+		setMethod(POST)
 		addAlbum("parent-id", parentAlbum)
-		addHttpRequestParameter("parent", "parent-id")
-		addHttpRequestParameter("title", "title")
-		addUploadedFile("image", "image.png", "image/png", "no-image.png")
-		page.handleRequest(freenetRequest, templateContext)
-		verify(core, never()).createTemporaryImage(any(), any())
-		assertThat(templateContext["messages"] as String?, equalTo<String>("Page.UploadImage.Error.InvalidImage"))
+		addHttpRequestPart("parent", "parent-id")
+		addHttpRequestPart("title", "title")
+		addUploadedFile("image", "image.png", "image/png", "upload-image-invalid-image.png")
+		addTranslation("Page.UploadImage.Error.InvalidImage", "upload error - invalid image")
+		verifyNoRedirect {
+			verify(core, never()).createTemporaryImage(any(), any())
+			assertThat(templateContext["messages"] as String, equalTo("upload error - invalid image"))
+		}
 	}
 
 	@Test
 	fun `uploading a valid image uploads image and redirects to album browser`() {
-		request("", POST)
+		setMethod(POST)
 		addAlbum("parent-id", parentAlbum)
-		addHttpRequestParameter("parent", "parent-id")
-		addHttpRequestParameter("title", "Title")
-		addHttpRequestParameter("description", "Description")
-		addUploadedFile("image", "image.png", "image/png", "image.png")
+		addHttpRequestPart("parent", "parent-id")
+		addHttpRequestPart("title", "Title")
+		addHttpRequestPart("description", "Description @ http://localhost:8888/KSK@foo")
+		addHttpRequestHeader("Host", "localhost:8888")
+		addUploadedFile("image", "upload-image-value-image.png", "image/png", "upload-image-value-image.png")
 		val temporaryImage = TemporaryImage("temp-image")
 		val imageModifier = mockBuilder<Modifier>()
 		val image = mock<Image>().apply {
@@ -93,7 +110,7 @@ class UploadImagePageTest : WebPageTest() {
 			verify(imageModifier).setWidth(2)
 			verify(imageModifier).setHeight(1)
 			verify(imageModifier).setTitle("Title")
-			verify(imageModifier).setDescription("Description")
+			verify(imageModifier).setDescription("Description @ KSK@foo")
 			verify(imageModifier).update()
 		}
 	}