diff options
Diffstat (limited to 'test/plugs')
| -rw-r--r-- | test/plugs/admin_secret_authentication_plug_test.exs | 2 | ||||
| -rw-r--r-- | test/plugs/ensure_public_or_authenticated_plug_test.exs | 2 | ||||
| -rw-r--r-- | test/plugs/http_security_plug_test.exs | 6 | ||||
| -rw-r--r-- | test/plugs/instance_static_test.exs | 4 | ||||
| -rw-r--r-- | test/plugs/oauth_scopes_plug_test.exs | 2 | ||||
| -rw-r--r-- | test/plugs/rate_limiter_test.exs | 8 | ||||
| -rw-r--r-- | test/plugs/remote_ip_test.exs | 3 | ||||
| -rw-r--r-- | test/plugs/user_enabled_plug_test.exs | 2 | ||||
| -rw-r--r-- | test/plugs/user_is_admin_plug_test.exs | 8 | 
9 files changed, 14 insertions, 23 deletions
| diff --git a/test/plugs/admin_secret_authentication_plug_test.exs b/test/plugs/admin_secret_authentication_plug_test.exs index 2e300ac0c..100016c62 100644 --- a/test/plugs/admin_secret_authentication_plug_test.exs +++ b/test/plugs/admin_secret_authentication_plug_test.exs @@ -23,7 +23,7 @@ defmodule Pleroma.Plugs.AdminSecretAuthenticationPlugTest do    end    describe "when secret set it assigns an admin user" do -    clear_config([:admin_token]) +    setup do: clear_config([:admin_token])      test "with `admin_token` query parameter", %{conn: conn} do        Pleroma.Config.put(:admin_token, "password123") diff --git a/test/plugs/ensure_public_or_authenticated_plug_test.exs b/test/plugs/ensure_public_or_authenticated_plug_test.exs index 3fcb4d372..411252274 100644 --- a/test/plugs/ensure_public_or_authenticated_plug_test.exs +++ b/test/plugs/ensure_public_or_authenticated_plug_test.exs @@ -9,7 +9,7 @@ defmodule Pleroma.Plugs.EnsurePublicOrAuthenticatedPlugTest do    alias Pleroma.Plugs.EnsurePublicOrAuthenticatedPlug    alias Pleroma.User -  clear_config([:instance, :public]) +  setup do: clear_config([:instance, :public])    test "it halts if not public and no user is assigned", %{conn: conn} do      Config.put([:instance, :public], false) diff --git a/test/plugs/http_security_plug_test.exs b/test/plugs/http_security_plug_test.exs index 944a9a139..84e4c274f 100644 --- a/test/plugs/http_security_plug_test.exs +++ b/test/plugs/http_security_plug_test.exs @@ -7,9 +7,9 @@ defmodule Pleroma.Web.Plugs.HTTPSecurityPlugTest do    alias Pleroma.Config    alias Plug.Conn -  clear_config([:http_securiy, :enabled]) -  clear_config([:http_security, :sts]) -  clear_config([:http_security, :referrer_policy]) +  setup do: clear_config([:http_securiy, :enabled]) +  setup do: clear_config([:http_security, :sts]) +  setup do: clear_config([:http_security, :referrer_policy])    describe "http security enabled" do      setup do diff --git a/test/plugs/instance_static_test.exs b/test/plugs/instance_static_test.exs index 8cd9b5712..b8f070d6a 100644 --- a/test/plugs/instance_static_test.exs +++ b/test/plugs/instance_static_test.exs @@ -12,9 +12,7 @@ defmodule Pleroma.Web.RuntimeStaticPlugTest do      on_exit(fn -> File.rm_rf(@dir) end)    end -  clear_config([:instance, :static_dir]) do -    Pleroma.Config.put([:instance, :static_dir], @dir) -  end +  setup do: clear_config([:instance, :static_dir], @dir)    test "overrides index" do      bundled_index = get(build_conn(), "/") diff --git a/test/plugs/oauth_scopes_plug_test.exs b/test/plugs/oauth_scopes_plug_test.exs index 1b3aa85b6..e79ecf263 100644 --- a/test/plugs/oauth_scopes_plug_test.exs +++ b/test/plugs/oauth_scopes_plug_test.exs @@ -193,7 +193,7 @@ defmodule Pleroma.Plugs.OAuthScopesPlugTest do    end    describe "transform_scopes/2" do -    clear_config([:auth, :enforce_oauth_admin_scope_usage]) +    setup do: clear_config([:auth, :enforce_oauth_admin_scope_usage])      setup do        {:ok, %{f: &OAuthScopesPlug.transform_scopes/2}} diff --git a/test/plugs/rate_limiter_test.exs b/test/plugs/rate_limiter_test.exs index c6e494c13..0ce9f3a0a 100644 --- a/test/plugs/rate_limiter_test.exs +++ b/test/plugs/rate_limiter_test.exs @@ -12,14 +12,12 @@ defmodule Pleroma.Plugs.RateLimiterTest do    import Pleroma.Tests.Helpers, only: [clear_config: 1, clear_config: 2]    # Note: each example must work with separate buckets in order to prevent concurrency issues - -  clear_config([Pleroma.Web.Endpoint, :http, :ip]) -  clear_config(:rate_limit) +  setup do: clear_config([Pleroma.Web.Endpoint, :http, :ip]) +  setup do: clear_config(:rate_limit)    describe "config" do      @limiter_name :test_init - -    clear_config([Pleroma.Plugs.RemoteIp, :enabled]) +    setup do: clear_config([Pleroma.Plugs.RemoteIp, :enabled])      test "config is required for plug to work" do        Config.put([:rate_limit, @limiter_name], {1, 1}) diff --git a/test/plugs/remote_ip_test.exs b/test/plugs/remote_ip_test.exs index 9c3737b0b..752ab32e7 100644 --- a/test/plugs/remote_ip_test.exs +++ b/test/plugs/remote_ip_test.exs @@ -9,8 +9,7 @@ defmodule Pleroma.Plugs.RemoteIpTest do    alias Pleroma.Plugs.RemoteIp    import Pleroma.Tests.Helpers, only: [clear_config: 1, clear_config: 2] - -  clear_config(RemoteIp) +  setup do: clear_config(RemoteIp)    test "disabled" do      Pleroma.Config.put(RemoteIp, enabled: false) diff --git a/test/plugs/user_enabled_plug_test.exs b/test/plugs/user_enabled_plug_test.exs index 931513d83..b219d8abf 100644 --- a/test/plugs/user_enabled_plug_test.exs +++ b/test/plugs/user_enabled_plug_test.exs @@ -8,7 +8,7 @@ defmodule Pleroma.Plugs.UserEnabledPlugTest do    alias Pleroma.Plugs.UserEnabledPlug    import Pleroma.Factory -  clear_config([:instance, :account_activation_required]) +  setup do: clear_config([:instance, :account_activation_required])    test "doesn't do anything if the user isn't set", %{conn: conn} do      ret_conn = diff --git a/test/plugs/user_is_admin_plug_test.exs b/test/plugs/user_is_admin_plug_test.exs index 015d51018..fd6a50e53 100644 --- a/test/plugs/user_is_admin_plug_test.exs +++ b/test/plugs/user_is_admin_plug_test.exs @@ -9,9 +9,7 @@ defmodule Pleroma.Plugs.UserIsAdminPlugTest do    import Pleroma.Factory    describe "unless [:auth, :enforce_oauth_admin_scope_usage]," do -    clear_config([:auth, :enforce_oauth_admin_scope_usage]) do -      Pleroma.Config.put([:auth, :enforce_oauth_admin_scope_usage], false) -    end +    setup do: clear_config([:auth, :enforce_oauth_admin_scope_usage], false)      test "accepts a user that is an admin" do        user = insert(:user, is_admin: true) @@ -42,9 +40,7 @@ defmodule Pleroma.Plugs.UserIsAdminPlugTest do    end    describe "with [:auth, :enforce_oauth_admin_scope_usage]," do -    clear_config([:auth, :enforce_oauth_admin_scope_usage]) do -      Pleroma.Config.put([:auth, :enforce_oauth_admin_scope_usage], true) -    end +    setup do: clear_config([:auth, :enforce_oauth_admin_scope_usage], true)      setup do        admin_user = insert(:user, is_admin: true) | 
