fix test breakage

Author: H-Shay

synapse/rest/client/media.py

@@ -195,9 +195,10 @@ async def on_GET(
 def register_servlets(hs: "HomeServer", http_server: HttpServer) -> None:
     if hs.config.experimental.msc3916_authenticated_media_enabled:
         media_repo = hs.get_media_repository()
-        UnstablePreviewURLServlet(hs, media_repo, media_repo.media_storage).register(
-            http_server
-        )
+        if hs.config.media.url_preview_enabled:
+            UnstablePreviewURLServlet(hs, media_repo, media_repo.media_storage).register(
+                http_server
+            )
         UnstableMediaConfigResource(hs).register(http_server)
         UnstableThumbnailResource(hs, media_repo, media_repo.media_storage).register(
             http_server

tests/media/test_media_storage.py

@@ -290,14 +290,6 @@ def write_err(f: Failure) -> Failure:
         config = self.default_config()
         config["media_store_path"] = self.media_store_path
         config["max_image_pixels"] = 2000000
-        config["url_preview_enabled"] = True
-        config["max_spider_size"] = 9999999
-        config["url_preview_ip_range_blacklist"] = (
-            "192.168.1.1",
-            "1.0.0.0/8",
-            "3fff:ffff:ffff:ffff:ffff:ffff:ffff:ffff",
-            "2001:800::/21",
-        )
 
         provider_config = {
             "module": "synapse.media.storage_provider.FileStorageProviderBackend",

tests/rest/client/test_media.py

@@ -168,10 +168,10 @@ def test_remote_media_thumbnail_normally_unblocked(self) -> None:
 
 
 class UnstableURLPreviewTests(unittest.HomeserverTestCase):
-    servlets = [media.register_servlets]
     if not lxml:
         skip = "url preview feature requires lxml"
 
+    servlets = [media.register_servlets]
     hijack_auth = True
     user_id = "@test:user"
     end_content = (