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 SearchPage
[Sone.git]
/
src
/
test
/
kotlin
/
net
/
pterodactylus
/
sone
/
web
/
pages
/
FollowSonePageTest.kt
diff --git
a/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
b/src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
index
f595b4a
..
906ebea
100644
(file)
--- a/
src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
+++ b/
src/test/kotlin/net/pterodactylus/sone/web/pages/FollowSonePageTest.kt
@@
-1,11
+1,14
@@
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.pages.FollowSonePage
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.util.web.Method.GET
+import net.pterodactylus.sone.test.whenever
+import net.pterodactylus.sone.web.baseInjector
import net.pterodactylus.util.web.Method.POST
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.mockito.ArgumentMatchers
import org.mockito.ArgumentMatchers.any
import org.junit.Test
import org.mockito.ArgumentMatchers
import org.mockito.ArgumentMatchers.any
@@
-16,25
+19,36
@@
import org.mockito.Mockito.verify
/**
* Unit test for [FollowSonePage].
*/
/**
* Unit test for [FollowSonePage].
*/
-class FollowSonePageTest
: WebPageTest(
) {
+class FollowSonePageTest
: WebPageTest(::FollowSonePage
) {
- private val page = FollowSonePage(template, webInterface)
+ @Test
+ fun `page returns correct path`() {
+ assertThat(page.path, equalTo("followSone.html"))
+ }
+
+ @Test
+ fun `page requires login`() {
+ assertThat(page.requiresLogin(), equalTo(true))
+ }
- override fun getPage() = page
+ @Test
+ fun `page returns correct title`() {
+ whenever(l10n.getString("Page.FollowSone.Title")).thenReturn("follow sone page title")
+ assertThat(page.getPageTitle(freenetRequest), equalTo("follow sone page title"))
+ }
@Test
fun `get request does not redirect`() {
@Test
fun `get request does not redirect`() {
- request("", GET)
- page.handleRequest(freenetRequest, templateContext)
+ page.processTemplate(freenetRequest, templateContext)
}
@Test
fun `a single sone can be followed`() {
}
@Test
fun `a single sone can be followed`() {
-
request("",
POST)
+
setMethod(
POST)
val sone = mock<Sone>()
addSone("sone-id", sone)
val sone = mock<Sone>()
addSone("sone-id", sone)
- addHttpRequestPar
ameter
("sone", "sone-id")
- addHttpRequestPar
ameter
("returnPage", "return.html")
+ addHttpRequestPar
t
("sone", "sone-id")
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html") {
verify(core).followSone(currentSone, "sone-id")
verify(core).markSoneKnown(sone)
verifyRedirect("return.html") {
verify(core).followSone(currentSone, "sone-id")
verify(core).markSoneKnown(sone)
@@
-43,13
+57,13
@@
class FollowSonePageTest : WebPageTest() {
@Test
fun `multiple sones can be followed`() {
@Test
fun `multiple sones can be followed`() {
-
request("",
POST)
+
setMethod(
POST)
val firstSone = mock<Sone>()
addSone("sone-id1", firstSone)
val secondSone = mock<Sone>()
addSone("sone-id2", secondSone)
val firstSone = mock<Sone>()
addSone("sone-id1", firstSone)
val secondSone = mock<Sone>()
addSone("sone-id2", secondSone)
- addHttpRequestPar
ameter
("sone", "sone-id1,sone-id2")
- addHttpRequestPar
ameter
("returnPage", "return.html")
+ addHttpRequestPar
t
("sone", "sone-id1,sone-id2")
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html") {
verify(core).followSone(currentSone, "sone-id1")
verify(core).followSone(currentSone, "sone-id2")
verifyRedirect("return.html") {
verify(core).followSone(currentSone, "sone-id1")
verify(core).followSone(currentSone, "sone-id2")
@@
-60,13
+74,18
@@
class FollowSonePageTest : WebPageTest() {
@Test
fun `a non-existing sone is not followed`() {
@Test
fun `a non-existing sone is not followed`() {
-
request("",
POST)
- addHttpRequestPar
ameter
("sone", "sone-id")
- addHttpRequestPar
ameter
("returnPage", "return.html")
+
setMethod(
POST)
+ addHttpRequestPar
t
("sone", "sone-id")
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html") {
verify(core, never()).followSone(ArgumentMatchers.eq(currentSone), anyString())
verify(core, never()).markSoneKnown(any<Sone>())
}
}
verifyRedirect("return.html") {
verify(core, never()).followSone(ArgumentMatchers.eq(currentSone), anyString())
verify(core, never()).markSoneKnown(any<Sone>())
}
}
+ @Test
+ fun `page can be created by dependency injection`() {
+ assertThat(baseInjector.getInstance<FollowSonePage>(), notNullValue())
+ }
+
}
}