From: David ‘Bombe’ Roden Date: Tue, 27 Jun 2017 20:31:02 +0000 (+0200) Subject: Convert upload image page test to use new web page test base X-Git-Tag: 0.9.7^2~114 X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=commitdiff_plain;h=655d836ea81170e883cf229573bc5edd0169cee7 Convert upload image page test to use new web page test base --- 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 0607c61..9c0d144 100644 --- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt +++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UploadImagePageTest.kt @@ -20,15 +20,12 @@ import org.mockito.Mockito.verify /** * Unit test for [UploadImagePage]. */ -class UploadImagePageTest : WebPageTest() { +class UploadImagePageTest: WebPageTest2(::UploadImagePage) { private val parentAlbum = mock().apply { whenever(id).thenReturn("parent-id") whenever(sone).thenReturn(currentSone) } - private val page = UploadImagePage(template, webInterface) - - override fun getPage() = page @Test fun `page returns correct path`() { diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/WebPageTest2.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/WebPageTest2.kt index 5682b11..e8fbd98 100644 --- a/src/test/kotlin/net/pterodactylus/sone/web/pages/WebPageTest2.kt +++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/WebPageTest2.kt @@ -2,7 +2,9 @@ package net.pterodactylus.sone.web.pages import com.google.common.eventbus.EventBus import freenet.clients.http.ToadletContext +import freenet.support.SimpleReadOnlyArrayBucket import freenet.support.api.HTTPRequest +import freenet.support.api.HTTPUploadedFile import net.pterodactylus.sone.core.Preferences import net.pterodactylus.sone.data.Album import net.pterodactylus.sone.data.Image @@ -27,7 +29,6 @@ import net.pterodactylus.util.web.Method import net.pterodactylus.util.web.Method.GET import net.pterodactylus.util.web.Response import org.junit.Assert.fail -import org.junit.Before import org.mockito.ArgumentMatchers.anyBoolean import org.mockito.ArgumentMatchers.anyInt import org.mockito.ArgumentMatchers.anyLong @@ -62,6 +63,9 @@ open class WebPageTest2(pageSupplier: (Template, WebInterface) -> SoneTemplatePa private val requestHeaders = mutableMapOf() private val getRequestParameters = mutableMapOf>() private val postRequestParameters = mutableMapOf() + private val uploadedFileNames = mutableMapOf() + private val uploadedFileContentTypes = mutableMapOf() + private val uploadedFileResources = mutableMapOf() private val ownIdentities = mutableSetOf() private val allSones = mutableMapOf() private val localSones = mutableMapOf() @@ -120,6 +124,17 @@ open class WebPageTest2(pageSupplier: (Template, WebInterface) -> SoneTemplatePa whenever(httpRequest.getMultipleIntParam(anyString())).then { getRequestParameters[it[0]]?.map { it.toIntOrNull() ?: 0 } ?: emptyArray() } whenever(httpRequest.isPartSet(anyString())).then { it[0] in postRequestParameters } whenever(httpRequest.getPartAsStringFailsafe(anyString(), anyInt())).then { postRequestParameters[it[0]]?.decode()?.take(it[1]) ?: "" } + whenever(httpRequest.getUploadedFile(anyString())).then { + it.get(0).takeIf { it in uploadedFileNames } + ?.let { name -> UploadedFile(uploadedFileNames[name]!!, uploadedFileContentTypes[name]!!, uploadedFileResources[name]!!) + } + } + } + + private class UploadedFile(private val filename: String, private val contentType: String, private val resourceName: String): HTTPUploadedFile { + override fun getFilename() = filename + override fun getContentType() = contentType + override fun getData() = javaClass.getResourceAsStream(resourceName).readBytes().let(::SimpleReadOnlyArrayBucket) } private fun ByteArray.decode(charset: Charset = UTF_8) = String(this, charset) @@ -203,6 +218,12 @@ open class WebPageTest2(pageSupplier: (Template, WebInterface) -> SoneTemplatePa whenever(core.getTemporaryImage(id)).thenReturn(temporaryImage) } + fun addUploadedFile(name: String, filename: String, contentType: String, resource: String) { + uploadedFileNames[name] = filename + uploadedFileContentTypes[name] = contentType + uploadedFileResources[name] = resource + } + fun verifyNoRedirect(assertions: () -> Unit) { var caughtException: Exception? = null try {