From 131ac29df456263fa471fed5875228e0402db9d9 Mon Sep 17 00:00:00 2001
From: Dan Brown <ssddanbrown@googlemail.com>
Date: Fri, 23 May 2025 17:19:34 +0100
Subject: [PATCH] Images: Added testing to cover animated avif handling

---
 tests/Uploads/ImageTest.php   |  15 ++++++++++++++-
 tests/test-data/animated.avif | Bin 0 -> 1168 bytes
 2 files changed, 14 insertions(+), 1 deletion(-)
 create mode 100644 tests/test-data/animated.avif

diff --git a/tests/Uploads/ImageTest.php b/tests/Uploads/ImageTest.php
index 2c36f5f35..a2f03df34 100644
--- a/tests/Uploads/ImageTest.php
+++ b/tests/Uploads/ImageTest.php
@@ -68,7 +68,20 @@ class ImageTest extends TestCase
         $this->files->deleteAtRelativePath($imgDetails['path']);
 
         $this->assertStringContainsString('thumbs-', $imgDetails['response']->thumbs->gallery);
-        $this->assertStringNotContainsString('thumbs-', $imgDetails['response']->thumbs->display);
+        $this->assertStringNotContainsString('scaled-', $imgDetails['response']->thumbs->display);
+    }
+
+    public function test_image_display_thumbnail_generation_for_animated_avif_images_uses_original_file()
+    {
+        $page = $this->entities->page();
+        $admin = $this->users->admin();
+        $this->actingAs($admin);
+
+        $imgDetails = $this->files->uploadGalleryImageToPage($this, $page, 'animated.avif');
+        $this->files->deleteAtRelativePath($imgDetails['path']);
+
+        $this->assertStringContainsString('thumbs-', $imgDetails['response']->thumbs->gallery);
+        $this->assertStringNotContainsString('scaled-', $imgDetails['response']->thumbs->display);
     }
 
     public function test_image_edit()
diff --git a/tests/test-data/animated.avif b/tests/test-data/animated.avif
new file mode 100644
index 0000000000000000000000000000000000000000..92f71459bb8182f0f6d2854dca8630a315676010
GIT binary patch
literal 1168
zcmb7DUq}>T5TD(>>^v+El{!k~9<DMoyFDoK$t(LKlu=<1fiLU+=|0`hU3S;S#7K#X
ztmr9-2%?9cA|f>C!P3BYee5CV9ukBKMB$>p*UavnUFJ)(%+Abj=9_Q6`Q`(F7Q?$}
zD?W1pi6pYYT|;KBb&xYd=1eiV4#|hn{gvyULUBm@a+>LcDuB&YudI_ijrO3UcPay#
zsRvP;ZRYVHM8JO|GcA>@BLD(^^am-AQbulKY`{Pnps)=MT`XCOpO!l;(+YC-G25}x
zae~>Zg|eQxwoZXakrKykHpB>!qR}Xd8pW48#-N6vI@L0r0o5D=G58SL%4}XfAr=-9
z#4DIdoZyz_qub<uPNSuNNLs|)i&Q@yW0n3g5M5(uR|D5o@VH{T=P3Q?&3XMf4VO#n
zL<U(e7eo5Tzh=*MYbf^#Af;;_4MBr$x^$VenzqIBl`IgRaE&QwCAqe?4_Sn>aBddG
z{|<-H&BF_?_j!=Evhb7^`WiGx4~HTwt!B_MleiVe3iYnnYodP3_1yCHP>vBA7nDSm
zIYT6ZL(+*VF_Dlssrc#vxm9j!mD6(4WW9>T_u{~Wet<-=h-&2s#SEsKLF%}w25p0p
zFgoFSK|9H!0Bm%ebgb14-dYJKUjt&nbEv@RV#t?5-!RHD5l#iQAn!(QM#h}M13;D5
zNO_bmxu$rrYJ%=OET*HGR2-&9LF{@jUO#&CUExyS*~wV#__LqM9qdc(qvY|0{Dt$`
zOmkYA83h>r0Z_ehJ_dJ}USB)aex~O}+or<Yhfia(pO+^JRXxj7?sR5Xdczwm=hzHR
z>+O#)3QG&)_r6P8o_|}J8mwz{pFUXL{_DlgY^Et)B}O8pzyb!Y>?ssyw6~8ZU-6dx
Q_m?`Z4o@#8y61X-181VpD*ylh

literal 0
HcmV?d00001