X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FUnbookmarkPageTest.kt;h=ebd06f4650d745c8570c2ac368ea2f36ed7db70f;hp=b18211fb104829209bb1fd54390552dd11010b5d;hb=3db634e425d3482bcf00dfd82e4b0d46785d5959;hpb=de7568a82eb4150bf6d2b0553841b7b69f84c968

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnbookmarkPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnbookmarkPageTest.kt
index b18211f..ebd06f4 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/UnbookmarkPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/UnbookmarkPageTest.kt
@@ -1,27 +1,46 @@
 package net.pterodactylus.sone.web.pages
 
 import net.pterodactylus.sone.data.Post
+import net.pterodactylus.sone.test.capture
+import net.pterodactylus.sone.test.getInstance
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.UnbookmarkPage
+import net.pterodactylus.sone.web.baseInjector
 import net.pterodactylus.util.web.Method.POST
+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 org.mockito.Mockito.any
 import org.mockito.Mockito.never
+import org.mockito.Mockito.times
 import org.mockito.Mockito.verify
 
 /**
  * Unit test for [UnbookmarkPage].
  */
-class UnbookmarkPageTest : WebPageTest() {
+class UnbookmarkPageTest: WebPageTest(::UnbookmarkPage) {
 
-	private val page = UnbookmarkPage(template, webInterface)
+	@Test
+	fun `page returns correct path`() {
+		assertThat(page.path, equalTo("unbookmark.html"))
+	}
+
+	@Test
+	fun `page does not require login`() {
+		assertThat(page.requiresLogin(), equalTo(false))
+	}
 
-	override fun getPage() = page
+	@Test
+	fun `page returns correct title`() {
+		addTranslation("Page.Unbookmark.Title", "unbookmark page title")
+		assertThat(page.getPageTitle(soneRequest), equalTo("unbookmark page title"))
+	}
 
 	@Test
 	fun `get request does not redirect`() {
-		page.handleRequest(freenetRequest, templateContext)
+		verifyNoRedirect { }
 	}
 
 	@Test
@@ -33,18 +52,17 @@ class UnbookmarkPageTest : WebPageTest() {
 		val notLoadedPost2 = mock<Post>()
 		whenever(core.bookmarkedPosts).thenReturn(setOf(loadedPost1, loadedPost2, notLoadedPost1, notLoadedPost2))
 		verifyRedirect("bookmarks.html") {
-			verify(core).unbookmarkPost(notLoadedPost1)
-			verify(core).unbookmarkPost(notLoadedPost2)
-			verify(core, never()).unbookmarkPost(loadedPost1)
-			verify(core, never()).unbookmarkPost(loadedPost2)
+			val postCaptor = capture<Post>()
+			verify(core, times(2)).unbookmarkPost(postCaptor.capture())
+			assertThat(postCaptor.allValues, contains(notLoadedPost1, notLoadedPost2))
 		}
 	}
 
 	@Test
 	fun `post request does not unbookmark not-present post but redirects to return page`() {
-		request("", POST)
-		addHttpRequestParameter("post", "post-id")
-		addHttpRequestParameter("returnPage", "return.html")
+		setMethod(POST)
+		addHttpRequestPart("post", "post-id")
+		addHttpRequestPart("returnPage", "return.html")
 		verifyRedirect("return.html") {
 			verify(core, never()).unbookmarkPost(any())
 		}
@@ -52,9 +70,9 @@ class UnbookmarkPageTest : WebPageTest() {
 
 	@Test
 	fun `post request unbookmarks present post and redirects to return page`() {
-		request("", POST)
-		addHttpRequestParameter("post", "post-id")
-		addHttpRequestParameter("returnPage", "return.html")
+		setMethod(POST)
+		addHttpRequestPart("post", "post-id")
+		addHttpRequestPart("returnPage", "return.html")
 		val post = mock<Post>().apply { whenever(isLoaded).thenReturn(true) }
 		addPost("post-id", post)
 		verifyRedirect("return.html") {
@@ -62,4 +80,9 @@ class UnbookmarkPageTest : WebPageTest() {
 		}
 	}
 
+	@Test
+	fun `page can be created by dependency injection`() {
+	    assertThat(baseInjector.getInstance<UnbookmarkPage>(), notNullValue())
+	}
+
 }