DEV: Use existing image for logo in tests (#36627)

davidtaylorhq · web-flow · commit 2677ac1e4fab · 2025-12-12T12:59:18.000Z
Stops the 404 errors in qunit tests
diff --git a/frontend/discourse/tests/acceptance/image-aspect-ratio-test.js b/frontend/discourse/tests/acceptance/image-aspect-ratio-test.js
@@ -7,7 +7,7 @@ acceptance("Image aspect ratio", function () {
     await visit("/t/2480");
 
     assert
-      .dom("#post_3 img[src='/assets/logo.png']")
+      .dom("#post_3 img[src='/images/discourse-logo-sketch.png']")
       .hasStyle({ aspectRatio: "690 / 388" });
   });
 });
diff --git a/frontend/discourse/tests/acceptance/transformers/home-logo-image-url-test.js b/frontend/discourse/tests/acceptance/transformers/home-logo-image-url-test.js
@@ -18,7 +18,7 @@ acceptance("home-logo-image-url transformer", function () {
       .dom("#site-logo")
       .hasAttribute(
         "src",
-        "/transformed/assets/logo.png",
+        "/transformed/images/discourse-logo-sketch.png",
         "it transforms the logo url"
       );
   });
diff --git a/frontend/discourse/tests/fixtures/topic.js b/frontend/discourse/tests/fixtures/topic.js
@@ -5619,7 +5619,7 @@ export default {
           avatar_template: "/images/avatar.png",
           created_at: "2020-07-24T17:50:17.274Z",
           cooked:
-            "<aside class=\"onebox allowlistedgeneric\" data-onebox-src=\"https://example.com/57350945\"><header class=\"source\"><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">XYZ News Site</a> </header> <article class=\"onebox-body\"> <div class=\"aspect-image\" style=\"--aspect-ratio:690/388;\"><img src=\"/assets/logo.png\" class=\"thumbnail d-lazyload\" width=\"690\" height=\"388\"></div> <h3><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">News Headline</a></h3> <p>Article summary</p> </article> <div class=\"onebox-metadata\"> </div> <div style=\"clear: both\"></div> </aside>",
+            "<aside class=\"onebox allowlistedgeneric\" data-onebox-src=\"https://example.com/57350945\"><header class=\"source\"><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">XYZ News Site</a> </header> <article class=\"onebox-body\"> <div class=\"aspect-image\" style=\"--aspect-ratio:690/388;\"><img src=\"/images/discourse-logo-sketch.png\" class=\"thumbnail d-lazyload\" width=\"690\" height=\"388\"></div> <h3><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">News Headline</a></h3> <p>Article summary</p> </article> <div class=\"onebox-metadata\"> </div> <div style=\"clear: both\"></div> </aside>",
           post_number: 3,
           post_type: 1,
           updated_at: "2020-07-24T17:50:17.274Z",
@@ -5964,7 +5964,7 @@ export default {
           avatar_template: "/images/avatar.png",
           created_at: "2020-07-24T17:50:17.274Z",
           cooked:
-            "<aside class=\"onebox allowlistedgeneric\" data-onebox-src=\"https://example.com/57350945\"><header class=\"source\"><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">XYZ News Site</a> </header> <article class=\"onebox-body\"> <div class=\"aspect-image\" style=\"--aspect-ratio:690/388;\"><img src=\"/assets/logo.png\" class=\"thumbnail d-lazyload\" width=\"690\" height=\"388\"></div> <h3><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">News Headline</a></h3> <p>Article summary</p> </article> <div class=\"onebox-metadata\"> </div> <div style=\"clear: both\"></div> </aside>",
+            "<aside class=\"onebox allowlistedgeneric\" data-onebox-src=\"https://example.com/57350945\"><header class=\"source\"><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">XYZ News Site</a> </header> <article class=\"onebox-body\"> <div class=\"aspect-image\" style=\"--aspect-ratio:690/388;\"><img src=\"/images/discourse-logo-sketch.png\" class=\"thumbnail d-lazyload\" width=\"690\" height=\"388\"></div> <h3><a href=\"https://example.com/57350945\" target=\"_blank\" rel=\"noopener\">News Headline</a></h3> <p>Article summary</p> </article> <div class=\"onebox-metadata\"> </div> <div style=\"clear: both\"></div> </aside>",
           post_number: 3,
           post_type: 1,
           updated_at: "2020-07-24T17:50:17.274Z",
diff --git a/frontend/discourse/tests/helpers/site-settings.js b/frontend/discourse/tests/helpers/site-settings.js
@@ -2,8 +2,8 @@ import { createSiteSettingsFromPreloaded } from "discourse/services/site-setting
 
 const CLIENT_SETTING_TEST_OVERRIDES = {
   title: "QUnit Discourse Tests",
-  site_logo_url: "/assets/logo.png",
-  site_logo_small_url: "/assets/logo-single.png",
+  site_logo_url: "/images/discourse-logo-sketch.png",
+  site_logo_small_url: "/images/discourse-logo-sketch-small.png",
   site_mobile_logo_url: "",
   site_favicon_url: "/images/discourse-logo-sketch-small.png",
   enable_twitter_logins: true,
