Remove unnecessary type parameters
[Sone.git] / src / test / kotlin / net / pterodactylus / sone / web / pages / NewPageTest.kt
index 72d87f8..5c71e5d 100644 (file)
@@ -2,11 +2,17 @@ package net.pterodactylus.sone.web.pages
 
 import net.pterodactylus.sone.data.Post
 import net.pterodactylus.sone.data.PostReply
-import net.pterodactylus.sone.test.asOptional
+import net.pterodactylus.sone.test.getInstance
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
+import net.pterodactylus.sone.utils.Pagination
+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.containsInAnyOrder
+import org.hamcrest.Matchers.equalTo
+import org.hamcrest.Matchers.notNullValue
 import org.junit.Before
 import org.junit.Test
 import java.util.Arrays.asList
@@ -14,11 +20,7 @@ import java.util.Arrays.asList
 /**
  * Unit test for [NewPage].
  */
-class NewPageTest: WebPageTest() {
-
-       private val page = NewPage(template, webInterface)
-
-       override fun getPage() = page
+class NewPageTest: WebPageTest(::NewPage) {
 
        @Before
        fun setupNumberOfPostsPerPage() {
@@ -26,10 +28,26 @@ class NewPageTest: WebPageTest() {
        }
 
        @Test
+       fun `page returns correct path`() {
+               assertThat(page.path, equalTo("new.html"))
+       }
+
+       @Test
+       fun `page does not require login`() {
+               assertThat(page.requiresLogin(), equalTo(false))
+       }
+
+       @Test
+       fun `page returns correct title`() {
+               addTranslation("Page.New.Title", "new page title")
+               assertThat(page.getPageTitle(freenetRequest), equalTo("new page title"))
+       }
+
+       @Test
        fun `posts are not duplicated when they come from both new posts and new replies notifications`() {
-               val extraPost = mock<Post>()
-               val posts = asList(mock<Post>(), mock<Post>())
-               val postReplies = asList(mock<PostReply>(), mock<PostReply>())
+               val extraPost = mock<Post>().withTime(2000)
+               val posts = asList(mock<Post>().withTime(1000), mock<Post>().withTime(3000))
+               val postReplies = asList(mock<PostReply>(), mock())
                whenever(postReplies[0].post).thenReturn(posts[0].asOptional())
                whenever(postReplies[1].post).thenReturn(extraPost.asOptional())
                whenever(webInterface.getNewPosts(currentSone)).thenReturn(posts)
@@ -37,8 +55,38 @@ class NewPageTest: WebPageTest() {
 
                verifyNoRedirect {
                        val renderedPosts = templateContext.get<List<Post>>("posts", List::class.java)
-                       assertThat(renderedPosts, containsInAnyOrder(posts[0], posts[1], extraPost))
+                       assertThat(renderedPosts, containsInAnyOrder(posts[1], extraPost, posts[0]))
                }
        }
 
+       private fun Post.withTime(time: Long) = apply { whenever(this.time).thenReturn(time) }
+
+       @Test
+       @Suppress("UNCHECKED_CAST")
+       fun `posts are paginated properly`() {
+               webInterface.core.preferences.postsPerPage = 2
+               val posts = listOf(mock<Post>().withTime(2000), mock<Post>().withTime(3000), mock<Post>().withTime(1000))
+               whenever(webInterface.getNewPosts(currentSone)).thenReturn(posts)
+               verifyNoRedirect {
+                       assertThat((templateContext["pagination"] as Pagination<Post>).items, contains(posts[1], posts[0]))
+               }
+       }
+
+       @Test
+       @Suppress("UNCHECKED_CAST")
+       fun `posts are paginated properly on second page`() {
+               webInterface.core.preferences.postsPerPage = 2
+               addHttpRequestParameter("page", "1")
+               val posts = listOf(mock<Post>().withTime(2000), mock<Post>().withTime(3000), mock<Post>().withTime(1000))
+               whenever(webInterface.getNewPosts(currentSone)).thenReturn(posts)
+               verifyNoRedirect {
+                       assertThat((templateContext["pagination"] as Pagination<Post>).items, contains(posts[2]))
+               }
+       }
+
+       @Test
+       fun `page can be created by dependency injection`() {
+           assertThat(baseInjector.getInstance<NewPage>(), notNullValue())
+       }
+
 }