Add test for distrust ajax page
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Tue, 5 Sep 2017 09:28:27 +0000 (11:28 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Tue, 5 Sep 2017 09:28:27 +0000 (11:28 +0200)
src/test/kotlin/net/pterodactylus/sone/web/ajax/DistrustAjaxPageTest.kt [new file with mode: 0644]
src/test/kotlin/net/pterodactylus/sone/web/ajax/JsonPageTest.kt

diff --git a/src/test/kotlin/net/pterodactylus/sone/web/ajax/DistrustAjaxPageTest.kt b/src/test/kotlin/net/pterodactylus/sone/web/ajax/DistrustAjaxPageTest.kt
new file mode 100644 (file)
index 0000000..770d068
--- /dev/null
@@ -0,0 +1,47 @@
+package net.pterodactylus.sone.web.ajax
+
+import net.pterodactylus.sone.data.Sone
+import net.pterodactylus.sone.test.mock
+import org.hamcrest.MatcherAssert.assertThat
+import org.hamcrest.Matchers.equalTo
+import org.junit.Test
+import org.mockito.Mockito.verify
+
+/**
+ * Unit test for [DistrustAjaxPage].
+ */
+class DistrustAjaxPageTest : JsonPageTest("distrustSone.ajax", pageSupplier = ::DistrustAjaxPage) {
+
+       @Test
+       fun `request with missing sone results in invalid-sone-id`() {
+               assertThat(json.isSuccess, equalTo(false))
+               assertThat(json.error, equalTo("invalid-sone-id"))
+       }
+
+       @Test
+       fun `request with invalid sone results in invalid-sone-id`() {
+               addRequestParameter("sone", "invalid-sone")
+               assertThat(json.isSuccess, equalTo(false))
+               assertThat(json.error, equalTo("invalid-sone-id"))
+       }
+
+       @Test
+       fun `request with valid sone results in distrusted sone`() {
+               val sone = mock<Sone>()
+               addSone(sone, "sone-id")
+               addRequestParameter("sone", "sone-id")
+               assertThat(json.isSuccess, equalTo(true))
+               verify(core).distrustSone(currentSone, sone)
+       }
+
+       @Test
+       fun `request with valid sone results in correct trust value being sent back`() {
+               core.preferences.negativeTrust = -33
+               val sone = mock<Sone>()
+               addSone(sone, "sone-id")
+               addRequestParameter("sone", "sone-id")
+               assertThat(json.isSuccess, equalTo(true))
+               assertThat(json["trustValue"].asInt(), equalTo(-33))
+       }
+
+}
index f3558fa..cb5ee5a 100644 (file)
@@ -1,11 +1,13 @@
 package net.pterodactylus.sone.web.ajax
 
+import com.google.common.eventbus.EventBus
 import freenet.clients.http.ToadletContext
 import freenet.support.SimpleReadOnlyArrayBucket
 import freenet.support.api.HTTPRequest
 import net.pterodactylus.sone.core.Core
 import net.pterodactylus.sone.core.ElementLoader
 import net.pterodactylus.sone.core.LinkedElement
+import net.pterodactylus.sone.core.Preferences
 import net.pterodactylus.sone.data.Post
 import net.pterodactylus.sone.data.PostReply
 import net.pterodactylus.sone.data.Profile
@@ -44,6 +46,8 @@ abstract class JsonPageTest(
 
        protected val webInterface = mock<WebInterface>()
        protected val core = mock<Core>()
+       protected val eventBus = mock<EventBus>()
+       protected val preferences = Preferences(eventBus)
        protected val elementLoader = mock<ElementLoader>()
        protected open val page: JsonPage by lazy { pageSupplier(webInterface) }
        protected val json by lazy { page.createJsonObject(freenetRequest)!! }
@@ -80,6 +84,7 @@ abstract class JsonPageTest(
 
        @Before
        fun setupCore() {
+               whenever(core.preferences).thenReturn(preferences)
                whenever(core.getSone(anyString())).thenAnswer { (localSones + remoteSones)[it.getArgument(0)].asOptional() }
                whenever(core.getLocalSone(anyString())).thenAnswer { localSones[it[0]] }
                whenever(core.getPost(anyString())).thenAnswer { (posts + newPosts)[it[0]].asOptional() }
@@ -158,8 +163,8 @@ abstract class JsonPageTest(
                notifications[notificationId ?: notification.id] = notification
        }
 
-       protected fun addSone(sone: Sone) {
-               remoteSones += sone.id to sone
+       protected fun addSone(sone: Sone, soneId: String? = null) {
+               remoteSones += (soneId ?: sone.id) to sone
        }
 
        protected fun addLocalSone(id: String, sone: Sone) {