From fb8f95b6ec904fc6767cd774211532d79cf5a2a7 Mon Sep 17 00:00:00 2001
From: =?utf8?q?David=20=E2=80=98Bombe=E2=80=99=20Roden?=
 <bombe@pterodactylus.net>
Date: Sat, 3 Jun 2017 11:13:28 +0200
Subject: [PATCH] Add more tests for search page

---
 .../pterodactylus/sone/web/pages/SearchPageTest.kt | 70 +++++++++++++++++++++-
 1 file changed, 69 insertions(+), 1 deletion(-)

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt
index 106b350..4f84c87 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt
@@ -8,10 +8,14 @@ import net.pterodactylus.sone.data.PostReply
 import net.pterodactylus.sone.data.Profile
 import net.pterodactylus.sone.data.Sone
 import net.pterodactylus.sone.test.asOptional
+import net.pterodactylus.sone.test.isOnPage
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
+import net.pterodactylus.sone.utils.Pagination
+import net.pterodactylus.util.template.TemplateContext
 import org.hamcrest.MatcherAssert.assertThat
 import org.hamcrest.Matchers.contains
+import org.hamcrest.Matchers.equalTo
 import org.junit.Test
 
 /**
@@ -24,6 +28,22 @@ class SearchPageTest : WebPageTest() {
 	override fun getPage() = page
 
 	@Test
+	fun `page returns correct path`() {
+	    assertThat(page.path, equalTo("search.html"))
+	}
+
+	@Test
+	fun `page does not require login`() {
+	    assertThat(page.requiresLogin(), equalTo(false))
+	}
+
+	@Test
+	fun `page returns correct title`() {
+	    addTranslation("Page.Search.Title", "search page title")
+		assertThat(page.getPageTitle(freenetRequest), equalTo("search page title"))
+	}
+
+	@Test
 	fun `empty query redirects to index page`() {
 		verifyRedirect("index.html")
 	}
@@ -226,7 +246,7 @@ class SearchPageTest : WebPageTest() {
 		whenever(this.text).thenReturn(text)
 	}
 
-	private fun createSone(id: String, firstName: String, middleName: String, lastName: String) = mock<Sone>().apply {
+	private fun createSone(id: String, firstName: String? = null, middleName: String? = null, lastName: String? = null) = mock<Sone>().apply {
 		whenever(this.id).thenReturn(id)
 		whenever(this.name).thenReturn(id)
 		whenever(this.profile).thenReturn(Profile(this).apply {
@@ -259,6 +279,54 @@ class SearchPageTest : WebPageTest() {
 		}
 	}
 
+	@Test
+	fun `sone hits are paginated correctly`() {
+		core.preferences.postsPerPage = 2
+		val sones = listOf(createSone("1Sone"), createSone("Other1"), createSone("22Sone"), createSone("333Sone"), createSone("Other2"))
+				.onEach { addSone(it.id, it) }
+		addHttpRequestParameter("query", "sone")
+		verifyNoRedirect {
+			assertThat(this["sonePagination"], isOnPage(0).hasPages(2))
+			assertThat(this["soneHits"], contains(sones[0], sones[2]))
+		}
+	}
+
+	@Test
+	fun `sone hits page 2 is shown correctly`() {
+		core.preferences.postsPerPage = 2
+		val sones = listOf(createSone("1Sone"), createSone("Other1"), createSone("22Sone"), createSone("333Sone"), createSone("Other2"))
+				.onEach { addSone(it.id, it) }
+		addHttpRequestParameter("query", "sone")
+		addHttpRequestParameter("sonePage", "1")
+		verifyNoRedirect {
+			assertThat(this["sonePagination"], isOnPage(1).hasPages(2))
+			assertThat(this["soneHits"], contains(sones[3]))
+		}
+	}
+
+	@Test
+	fun `post hits are paginated correctly`() {
+		core.preferences.postsPerPage = 2
+		val sones = listOf(createSoneWithPost("match1", "1Sone"), createSoneWithPost("no-match1", "Other1"), createSoneWithPost("match2", "22Sone"), createSoneWithPost("match3", "333Sone"), createSoneWithPost("no-match2", "Other2"))
+		addHttpRequestParameter("query", "sone")
+		verifyNoRedirect {
+			assertThat(this["postPagination"], isOnPage(0).hasPages(2))
+			assertThat(this["postHits"], contains(sones[0], sones[2]))
+		}
+	}
+
+	@Test
+	fun `post hits page 2 is shown correctly`() {
+		core.preferences.postsPerPage = 2
+		val sones = listOf(createSoneWithPost("match1", "1Sone"), createSoneWithPost("no-match1", "Other1"), createSoneWithPost("match2", "22Sone"), createSoneWithPost("match3", "333Sone"), createSoneWithPost("no-match2", "Other2"))
+		addHttpRequestParameter("query", "sone")
+		addHttpRequestParameter("postPage", "1")
+		verifyNoRedirect {
+			assertThat(this["postPagination"], isOnPage(1).hasPages(2))
+			assertThat(this["postHits"], contains(sones[3]))
+		}
+	}
+
 	@Suppress("UNCHECKED_CAST")
 	private operator fun <T> get(key: String): T? = templateContext[key] as? T
 
-- 
2.7.4