projects
/
Sone.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Optimize some imports
[Sone.git]
/
src
/
test
/
kotlin
/
net
/
pterodactylus
/
sone
/
web
/
pages
/
DeleteSonePageTest.kt
diff --git
a/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteSonePageTest.kt
b/src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteSonePageTest.kt
index
b94c3fa
..
7e237d6
100644
(file)
--- a/
src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteSonePageTest.kt
+++ b/
src/test/kotlin/net/pterodactylus/sone/web/pages/DeleteSonePageTest.kt
@@
-1,15
+1,13
@@
package net.pterodactylus.sone.web.pages
package net.pterodactylus.sone.web.pages
+import net.pterodactylus.sone.test.getInstance
import net.pterodactylus.sone.test.whenever
import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.DeleteSonePage
-import net.pterodactylus.util.web.Method.GET
+import net.pterodactylus.sone.web.baseInjector
import net.pterodactylus.util.web.Method.POST
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
import net.pterodactylus.util.web.Method.POST
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
+import org.hamcrest.Matchers.notNullValue
import org.junit.Test
import org.junit.Test
-import org.mockito.ArgumentMatchers
-import org.mockito.ArgumentMatchers.any
import org.mockito.Mockito.any
import org.mockito.Mockito.never
import org.mockito.Mockito.verify
import org.mockito.Mockito.any
import org.mockito.Mockito.never
import org.mockito.Mockito.verify
@@
-17,11
+15,7
@@
import org.mockito.Mockito.verify
/**
* Unit test for [DeleteSonePage].
*/
/**
* Unit test for [DeleteSonePage].
*/
-class DeleteSonePageTest : WebPageTest() {
-
- private val page = DeleteSonePage(template, webInterface)
-
- override fun getPage() = page
+class DeleteSonePageTest: WebPageTest(::DeleteSonePage) {
@Test
fun `page returns correct path`() {
@Test
fun `page returns correct path`() {
@@
-41,13
+35,12
@@
class DeleteSonePageTest : WebPageTest() {
@Test
fun `get request does not redirect`() {
@Test
fun `get request does not redirect`() {
- request("", GET)
page.processTemplate(freenetRequest, templateContext)
}
@Test
fun `post request without delete confirmation redirects to index`() {
page.processTemplate(freenetRequest, templateContext)
}
@Test
fun `post request without delete confirmation redirects to index`() {
-
request("",
POST)
+
setMethod(
POST)
verifyRedirect("index.html") {
verify(core, never()).deleteSone(any())
}
verifyRedirect("index.html") {
verify(core, never()).deleteSone(any())
}
@@
-55,11
+48,16
@@
class DeleteSonePageTest : WebPageTest() {
@Test
fun `post request with delete confirmation deletes sone and redirects to index`() {
@Test
fun `post request with delete confirmation deletes sone and redirects to index`() {
-
request("",
POST)
- addHttpRequestPar
ameter
("deleteSone", "true")
+
setMethod(
POST)
+ addHttpRequestPar
t
("deleteSone", "true")
verifyRedirect("index.html") {
verify(core).deleteSone(currentSone)
}
}
verifyRedirect("index.html") {
verify(core).deleteSone(currentSone)
}
}
+ @Test
+ fun `page can be created by dependency injection`() {
+ assertThat(baseInjector.getInstance<DeleteSonePage>(), notNullValue())
+ }
+
}
}