Deprecate use Plug.Test (#1252)

Author: mohammedzeglam-pg

README.md

@@ -284,8 +284,9 @@ Plug ships with a `Plug.Test` module that makes testing your plugs easy. Here is
 ```elixir
 defmodule MyPlugTest do
   use ExUnit.Case, async: true
-  use Plug.Test
-
+  import Plug.Test
+  import Plug.Conn
+  
   @opts MyRouter.init([])
 
   test "returns hello world" do

lib/plug/test.ex

@@ -5,7 +5,8 @@ defmodule Plug.Test do
   This module can be used in your test cases, like this:
 
       use ExUnit.Case, async: true
-      use Plug.Test
+      import Plug.Test
+      import Plug.Conn
 
   Using this module will:
 
@@ -22,6 +23,9 @@ defmodule Plug.Test do
   """
 
   @doc false
+  @deprecated """
+  Please use `import Plug.Test` and `import Plug.Conn` directly instead.
+  """
   defmacro __using__(_) do
     quote do
       import Plug.Test

test/plug/basic_auth_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.BasicAuthTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   import Plug.BasicAuth
 

test/plug/builder_test.exs

@@ -82,7 +82,8 @@ defmodule Plug.BuilderTest do
   end
 
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   test "exports the init/1 function" do
     assert Sample.init(:ok) == :ok

test/plug/conn/wrapper_error_test.exs

@@ -1,6 +1,6 @@
 defmodule Plug.Conn.WrapperErrorTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
 
   test "reraise/3" do
     conn = conn(:get, "/")

test/plug/conn_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.ConnTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   alias Plug.Conn
   alias Plug.ProcessStore

test/plug/csrf_protection_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.CSRFProtectionTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   alias Plug.CSRFProtection
   alias Plug.CSRFProtection.InvalidCSRFTokenError

test/plug/debugger_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.DebuggerTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   import ExUnit.CaptureLog
 

test/plug/error_handler_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.ErrorHandlerTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   defmodule ForbiddenError do
     defexception plug_status: 403, message: "oops"

test/plug/head_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.HeadTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   @opts Plug.Head.init([])
 

test/plug/logger_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.LoggerTest do
   use ExUnit.Case
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   import ExUnit.CaptureLog
 

test/plug/method_override_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.MethodOverrideTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   def urlencoded_conn(method, body) do
     method

test/plug/parsers/json_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.Parsers.JSONTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   defmodule JSON do
     def decode!("[1, 2, 3]") do

test/plug/parsers_test.exs

@@ -1,7 +1,8 @@
 defmodule Plug.ParsersTest do
   use ExUnit.Case, async: true
 
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   defmodule BodyReader do
     def read_body(conn, opts) do

test/plug/request_id_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.RequestIdTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   defp call(conn, opts) do
     Plug.RequestId.call(conn, Plug.RequestId.init(opts))

test/plug/rewrite_on_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.RewriteOnTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   defp call(conn, rewrite) do
     Plug.RewriteOn.call(conn, Plug.RewriteOn.init(rewrite))

test/plug/router_test.exs

@@ -226,7 +226,8 @@ defmodule Plug.RouterTest do
   end
 
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   test "dispatch root" do
     conn = call(Sample, conn(:get, "/"))

test/plug/session/cookie_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.Session.CookieTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   alias Plug.Session.COOKIE, as: CookieStore
 

test/plug/session_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.SessionTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   alias Plug.ProcessStore
   doctest Plug.Session.Store

test/plug/ssl_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.SSLTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   describe "configure" do
     import Plug.SSL, only: [configure: 1]

test/plug/static_test.exs

@@ -1,6 +1,7 @@
 defmodule Plug.StaticTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   @default_opts [
     at: "/public",

test/plug/telemetry_test.exs

@@ -2,7 +2,7 @@ Application.ensure_all_started(:telemetry)
 
 defmodule Plug.TelemetryTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
 
   defmodule MyPlug do
     use Plug.Builder

test/plug_test.exs

@@ -1,6 +1,7 @@
 defmodule PlugTest do
   use ExUnit.Case, async: true
-  use Plug.Test
+  import Plug.Test
+  import Plug.Conn
 
   import ExUnit.CaptureLog