X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2FDismissNotificationPageTest.kt;h=58e1eed76beadaa225bead2324c4efb7952fa139;hp=a794f4c21a150794a908addbb379902d25ebc022;hb=a9c1c1770651720ec3b271464a4b490ea4a1334f;hpb=7eac5c60bdb241d7b98e548cf974d41e22af1b5c diff --git a/src/test/kotlin/net/pterodactylus/sone/web/DismissNotificationPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/DismissNotificationPageTest.kt index a794f4c..58e1eed 100644 --- a/src/test/kotlin/net/pterodactylus/sone/web/DismissNotificationPageTest.kt +++ b/src/test/kotlin/net/pterodactylus/sone/web/DismissNotificationPageTest.kt @@ -2,13 +2,11 @@ package net.pterodactylus.sone.web import net.pterodactylus.sone.test.mock import net.pterodactylus.sone.test.whenever -import net.pterodactylus.sone.web.WebTestUtils.redirectsTo import net.pterodactylus.util.notify.Notification import net.pterodactylus.util.web.Method.GET import org.junit.Test import org.mockito.Mockito.never import org.mockito.Mockito.verify -import kotlin.test.fail /** * Unit test for [DismissNotificationPage]. @@ -18,12 +16,13 @@ class DismissNotificationPageTest : WebPageTest() { private val page = DismissNotificationPage(template, webInterface) private val notification = mock() + override fun getPage() = page + @Test fun `get request with invalid notification ID redirects to return page`() { request("", GET) addHttpRequestParameter("returnPage", "return.html") - expectedException.expect(redirectsTo("return.html")) - page.handleRequest(freenetRequest, templateContext) + verifyRedirect("return.html") } @Test @@ -32,11 +31,7 @@ class DismissNotificationPageTest : WebPageTest() { addNotification("notification-id", notification) addHttpRequestParameter("notification", "notification-id") addHttpRequestParameter("returnPage", "return.html") - expectedException.expect(redirectsTo("return.html")) - try { - page.handleRequest(freenetRequest, templateContext) - fail() - } finally { + verifyRedirect("return.html") { verify(notification, never()).dismiss() } } @@ -48,11 +43,7 @@ class DismissNotificationPageTest : WebPageTest() { addNotification("notification-id", notification) addHttpRequestParameter("notification", "notification-id") addHttpRequestParameter("returnPage", "return.html") - expectedException.expect(redirectsTo("return.html")) - try { - page.handleRequest(freenetRequest, templateContext) - fail() - } finally { + verifyRedirect("return.html") { verify(notification).dismiss() } }