🎨 Fix formatting

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

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/CreateAlbumPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/CreateAlbumPageTest.kt
index 9851530..224791f 100644 (file)
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/CreateAlbumPageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/CreateAlbumPageTest.kt
@@ -1,28 +1,21 @@
 package net.pterodactylus.sone.web.pages
 
-import net.pterodactylus.sone.data.Album
-import net.pterodactylus.sone.data.Album.Modifier
-import net.pterodactylus.sone.data.Album.Modifier.AlbumTitleMustNotBeEmpty
-import net.pterodactylus.sone.test.deepMock
-import net.pterodactylus.sone.test.selfMock
-import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.CreateAlbumPage
-import net.pterodactylus.util.web.Method.POST
-import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers.equalTo
-import org.junit.Before
-import org.junit.Test
-import org.mockito.Mockito.verify
+import net.pterodactylus.sone.data.*
+import net.pterodactylus.sone.data.Album.*
+import net.pterodactylus.sone.data.Album.Modifier.*
+import net.pterodactylus.sone.test.*
+import net.pterodactylus.sone.web.*
+import net.pterodactylus.sone.web.page.*
+import net.pterodactylus.util.web.Method.*
+import org.hamcrest.MatcherAssert.*
+import org.hamcrest.Matchers.*
+import org.junit.*
+import org.mockito.Mockito.*
 
 /**
  * Unit test for [CreateAlbumPage].
  */
-class CreateAlbumPageTest: WebPageTest() {
-
-       private val page = CreateAlbumPage(template, webInterface)
-
-       override fun getPage() = page
+class CreateAlbumPageTest : WebPageTest(::CreateAlbumPage) {
 
        private val parentAlbum = createAlbum("parent-id")
        private val newAlbum = createAlbum("album-id")
@@ -45,7 +38,7 @@ class CreateAlbumPageTest: WebPageTest() {
 
        @Test
        fun `missing name results in attribute being set in template context`() {
-               request("", POST)
+               setMethod(POST)
                page.processTemplate(freenetRequest, templateContext)
                assertThat(templateContext["nameMissing"], equalTo<Any>(true))
        }
@@ -60,7 +53,7 @@ class CreateAlbumPageTest: WebPageTest() {
 
        @Test
        fun `title and description are set correctly on the album`() {
-               request("", POST)
+               setMethod(POST)
                addAlbum("parent-id", parentAlbum)
                addHttpRequestPart("name", "new name")
                addHttpRequestPart("description", "new description")
@@ -76,7 +69,7 @@ class CreateAlbumPageTest: WebPageTest() {
 
        @Test
        fun `root album is used if no parent is specified`() {
-               request("", POST)
+               setMethod(POST)
                addHttpRequestPart("name", "new name")
                addHttpRequestPart("description", "new description")
                verifyRedirect("imageBrowser.html?album=album-id")
@@ -84,7 +77,7 @@ class CreateAlbumPageTest: WebPageTest() {
 
        @Test
        fun `empty album title redirects to error page`() {
-               request("", POST)
+               setMethod(POST)
                whenever(newAlbum.modify().update()).thenThrow(AlbumTitleMustNotBeEmpty::class.java)
                addHttpRequestPart("name", "new name")
                addHttpRequestPart("description", "new description")
@@ -93,7 +86,7 @@ class CreateAlbumPageTest: WebPageTest() {
 
        @Test
        fun `album description is filtered`() {
-               request("", POST)
+               setMethod(POST)
                addHttpRequestPart("name", "new name")
                addHttpRequestPart("description", "new http://localhost:12345/KSK@foo description")
                addHttpRequestHeader("Host", "localhost:12345")
@@ -102,4 +95,14 @@ class CreateAlbumPageTest: WebPageTest() {
                }
        }
 
+       @Test
+       fun `page can be created by dependency injection`() {
+               assertThat(baseInjector.getInstance<CreateAlbumPage>(), notNullValue())
+       }
+
+       @Test
+       fun `page is annotated with correct template path`() {
+               assertThat(page.templatePath, equalTo("/templates/createAlbum.html"))
+       }
+
 }