X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2Fpages%2FEditProfilePageTest.kt;h=b4e657044c7468cfa5bc22e117169ae47beeae55;hb=da022230e20fcc8e133f059ab9ffdc60eb50d202;hp=274f0fefdd1c66c577f46fe766ff7cf0ba0fa4dd;hpb=05fb821e72072bde52f383bdc5a988da67f66d0c;p=Sone.git

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/pages/EditProfilePageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/pages/EditProfilePageTest.kt
index 274f0fe..b4e6570 100644
--- a/src/test/kotlin/net/pterodactylus/sone/web/pages/EditProfilePageTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/web/pages/EditProfilePageTest.kt
@@ -4,9 +4,6 @@ import net.pterodactylus.sone.data.Image
 import net.pterodactylus.sone.data.Profile
 import net.pterodactylus.sone.test.mock
 import net.pterodactylus.sone.test.whenever
-import net.pterodactylus.sone.web.pages.EditProfilePage
-import net.pterodactylus.sone.web.pages.WebPageTest
-import net.pterodactylus.util.web.Method.GET
 import net.pterodactylus.util.web.Method.POST
 import org.hamcrest.MatcherAssert.assertThat
 import org.hamcrest.Matchers.contains
@@ -20,16 +17,12 @@ import org.mockito.Mockito.verify
 /**
  * Unit test for [EditProfilePage].
  */
-class EditProfilePageTest : WebPageTest() {
-
-	private val page = EditProfilePage(template, webInterface)
+class EditProfilePageTest: WebPageTest(::EditProfilePage) {
 
 	private val profile = Profile(currentSone)
 	private val firstField = profile.addField("First Field")
 	private val secondField = profile.addField("Second Field")
 
-	override fun getPage() = page
-
 	@Before
 	fun setupProfile() {
 		val avatar = mock<Image>()
@@ -63,7 +56,6 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `get request stores fields of current sone’s profile in template context`() {
-		request("", GET)
 		page.processTemplate(freenetRequest, templateContext)
 		assertThat(templateContext["firstName"], equalTo<Any>("First"))
 		assertThat(templateContext["middleName"], equalTo<Any>("Middle"))
@@ -77,7 +69,7 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `post request without any command stores fields of current sone’s profile in template context`() {
-		request("", POST)
+		setMethod(POST)
 		page.processTemplate(freenetRequest, templateContext)
 		assertThat(templateContext["firstName"], equalTo<Any>("First"))
 		assertThat(templateContext["middleName"], equalTo<Any>("Middle"))
@@ -90,10 +82,11 @@ class EditProfilePageTest : WebPageTest() {
 	}
 
 	private fun <T> verifySingleFieldCanBeChanged(fieldName: String, newValue: T, expectedValue: T = newValue, fieldAccessor: () -> T) {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("save-profile", "true")
 		addHttpRequestPart(fieldName, newValue.toString())
 		verifyRedirect("editProfile.html") {
+			verify(currentSone).profile = profile
 			verify(core).touchConfiguration()
 			assertThat(fieldAccessor(), equalTo(expectedValue))
 		}
@@ -155,7 +148,7 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `adding a field with a duplicate name sets error in template context`() {
-		request("", POST)
+		setMethod(POST)
 		profile.addField("new-field")
 		addHttpRequestPart("add-field", "true")
 		addHttpRequestPart("field-name", "new-field")
@@ -167,7 +160,7 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `adding a field with a new name sets adds field to profile and redirects to profile edit page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("add-field", "true")
 		addHttpRequestPart("field-name", "new-field")
 		verifyRedirect("editProfile.html#profile-fields") {
@@ -179,14 +172,14 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `deleting a field redirects to delete field page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("delete-field-${firstField.id}", "true")
 		verifyRedirect("deleteProfileField.html?field=${firstField.id}")
 	}
 
 	@Test
 	fun `moving a field up moves the field up and redirects to the edit profile page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("move-up-field-${secondField.id}", "true")
 		verifyRedirect("editProfile.html#profile-fields") {
 			assertThat(profile.fields, contains(secondField, firstField))
@@ -196,14 +189,14 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `moving an invalid field up does not redirect`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("move-up-field-foo", "true")
 		page.processTemplate(freenetRequest, templateContext)
 	}
 
 	@Test
 	fun `moving a field down moves the field down and redirects to the edit profile page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("move-down-field-${firstField.id}", "true")
 		verifyRedirect("editProfile.html#profile-fields") {
 			assertThat(profile.fields, contains(secondField, firstField))
@@ -213,14 +206,14 @@ class EditProfilePageTest : WebPageTest() {
 
 	@Test
 	fun `moving an invalid field down does not redirect`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("move-down-field-foo", "true")
 		page.processTemplate(freenetRequest, templateContext)
 	}
 
 	@Test
 	fun `editing a field redirects to the edit profile page`() {
-		request("", POST)
+		setMethod(POST)
 		addHttpRequestPart("edit-field-${firstField.id}", "true")
 		verifyRedirect("editProfileField.html?field=${firstField.id}")
 	}