✨ Annotate LoginPage with MenuName

[Sone.git] / src / test / kotlin / net / pterodactylus / sone / web / pages / LoginPageTest.kt

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/LoginPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/LoginPageTest.kt
index 0da820c..794e68b 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/LoginPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/LoginPageTest.kt
@@ -3,17 +3,18 @@ package net.pterodactylus.sone.web.pages
 import net.pterodactylus.sone.data.Sone
 import net.pterodactylus.sone.freenet.wot.Identity
 import net.pterodactylus.sone.freenet.wot.OwnIdentity
+import net.pterodactylus.sone.test.getInstance
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.thenReturnMock
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.LoginPage
-import net.pterodactylus.util.web.Method.GET
+import net.pterodactylus.sone.web.baseInjector
+import net.pterodactylus.sone.web.page.*
 import net.pterodactylus.util.web.Method.POST
 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.hamcrest.Matchers.nullValue
 import org.junit.Before
 import org.junit.Test
@@ -22,14 +23,10 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [LoginPage].
  */
-class LoginPageTest : WebPageTest() {
-
-       private val page = LoginPage(template, webInterface)
+class LoginPageTest: WebPageTest(::LoginPage) {
 
        private val sones = listOf(createSone("Sone", "Test"), createSone("Test"), createSone("Sone"))
 
-       override fun getPage() = page
-
        private fun createSone(vararg contexts: String) = mock<Sone>().apply {
                whenever(id).thenReturn(hashCode().toString())
                val identity = mock<OwnIdentity>().apply {
@@ -63,7 +60,6 @@ class LoginPageTest : WebPageTest() {
        @Test
        @Suppress("UNCHECKED_CAST")
        fun `get request stores sones in template context`() {
-               request("", GET)
                page.processTemplate(freenetRequest, templateContext)
                assertThat(templateContext["sones"] as Iterable<Sone>, containsInAnyOrder(sones[0], sones[1], sones[2]))
        }
@@ -71,7 +67,6 @@ class LoginPageTest : WebPageTest() {
        @Test
        @Suppress("UNCHECKED_CAST")
        fun `get request stores identities without sones in template context`() {
-               request("", GET)
                page.processTemplate(freenetRequest, templateContext)
                assertThat(templateContext["identitiesWithoutSone"] as Iterable<Identity>, contains(sones[1].identity))
        }
@@ -79,7 +74,7 @@ class LoginPageTest : WebPageTest() {
        @Test
        @Suppress("UNCHECKED_CAST")
        fun `post request with invalid sone sets sones and identities without sone in template context`() {
-               request("", POST)
+               setMethod(POST)
                page.processTemplate(freenetRequest, templateContext)
                assertThat(templateContext["sones"] as Iterable<Sone>, containsInAnyOrder(sones[0], sones[1], sones[2]))
                assertThat(templateContext["identitiesWithoutSone"] as Iterable<Identity>, contains(sones[1].identity))
@@ -87,8 +82,8 @@ class LoginPageTest : WebPageTest() {
 
        @Test
        fun `post request with valid sone logs in the sone and redirects to index page`() {
-               request("", POST)
-               addHttpRequestParameter("sone-id", "sone2")
+               setMethod(POST)
+               addHttpRequestPart("sone-id", "sone2")
                verifyRedirect("index.html") {
                        verify(webInterface).setCurrentSone(toadletContext, sones[1])
                }
@@ -96,8 +91,8 @@ class LoginPageTest : WebPageTest() {
 
        @Test
        fun `post request with valid sone and target redirects to target page`() {
-               request("", POST)
-               addHttpRequestParameter("sone-id", "sone2")
+               setMethod(POST)
+               addHttpRequestPart("sone-id", "sone2")
                addHttpRequestParameter("target", "foo.html")
                verifyRedirect("foo.html") {
                        verify(webInterface).setCurrentSone(toadletContext, sones[1])
@@ -117,7 +112,7 @@ class LoginPageTest : WebPageTest() {
 
        @Test
        fun `page is not enabled if full access required and request is not full access`() {
-               core.preferences.isRequireFullAccess = true
+               core.preferences.newRequireFullAccess = true
                assertThat(page.isEnabled(toadletContext), equalTo(false))
        }
 
@@ -134,7 +129,7 @@ class LoginPageTest : WebPageTest() {
 
        @Test
        fun `page is enabled if full access required and request is full access and there is no current sone`() {
-               core.preferences.isRequireFullAccess = true
+               core.preferences.newRequireFullAccess = true
                unsetCurrentSone()
                whenever(toadletContext.isAllowedFullAccess).thenReturn(true)
                assertThat(page.isEnabled(toadletContext), equalTo(true))
@@ -142,9 +137,19 @@ class LoginPageTest : WebPageTest() {
 
        @Test
        fun `page is not enabled if full access required and request is full access but there is a current sone`() {
-               core.preferences.isRequireFullAccess = true
+               core.preferences.newRequireFullAccess = true
                whenever(toadletContext.isAllowedFullAccess).thenReturn(true)
                assertThat(page.isEnabled(toadletContext), equalTo(false))
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+           assertThat(baseInjector.getInstance<LoginPage>(), notNullValue())
+       }
+
+       @Test
+       fun `page is annotated with correct menuname`() {
+           assertThat(page.menuName, equalTo("Login"))
+       }
+
 }