X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FSearchPageTest.kt;h=2da2b24925ce98cc5a39658de3f872a92d50cb78;hp=4f84c87f24147f0c55e5aa7d44f3145cb5a6c182;hb=03c29a3838e23ed0b9731ca4d84cf58038c30dfe;hpb=fb8f95b6ec904fc6767cd774211532d79cf5a2a7

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 4f84c87..2da2b24 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/SearchPageTest.kt
@@ -1,45 +1,49 @@
 package net.pterodactylus.sone.web.pages
 
 import com.google.common.base.Optional.absent
+import com.google.common.base.Ticker
 import net.pterodactylus.sone.data.Album
 import net.pterodactylus.sone.data.Image
 import net.pterodactylus.sone.data.Post
 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.getInstance
 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 net.pterodactylus.sone.utils.asOptional
+import net.pterodactylus.sone.web.baseInjector
 import org.hamcrest.MatcherAssert.assertThat
 import org.hamcrest.Matchers.contains
 import org.hamcrest.Matchers.equalTo
+import org.hamcrest.Matchers.notNullValue
 import org.junit.Test
+import java.util.concurrent.TimeUnit
+import java.util.concurrent.atomic.AtomicInteger
 
 /**
  * Unit test for [SearchPage].
  */
-class SearchPageTest : WebPageTest() {
+class SearchPageTest: WebPageTest({ template, webInterface -> SearchPage(template, webInterface, ticker) }) {
 
-	private val page = SearchPage(template, webInterface)
-
-	override fun getPage() = page
+	companion object {
+		val ticker = mock<Ticker>()
+	}
 
 	@Test
 	fun `page returns correct path`() {
-	    assertThat(page.path, equalTo("search.html"))
+		assertThat(page.path, equalTo("search.html"))
 	}
 
 	@Test
 	fun `page does not require login`() {
-	    assertThat(page.requiresLogin(), equalTo(false))
+		assertThat(page.requiresLogin(), equalTo(false))
 	}
 
 	@Test
 	fun `page returns correct title`() {
-	    addTranslation("Page.Search.Title", "search page title")
+		addTranslation("Page.Search.Title", "search page title")
 		assertThat(page.getPageTitle(freenetRequest), equalTo("search page title"))
 	}
 
@@ -62,28 +66,28 @@ class SearchPageTest : WebPageTest() {
 
 	@Test
 	fun `searching for sone link redirects to view sone page`() {
-		addSone("sone-id", mock<Sone>())
-		addHttpRequestParameter("query", "sone://sone-id")
-		verifyRedirect("viewSone.html?sone=sone-id")
+		addSone("Sone-ID", mock())
+		addHttpRequestParameter("query", "sone://Sone-ID")
+		verifyRedirect("viewSone.html?sone=Sone-ID")
 	}
 
 	@Test
 	fun `searching for sone link without prefix redirects to view sone page`() {
-		addSone("sone-id", mock<Sone>())
+		addSone("sone-id", mock())
 		addHttpRequestParameter("query", "sone-id")
 		verifyRedirect("viewSone.html?sone=sone-id")
 	}
 
 	@Test
 	fun `searching for a post link redirects to post page`() {
-		addPost("post-id", mock<Post>())
-		addHttpRequestParameter("query", "post://post-id")
-		verifyRedirect("viewPost.html?post=post-id")
+		addPost("Post-id", mock())
+		addHttpRequestParameter("query", "post://Post-id")
+		verifyRedirect("viewPost.html?post=Post-id")
 	}
 
 	@Test
 	fun `searching for a post ID without prefix redirects to post page`() {
-		addPost("post-id", mock<Post>())
+		addPost("post-id", mock())
 		addHttpRequestParameter("query", "post-id")
 		verifyRedirect("viewPost.html?post=post-id")
 	}
@@ -91,8 +95,8 @@ class SearchPageTest : WebPageTest() {
 	@Test
 	fun `searching for a reply link redirects to the post page`() {
 		val postReply = mock<PostReply>().apply { whenever(postId).thenReturn("post-id") }
-		addPostReply("reply-id", postReply)
-		addHttpRequestParameter("query", "reply://reply-id")
+		addPostReply("Reply-id", postReply)
+		addHttpRequestParameter("query", "reply://Reply-id")
 		verifyRedirect("viewPost.html?post=post-id")
 	}
 
@@ -106,28 +110,28 @@ class SearchPageTest : WebPageTest() {
 
 	@Test
 	fun `searching for an album link redirects to the image browser`() {
-		addAlbum("album-id", mock<Album>())
+		addAlbum("album-id", mock())
 		addHttpRequestParameter("query", "album://album-id")
 		verifyRedirect("imageBrowser.html?album=album-id")
 	}
 
 	@Test
 	fun `searching for an album ID redirects to the image browser`() {
-		addAlbum("album-id", mock<Album>())
+		addAlbum("album-id", mock())
 		addHttpRequestParameter("query", "album-id")
 		verifyRedirect("imageBrowser.html?album=album-id")
 	}
 
 	@Test
 	fun `searching for an image link redirects to the image browser`() {
-		addImage("image-id", mock<Image>())
+		addImage("image-id", mock())
 		addHttpRequestParameter("query", "image://image-id")
 		verifyRedirect("imageBrowser.html?image=image-id")
 	}
 
 	@Test
 	fun `searching for an image ID redirects to the image browser`() {
-		addImage("image-id", mock<Image>())
+		addImage("image-id", mock())
 		addHttpRequestParameter("query", "image-id")
 		verifyRedirect("imageBrowser.html?image=image-id")
 	}
@@ -327,7 +331,45 @@ class SearchPageTest : WebPageTest() {
 		}
 	}
 
+	@Test
+	fun `post search results are cached`() {
+		val post = createPost("with-match", "text")
+		val callCounter = AtomicInteger()
+		whenever(post.text).thenAnswer { callCounter.incrementAndGet(); "text" }
+		val sone = createSoneWithPost(post)
+		addSone("sone", sone)
+		addHttpRequestParameter("query", "text")
+		verifyNoRedirect {
+			assertThat(this["postHits"], contains(post))
+		}
+		verifyNoRedirect {
+			assertThat(callCounter.get(), equalTo(1))
+		}
+	}
+
+	@Test
+	fun `post search results are cached for five minutes`() {
+		val post = createPost("with-match", "text")
+		val callCounter = AtomicInteger()
+		whenever(post.text).thenAnswer { callCounter.incrementAndGet(); "text" }
+		val sone = createSoneWithPost(post)
+		addSone("sone", sone)
+		addHttpRequestParameter("query", "text")
+		verifyNoRedirect {
+			assertThat(this["postHits"], contains(post))
+		}
+		whenever(ticker.read()).thenReturn(TimeUnit.MINUTES.toNanos(5) + 1)
+		verifyNoRedirect {
+			assertThat(callCounter.get(), equalTo(2))
+		}
+	}
+
 	@Suppress("UNCHECKED_CAST")
 	private operator fun <T> get(key: String): T? = templateContext[key] as? T
 
+	@Test
+	fun `page can be created by dependency injection`() {
+	    assertThat(baseInjector.getInstance<SearchPage>(), notNullValue())
+	}
+
 }