X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Ftest%2Fkotlin%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FDefaultElementLoaderTest.kt;h=424aaff7b15324e619a2f16fefbcca066552304e;hb=7bb75fc81cd59d728fbc076a0d7356e43720c92a;hp=720cd675e15b75b873923213a43b15e8992caabd;hpb=6e91e9356231217afcecb12275882c0ebf0f34a8;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 720cd67..424aaff 100644
--- a/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
@@ -1,5 +1,6 @@
 package net.pterodactylus.sone.core
 
+import com.google.common.base.Ticker
 import com.google.common.io.ByteStreams
 import freenet.keys.FreenetURI
 import net.pterodactylus.sone.core.FreenetInterface.BackgroundFetchCallback
@@ -7,13 +8,14 @@ import net.pterodactylus.sone.test.capture
 import net.pterodactylus.sone.test.mock
 import org.hamcrest.MatcherAssert.assertThat
 import org.hamcrest.Matchers.`is`
-import org.hamcrest.Matchers.instanceOf
 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].
@@ -23,13 +25,17 @@ class DefaultElementLoaderTest {
 	companion object {
 		private const val IMAGE_ID = "KSK@gpl.png"
 		private val freenetURI = FreenetURI(IMAGE_ID)
+		private const val decomposedKey = "CHK@DCiVgTWW9nnWHJc9EVwtFJ6jAfBSVyy~rgiPvhUKbS4,mNY85V0x7dYcv7SnEYo1PCC6y2wNWMDNt-y9UWQx9fI,AAMC--8/fru%CC%88hstu%CC%88ck.jpg"
+		private const val normalizedKey = "CHK@DCiVgTWW9nnWHJc9EVwtFJ6jAfBSVyy~rgiPvhUKbS4,mNY85V0x7dYcv7SnEYo1PCC6y2wNWMDNt-y9UWQx9fI,AAMC--8/frühstück.jpg"
+		private val sizeOkay = 2097152L
+		private val sizeNotOkay = sizeOkay + 1
 	}
 
 	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)
@@ -49,50 +55,67 @@ class DefaultElementLoaderTest {
 	}
 
 	@Test
-	fun `element loader does not cancel on image mime type`() {
+	fun `element loader does not cancel on image mime type with 2 mib size`() {
 		elementLoader.loadElement(IMAGE_ID)
 		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
-		assertThat(callback.value.cancelForMimeType(freenetURI, "image/png"), `is`(false))
+		assertThat(callback.value.shouldCancel(freenetURI, "image/png", sizeOkay), `is`(false))
+	}
+
+	@Test
+	fun `element loader does cancel on image mime type with more than 2 mib size`() {
+		elementLoader.loadElement(IMAGE_ID)
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
+		assertThat(callback.value.shouldCancel(freenetURI, "image/png", sizeNotOkay), `is`(true))
 	}
 
 	@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))
+		assertThat(callback.value.shouldCancel(freenetURI, "audio/mpeg", sizeOkay), `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))
+		assertThat(callback.value.shouldCancel(freenetURI, "video/mkv", sizeOkay), `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))
+		assertThat(callback.value.shouldCancel(freenetURI, "text/plain", sizeOkay), `is`(true))
 	}
 
 	@Test
 	fun `image loader can load image`() {
+		elementLoader.loadElement(decomposedKey)
+		verify(freenetInterface).startFetch(eq(FreenetURI(decomposedKey)), callback.capture())
+		callback.value.loaded(FreenetURI(normalizedKey), "image/png", read("/static/images/unknown-image-0.png"))
+		val linkedElement = elementLoader.loadElement(decomposedKey)
+		assertThat(linkedElement, `is`(LinkedElement(normalizedKey)))
+	}
+
+	@Test
+	fun `image is not loaded again after it failed`() {
 		elementLoader.loadElement(IMAGE_ID)
 		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))
+		callback.value.failed(freenetURI)
+		assertThat(elementLoader.loadElement(IMAGE_ID).failed, `is`(true))
+		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
 	}
 
 	@Test
-	fun `image can be loaded again after it failed`() {
+	fun `image is loaded again after failure cache is expired`() {
 		elementLoader.loadElement(IMAGE_ID)
 		verify(freenetInterface).startFetch(eq(freenetURI), callback.capture())
 		callback.value.failed(freenetURI)
-		elementLoader.loadElement(IMAGE_ID)
+		`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())
 	}