X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FImageBrowserPageTest.kt;h=1533f4badcd2a035e2c0ce7d9f102c2b6a104506;hp=19d24ed69cf42de4368829f0a185c96d027c0ee2;hb=d5cd41949b7e62fdc40ef7390661dcda8c18a139;hpb=cee09da2887a2a3d86d8e694c3fbdcd1f74cefce

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/ImageBrowserPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/ImageBrowserPageTest.kt
index 19d24ed..1533f4b 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/ImageBrowserPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/ImageBrowserPageTest.kt
@@ -1,93 +1,106 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.data.Album
-import net.pterodactylus.sone.data.Image
-import net.pterodactylus.sone.data.Sone
-import net.pterodactylus.sone.test.mock
-import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.ImageBrowserPage
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.util.web.Method.GET
-import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers.contains
-import org.hamcrest.Matchers.equalTo
-import org.junit.Test
+import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.sone.web.page.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
+import java.net.*
 
 /**
  * Unit test for [ImageBrowserPage].
  */
-class ImageBrowserPageTest : WebPageTest() {
-
-	private val page = ImageBrowserPage(template, webInterface)
+class ImageBrowserPageTest : WebPageTest(::ImageBrowserPage) {
 
 	@Test
 	fun `page returns correct path`() {
-	    assertThat(page.path, equalTo("imageBrowser.html"))
+		assertThat(page.path, equalTo("imageBrowser.html"))
 	}
 
 	@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.ImageBrowser.Title")).thenReturn("image browser page title")
-	    assertThat(page.getPageTitle(freenetRequest), equalTo("image browser page title"))
+		addTranslation("Page.ImageBrowser.Title", "image browser page title")
+		assertThat(page.getPageTitle(soneRequest), equalTo("image browser page title"))
 	}
 
 	@Test
 	fun `get request with album sets album and page in template context`() {
-		request("", GET)
 		val album = mock<Album>()
 		addAlbum("album-id", album)
 		addHttpRequestParameter("album", "album-id")
 		addHttpRequestParameter("page", "5")
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["albumRequested"], equalTo<Any>(true))
-		assertThat(templateContext["album"], equalTo<Any>(album))
-		assertThat(templateContext["page"], equalTo<Any>("5"))
+		verifyNoRedirect {
+			assertThat(templateContext["albumRequested"], equalTo<Any>(true))
+			assertThat(templateContext["album"], equalTo<Any>(album))
+			assertThat(templateContext["page"], equalTo<Any>("5"))
+		}
 	}
 
 	@Test
 	fun `get request with image sets image in template context`() {
-		request("", GET)
 		val image = mock<Image>()
 		addImage("image-id", image)
 		addHttpRequestParameter("image", "image-id")
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["imageRequested"], equalTo<Any>(true))
-		assertThat(templateContext["image"], equalTo<Any>(image))
+		verifyNoRedirect {
+			assertThat(templateContext["imageRequested"], equalTo<Any>(true))
+			assertThat(templateContext["image"], equalTo<Any>(image))
+		}
 	}
 
 	@Test
 	fun `get request with sone sets sone in template context`() {
-		request("", GET)
 		val sone = mock<Sone>()
 		addSone("sone-id", sone)
 		addHttpRequestParameter("sone", "sone-id")
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["soneRequested"], equalTo<Any>(true))
-		assertThat(templateContext["sone"], equalTo<Any>(sone))
+		verifyNoRedirect {
+			assertThat(templateContext["soneRequested"], equalTo<Any>(true))
+			assertThat(templateContext["sone"], equalTo<Any>(sone))
+		}
 	}
 
 	@Test
 	fun `get request with mode of gallery sets albums and page in template context`() {
-		request("", GET)
 		val firstSone = createSone("first album", "second album")
 		addSone("sone1", firstSone)
 		val secondSone = createSone("third album", "fourth album")
 		addSone("sone2", secondSone)
 		addHttpRequestParameter("mode", "gallery")
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["galleryRequested"], equalTo<Any>(true))
-		@Suppress("UNCHECKED_CAST")
-		assertThat(templateContext["albums"] as Iterable<Album>, contains(
-				firstSone.rootAlbum.albums[0],
-				secondSone.rootAlbum.albums[1],
-				firstSone.rootAlbum.albums[1],
-				secondSone.rootAlbum.albums[0]
-		))
+		verifyNoRedirect {
+			assertThat(templateContext["galleryRequested"], equalTo<Any>(true))
+			@Suppress("UNCHECKED_CAST")
+			assertThat(templateContext["albums"] as Iterable<Album>, contains(
+					firstSone.rootAlbum.albums[0],
+					secondSone.rootAlbum.albums[1],
+					firstSone.rootAlbum.albums[1],
+					secondSone.rootAlbum.albums[0]
+			))
+		}
+	}
+
+	@Test
+	fun `get request for gallery can show second page`() {
+		core.preferences.newImagesPerPage = 2
+		val firstSone = createSone("first album", "second album")
+		addSone("sone1", firstSone)
+		val secondSone = createSone("third album", "fourth album")
+		addSone("sone2", secondSone)
+		addHttpRequestParameter("mode", "gallery")
+		addHttpRequestParameter("page", "1")
+		verifyNoRedirect {
+			assertThat(templateContext["galleryRequested"], equalTo<Any>(true))
+			@Suppress("UNCHECKED_CAST")
+			assertThat(templateContext["albums"] as Iterable<Album>, contains(
+					firstSone.rootAlbum.albums[1],
+					secondSone.rootAlbum.albums[0]
+			))
+		}
 	}
 
 	private fun createSone(firstAlbumTitle: String, secondAlbumTitle: String): Sone {
@@ -108,15 +121,30 @@ class ImageBrowserPageTest : WebPageTest() {
 
 	@Test
 	fun `requesting nothing will show the albums of the current sone`() {
-		request("", GET)
-		page.handleRequest(freenetRequest, templateContext)
-		assertThat(templateContext["soneRequested"], equalTo<Any>(true))
-		assertThat(templateContext["sone"], equalTo<Any>(currentSone))
+		verifyNoRedirect {
+			assertThat(templateContext["soneRequested"], equalTo<Any>(true))
+			assertThat(templateContext["sone"], equalTo<Any>(currentSone))
+		}
 	}
 
 	@Test
 	fun `page is link-excepted`() {
-	    assertThat(page.isLinkExcepted(null), equalTo(true))
+		assertThat(page.isLinkExcepted(URI("")), equalTo(true))
+	}
+
+	@Test
+	fun `page can be created by dependency injection`() {
+		assertThat(baseInjector.getInstance<ImageBrowserPage>(), notNullValue())
+	}
+
+	@Test
+	fun `page is annotated with correct menuname`() {
+		assertThat(page.menuName, equalTo("ImageBrowser"))
+	}
+
+	@Test
+	fun `page is annotated with correct template path`() {
+		assertThat(page.templatePath, equalTo("/templates/imageBrowser.html"))
 	}
 
 }