From: David ‘Bombe’ Roden <bombe@pterodactylus.net>
Date: Sun, 18 Oct 2026 07:36:32 +0000 (+0200)
Subject: 🎨 Verify that failed images are not loaded again
X-Git-Tag: v83^2~5^2~3
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=f757a08f5908af8d6ae63d11032696a0f5a649b8;p=Sone.git

🎨 Verify that failed images are not loaded again
---

diff --git a/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt b/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
index 4e9a28d..fd70e03 100644
--- a/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
+++ b/src/test/kotlin/net/pterodactylus/sone/core/DefaultElementLoaderTest.kt
@@ -28,6 +28,7 @@ import org.hamcrest.Matchers.containsString
 import org.hamcrest.Matchers.equalTo
 import org.hamcrest.Matchers.hasEntry
 import org.hamcrest.Matchers.hasItem
+import org.hamcrest.Matchers.hasSize
 import org.hamcrest.Matchers.not
 import org.hamcrest.TypeSafeDiagnosingMatcher
 import org.junit.Rule
@@ -220,6 +221,15 @@ class DefaultElementLoaderTest {
 	}
 
 	@Test
+	fun `image is not loaded again after it failed`() {
+		runWithCallback(IMAGE_ID) { elementLoader, _, callback, fetchedUris ->
+			callback.failed(freenetURI)
+			elementLoader.loadElement(IMAGE_ID)
+			assertThat(fetchedUris(), hasSize(1))
+		}
+	}
+
+	@Test
 	fun `element loading failure is logged`() {
 		runWithCallback(IMAGE_ID) { _, _, callback, _ ->
 			callback.failed(freenetURI)