X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FEditImagePageTest.kt;h=5f5903150fac91f49dfa3f9e6ac2f1f61cd1845a;hp=f6eb02e6df318d12e88556b2e0247959153a45d1;hb=HEAD;hpb=2dd40fba7031cffb35a5156435547a5d964535c9

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 f6eb02e..5f59031 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/EditImagePageTest.kt
@@ -1,40 +1,31 @@
 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.Image.Modifier.ImageTitleMustNotBeEmpty
-import net.pterodactylus.sone.data.Sone
-import net.pterodactylus.sone.test.doThrow
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.mockBuilder
-import net.pterodactylus.sone.test.whenever
-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.impl.*
+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(::EditImagePage) {
+class EditImagePageTest : WebPageTest(::EditImagePage) {
 
-	private val image = mock<Image>()
-	private val modifier = mockBuilder<Modifier>()
 	private val sone = mock<Sone>()
-	private val album = mock<Album>()
+	private val image = ImageImpl("image-id").modify().setSone(sone).update()!!
+	private val album = AlbumImpl(sone, "album-id").also {
+		it.addImage(ImageImpl("1").modify().setSone(sone).update())
+		it.addImage(image)
+		it.addImage(ImageImpl("2").modify().setSone(sone).update())
+	}
 
 	@Before
 	fun setupImage() {
 		whenever(sone.isLocal).thenReturn(true)
-		whenever(album.id).thenReturn("album-id")
-		whenever(modifier.update()).thenReturn(image)
-		whenever(image.sone).thenReturn(sone)
-		whenever(image.album).thenReturn(album)
-		whenever(image.modify()).thenReturn(modifier)
 	}
 
 	@Test
@@ -44,13 +35,13 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 
 	@Test
 	fun `page requires login`() {
-	    assertThat(page.requiresLogin(), equalTo(true))
+		assertThat(page.requiresLogin(), equalTo(true))
 	}
 
 	@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"))
+		addTranslation("Page.EditImage.Title", "edit image page title")
+		assertThat(page.getPageTitle(soneRequest), equalTo("edit image page title"))
 	}
 
 	@Test
@@ -81,7 +72,7 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 		addHttpRequestPart("returnPage", "return.html")
 		addHttpRequestPart("moveLeft", "true")
 		verifyRedirect("return.html") {
-			verify(album).moveImageUp(image)
+			assertThat(album.images.indexOf(image), equalTo(0))
 			verify(core).touchConfiguration()
 		}
 	}
@@ -94,7 +85,7 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 		addHttpRequestPart("returnPage", "return.html")
 		addHttpRequestPart("moveRight", "true")
 		verifyRedirect("return.html") {
-			verify(album).moveImageDown(image)
+			assertThat(album.images.indexOf(image), equalTo(2))
 			verify(core).touchConfiguration()
 		}
 	}
@@ -106,7 +97,6 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 		addHttpRequestPart("image", "image-id")
 		addHttpRequestPart("returnPage", "return.html")
 		addHttpRequestPart("title", "   ")
-		whenever(modifier.update()).doThrow<ImageTitleMustNotBeEmpty>()
 		verifyRedirect("emptyImageTitle.html") {
 			verify(core, never()).touchConfiguration()
 		}
@@ -121,9 +111,8 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 		addHttpRequestPart("title", "Title")
 		addHttpRequestPart("description", "Description")
 		verifyRedirect("return.html") {
-			verify(modifier).setTitle("Title")
-			verify(modifier).setDescription("Description")
-			verify(modifier).update()
+			assertThat(image.title, equalTo("Title"))
+			assertThat(image.description, equalTo("Description"))
 			verify(core).touchConfiguration()
 		}
 	}
@@ -138,11 +127,15 @@ class EditImagePageTest: WebPageTest(::EditImagePage) {
 		addHttpRequestHeader("Host", "www.te.st")
 		addHttpRequestPart("description", "Get http://www.te.st/KSK@GPL.txt")
 		verifyRedirect("return.html") {
-			verify(modifier).setTitle("Title")
-			verify(modifier).setDescription("Get KSK@GPL.txt")
-			verify(modifier).update()
+			assertThat(image.title, equalTo("Title"))
+			assertThat(image.description, equalTo("Get KSK@GPL.txt"))
 			verify(core).touchConfiguration()
 		}
 	}
 
+	@Test
+	fun `page can be created by dependency injection`() {
+		assertThat(baseInjector.getInstance<EditImagePage>(), notNullValue())
+	}
+
 }