projects
/
Sone.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Add test for DI constructability of UntrustPage
[Sone.git]
/
src
/
test
/
kotlin
/
net
/
pterodactylus
/
sone
/
web
/
pages
/
LockSonePageTest.kt
diff --git
a/src/test/kotlin/net/pterodactylus/sone/web/pages/LockSonePageTest.kt
b/src/test/kotlin/net/pterodactylus/sone/web/pages/LockSonePageTest.kt
index
25678ca
..
f8ad87a
100644
(file)
--- a/
src/test/kotlin/net/pterodactylus/sone/web/pages/LockSonePageTest.kt
+++ b/
src/test/kotlin/net/pterodactylus/sone/web/pages/LockSonePageTest.kt
@@
-1,9
+1,13
@@
package net.pterodactylus.sone.web.pages
import net.pterodactylus.sone.data.Sone
package net.pterodactylus.sone.web.pages
import net.pterodactylus.sone.data.Sone
+import net.pterodactylus.sone.test.getInstance
import net.pterodactylus.sone.test.mock
import net.pterodactylus.sone.test.mock
+import net.pterodactylus.sone.web.baseInjector
+import net.pterodactylus.util.web.Method.POST
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
+import org.hamcrest.Matchers.notNullValue
import org.junit.Test
import org.mockito.ArgumentMatchers.any
import org.mockito.Mockito.never
import org.junit.Test
import org.mockito.ArgumentMatchers.any
import org.mockito.Mockito.never
@@
-12,11
+16,7
@@
import org.mockito.Mockito.verify
/**
* Unit test for [LockSonePage].
*/
/**
* Unit test for [LockSonePage].
*/
-class LockSonePageTest : WebPageTest() {
-
- private val page = LockSonePage(template, webInterface)
-
- override fun getPage() = page
+class LockSonePageTest: WebPageTest(::LockSonePage) {
@Test
fun `page returns correct path`() {
@Test
fun `page returns correct path`() {
@@
-24,7
+24,7
@@
class LockSonePageTest : WebPageTest() {
}
@Test
}
@Test
- fun `page
requires
login`() {
+ fun `page
does not require
login`() {
assertThat(page.requiresLogin(), equalTo(false))
}
assertThat(page.requiresLogin(), equalTo(false))
}
@@
-36,6
+36,7
@@
class LockSonePageTest : WebPageTest() {
@Test
fun `locking an invalid local sone redirects to return page`() {
@Test
fun `locking an invalid local sone redirects to return page`() {
+ setMethod(POST)
addHttpRequestPart("returnPage", "return.html")
verifyRedirect("return.html") {
verify(core, never()).lockSone(any<Sone>())
addHttpRequestPart("returnPage", "return.html")
verifyRedirect("return.html") {
verify(core, never()).lockSone(any<Sone>())
@@
-44,6
+45,7
@@
class LockSonePageTest : WebPageTest() {
@Test
fun `locking an valid local sone locks the sone and redirects to return page`() {
@Test
fun `locking an valid local sone locks the sone and redirects to return page`() {
+ setMethod(POST)
addHttpRequestPart("sone", "sone-id")
val sone = mock<Sone>()
addLocalSone("sone-id", sone)
addHttpRequestPart("sone", "sone-id")
val sone = mock<Sone>()
addLocalSone("sone-id", sone)
@@
-53,4
+55,9
@@
class LockSonePageTest : WebPageTest() {
}
}
}
}
+ @Test
+ fun `page can be created by dependency injection`() {
+ assertThat(baseInjector.getInstance<LockSonePage>(), notNullValue())
+ }
+
}
}