projects
/
Sone.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Add more tests for options page
[Sone.git]
/
src
/
test
/
kotlin
/
net
/
pterodactylus
/
sone
/
web
/
pages
/
DismissNotificationPageTest.kt
diff --git
a/src/test/kotlin/net/pterodactylus/sone/web/pages/DismissNotificationPageTest.kt
b/src/test/kotlin/net/pterodactylus/sone/web/pages/DismissNotificationPageTest.kt
index
b6524a8
..
5ce0dcb
100644
(file)
--- a/
src/test/kotlin/net/pterodactylus/sone/web/pages/DismissNotificationPageTest.kt
+++ b/
src/test/kotlin/net/pterodactylus/sone/web/pages/DismissNotificationPageTest.kt
@@
-2,10
+2,8
@@
package net.pterodactylus.sone.web.pages
import net.pterodactylus.sone.test.mock
import net.pterodactylus.sone.test.whenever
import net.pterodactylus.sone.test.mock
import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.sone.web.pages.DismissNotificationPage
import net.pterodactylus.util.notify.Notification
import net.pterodactylus.util.notify.Notification
-import net.pterodactylus.util.web.Method.
GE
T
+import net.pterodactylus.util.web.Method.
POS
T
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
import org.junit.Test
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.equalTo
import org.junit.Test
@@
-40,29
+38,29
@@
class DismissNotificationPageTest: WebPageTest() {
@Test
fun `get request with invalid notification ID redirects to return page`() {
@Test
fun `get request with invalid notification ID redirects to return page`() {
-
request("", GE
T)
- addHttpRequestPar
ameter
("returnPage", "return.html")
+
setMethod(POS
T)
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html")
}
@Test
fun `get request with non-dismissible notification never dismisses the notification but redirects to return page`() {
verifyRedirect("return.html")
}
@Test
fun `get request with non-dismissible notification never dismisses the notification but redirects to return page`() {
-
request("", GE
T)
+
setMethod(POS
T)
addNotification("notification-id", notification)
addNotification("notification-id", notification)
- addHttpRequestPar
ameter
("notification", "notification-id")
- addHttpRequestPar
ameter
("returnPage", "return.html")
+ addHttpRequestPar
t
("notification", "notification-id")
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html") {
verify(notification, never()).dismiss()
}
}
@Test
verifyRedirect("return.html") {
verify(notification, never()).dismiss()
}
}
@Test
- fun `
ge
t request with dismissible notification dismisses the notification and redirects to return page`() {
-
request("", GE
T)
+ fun `
pos
t request with dismissible notification dismisses the notification and redirects to return page`() {
+
setMethod(POS
T)
whenever(notification.isDismissable).thenReturn(true)
addNotification("notification-id", notification)
whenever(notification.isDismissable).thenReturn(true)
addNotification("notification-id", notification)
- addHttpRequestPar
ameter
("notification", "notification-id")
- addHttpRequestPar
ameter
("returnPage", "return.html")
+ addHttpRequestPar
t
("notification", "notification-id")
+ addHttpRequestPar
t
("returnPage", "return.html")
verifyRedirect("return.html") {
verify(notification).dismiss()
}
verifyRedirect("return.html") {
verify(notification).dismiss()
}