MediaProxy test: use config macros instead of directly putting values
authorrinpatch <rinpatch@sdf.org>
Wed, 27 May 2020 16:34:56 +0000 (19:34 +0300)
committerrinpatch <rinpatch@sdf.org>
Wed, 27 May 2020 16:34:56 +0000 (19:34 +0300)
They were not properly cleaned later and caused trouble for another tests

test/web/media_proxy/media_proxy_test.exs

index 69c2d5dae89fe8dc3a34da08ed7ad8e87ec9f806..69d2a71a68d2e3a727a9b0e90c58dadb108c75be 100644 (file)
@@ -124,15 +124,7 @@ defmodule Pleroma.Web.MediaProxyTest do
     end
 
     test "uses the configured base_url" do
-      base_url = Pleroma.Config.get([:media_proxy, :base_url])
-
-      if base_url do
-        on_exit(fn ->
-          Pleroma.Config.put([:media_proxy, :base_url], base_url)
-        end)
-      end
-
-      Pleroma.Config.put([:media_proxy, :base_url], "https://cache.pleroma.social")
+      clear_config([:media_proxy, :base_url], "https://cache.pleroma.social")
 
       url = "https://pleroma.soykaf.com/static/logo.png"
       encoded = url(url)
@@ -213,8 +205,8 @@ defmodule Pleroma.Web.MediaProxyTest do
     end
 
     test "does not change whitelisted urls" do
-      Pleroma.Config.put([:media_proxy, :whitelist], ["mycdn.akamai.com"])
-      Pleroma.Config.put([:media_proxy, :base_url], "https://cache.pleroma.social")
+      clear_config([:media_proxy, :whitelist], ["mycdn.akamai.com"])
+      clear_config([:media_proxy, :base_url], "https://cache.pleroma.social")
 
       media_url = "https://mycdn.akamai.com"