X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FDefaultElementLoaderTest.kt;h=5f8cb88867ae61ceb9d9f956e55bfa82c4211e37;hb=b91ce252d94a8876097b939e129dc33264cef2f5;hp=adc5bc96c28421a0acb4cd629172d21740165159;hpb=675710fc669a9f5ccfab42296a3aa0b822539e14;p=Sone.git

diff --git a/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt b/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
index adc5bc9..5f8cb88 100644
--- a/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
@@ -1,23 +1,21 @@
 package net.pterodactylus.sone.core
 
+import com.google.common.base.Ticker
 import com.google.common.io.ByteStreams
-import com.google.common.io.Files
 import freenet.keys.FreenetURI
 import net.pterodactylus.sone.core.FreenetInterface.BackgroundFetchCallback
 import net.pterodactylus.sone.test.capture
 import net.pterodactylus.sone.test.mock
-import org.hamcrest.MatcherAssert
 import org.hamcrest.MatcherAssert.assertThat
-import org.hamcrest.Matchers
 import org.hamcrest.Matchers.`is`
-import org.hamcrest.Matchers.instanceOf
-import org.hamcrest.Matchers.nullValue
 import org.junit.Test
 import org.mockito.ArgumentMatchers.any
 import org.mockito.ArgumentMatchers.eq
+import org.mockito.Mockito.`when`
 import org.mockito.Mockito.times
 import org.mockito.Mockito.verify
 import java.io.ByteArrayOutputStream
+import java.util.concurrent.TimeUnit
 
 /**
  * Unit test for [DefaultElementLoaderTest].
@@ -26,23 +24,25 @@ class DefaultElementLoaderTest {
 
 	companion object {
 		private const val IMAGE_ID = "KSK@gpl.png"
+		private val freenetURI = FreenetURI(IMAGE_ID)
 	}
 
 	private val freenetInterface = mock<FreenetInterface>()
-	private val elementLoader = DefaultElementLoader(freenetInterface)
+	private val ticker = mock<Ticker>()
+	private val elementLoader = DefaultElementLoader(freenetInterface, ticker)
 	private val callback = capture<BackgroundFetchCallback>()
 
 	@Test
 	fun `image loader starts request for link that is not known`() {
 		elementLoader.loadElement(IMAGE_ID)
-		verify(freenetInterface).startFetch(eq(FreenetURI(IMAGE_ID)), any<BackgroundFetchCallback>())
+		verify(freenetInterface).startFetch(eq(freenetURI), any<BackgroundFetchCallback>())
 	}
 
 	@Test
 	fun `element loader only starts request once`() {
 		elementLoader.loadElement(IMAGE_ID)
 		elementLoader.loadElement(IMAGE_ID)
-		verify(freenetInterface).startFetch(eq(FreenetURI(IMAGE_ID)), any<BackgroundFetchCallback>())
+		verify(freenetInterface).startFetch(eq(freenetURI), any<BackgroundFetchCallback>())
 	}
 
 	@Test
@@ -51,23 +51,61 @@ class DefaultElementLoaderTest {
 	}
 
 	@Test
+	fun `element loader does not cancel on image mime type`() {
+		elementLoader.loadElement(IMAGE_ID)
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		assertThat(callback.value.cancelForMimeType(freenetURI, "image/png"), `is`(false))
+	}
+
+	@Test
+	fun `element loader does  cancel on audio mime type`() {
+		elementLoader.loadElement(IMAGE_ID)
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		assertThat(callback.value.cancelForMimeType(freenetURI, "audio/mpeg"), `is`(true))
+	}
+
+	@Test
+	fun `element loader does not cancel on video mime type`() {
+		elementLoader.loadElement(IMAGE_ID)
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		assertThat(callback.value.cancelForMimeType(freenetURI, "video/mkv"), `is`(true))
+	}
+
+	@Test
+	fun `element loader does not cancel on text mime type`() {
+		elementLoader.loadElement(IMAGE_ID)
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		assertThat(callback.value.cancelForMimeType(freenetURI, "text/plain"), `is`(true))
+	}
+
+	@Test
 	fun `image loader can load image`() {
 		elementLoader.loadElement(IMAGE_ID)
-		verify(freenetInterface).startFetch(eq(FreenetURI(IMAGE_ID)), callback.capture())
-	    callback.value.loaded(FreenetURI(IMAGE_ID), "image/png", read("/static/images/unknown-image-0.png"))
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		callback.value.loaded(freenetURI, "image/png", read("/static/images/unknown-image-0.png"))
 		val linkedElement = elementLoader.loadElement(IMAGE_ID)
-		assertThat(linkedElement.link, `is`(IMAGE_ID))
-		assertThat(linkedElement.loading, `is`(false))
-		assertThat(linkedElement, instanceOf(LinkedImage::class.java))
+		assertThat(linkedElement, `is`(LinkedElement(IMAGE_ID)))
 	}
 
 	@Test
-	fun `image can be loaded again after it failed`() {
+	fun `image is not loaded again after it failed`() {
 		elementLoader.loadElement(IMAGE_ID)
-		verify(freenetInterface).startFetch(eq(FreenetURI(IMAGE_ID)), callback.capture())
-		callback.value.failed(FreenetURI(IMAGE_ID))
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		callback.value.failed(freenetURI)
+		assertThat(elementLoader.loadElement(IMAGE_ID).failed, `is`(true))
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+	}
+
+	@Test
+	fun `image is loaded again after failure cache is expired`() {
 		elementLoader.loadElement(IMAGE_ID)
-		verify(freenetInterface, times(2)).startFetch(eq(FreenetURI(IMAGE_ID)), callback.capture())
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		callback.value.failed(freenetURI)
+		`when`(ticker.read()).thenReturn(TimeUnit.MINUTES.toNanos(31))
+		val linkedElement = elementLoader.loadElement(IMAGE_ID)
+		assertThat(linkedElement.failed, `is`(false))
+		assertThat(linkedElement.loading, `is`(true))
+		verify(freenetInterface, times(2)).startFetch(eq(freenetURI), callback.capture())
 	}
 
 	private fun read(resource: String): ByteArray =