X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FPreferencesTest.kt;h=6da665869dfdf244677218d5e84c040b38cd7dac;hb=72e43af1e442398e304167c1e18e229d254fba7b;hp=ca58c40b0c7f238c3bddd215b14932ed53a97d46;hpb=fc8e9ea5c978d2f5a00894e093364ae62f18934b;p=Sone.git

diff --git a/src/test/kotlin/net/pterodactylus/sone/core/PreferencesTest.kt b/src/test/kotlin/net/pterodactylus/sone/core/PreferencesTest.kt
index ca58c40..6da6658 100644
--- a/src/test/kotlin/net/pterodactylus/sone/core/PreferencesTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/core/PreferencesTest.kt
@@ -180,69 +180,6 @@ class PreferencesTest {
 	}
 
 	@Test
-	fun `preferences retain positive trust`() {
-		preferences.newPositiveTrust = 15
-		assertThat(preferences.positiveTrust, equalTo(15))
-	}
-
-	@Test(expected = IllegalArgumentException::class)
-	fun `invalid positive trust is rejected`() {
-		preferences.newPositiveTrust = -15
-	}
-
-	@Test
-	fun `preferences return default value when positive trust is set to null`() {
-		preferences.newPositiveTrust = null
-		assertThat(preferences.positiveTrust, equalTo(75))
-	}
-
-	@Test
-	fun `preferences start with positive trust default value`() {
-		assertThat(preferences.positiveTrust, equalTo(75))
-	}
-
-	@Test
-	fun `preferences retain negative trust`() {
-		preferences.newNegativeTrust = -15
-		assertThat(preferences.negativeTrust, equalTo(-15))
-	}
-
-	@Test(expected = IllegalArgumentException::class)
-	fun `invalid negative trust is rejected`() {
-		preferences.newNegativeTrust = 150
-	}
-
-	@Test
-	fun `preferences return default value when negative trust is set to null`() {
-		preferences.newNegativeTrust = null
-		assertThat(preferences.negativeTrust, equalTo(-25))
-	}
-
-	@Test
-	fun `preferences start with negative trust default value`() {
-		assertThat(preferences.negativeTrust, equalTo(-25))
-	}
-
-	@Test
-	fun `preferences retain trust comment`() {
-		preferences.newTrustComment = "Trust"
-		assertThat(preferences.trustComment, equalTo("Trust"))
-	}
-
-	@Test
-	fun `preferences return default value when trust comment is set to null`() {
-		preferences.newTrustComment = null
-		assertThat(preferences.trustComment,
-				equalTo("Set from Sone Web Interface"))
-	}
-
-	@Test
-	fun `preferences start with trust comment default value`() {
-		assertThat(preferences.trustComment,
-				equalTo("Set from Sone Web Interface"))
-	}
-
-	@Test
 	fun `preferences retain fcp interface active of true`() {
 		preferences.newFcpInterfaceActive = true
 		assertThat(preferences.fcpInterfaceActive, equalTo(true))
@@ -311,16 +248,18 @@ class PreferencesTest {
 
 	@Test
 	fun `setting insertion delay to valid value sends change event`() {
-		preferences.newInsertionDelay = 30
-		verify(eventBus, atLeastOnce()).post(eventsCaptor.capture())
-		assertThat(eventsCaptor.allValues, hasItem(PreferenceChangedEvent("InsertionDelay", 30)))
+		testPreferencesChangedEvent("InsertionDelay", { preferences.newInsertionDelay = it }, 30)
 	}
 
 	@Test
 	fun `setting posts per page to valid value sends change event`() {
-		preferences.newPostsPerPage = 30
+		testPreferencesChangedEvent("PostsPerPage", { preferences.newPostsPerPage = it }, 31)
+	}
+
+	private fun <T : Any> testPreferencesChangedEvent(name: String, setter: (T) -> Unit, value: T) {
+		setter(value)
 		verify(eventBus, atLeastOnce()).post(eventsCaptor.capture())
-		assertThat(eventsCaptor.allValues, hasItem(PreferenceChangedEvent("PostsPerPage", 30)))
+		assertThat(eventsCaptor.allValues, hasItem(PreferenceChangedEvent(name, value)))
 	}
 
 }