From b9af9013546c8bd34caaa26d3332dc52030b40de Mon Sep 17 00:00:00 2001
From: thymusvulgaris <87661013+thymusvulgaris@users.noreply.github.com>
Date: Tue, 25 Jun 2024 13:55:15 +0100
Subject: [PATCH] Add Plug.Adapters.Test.Conn tests (#1235)

* Add Plug.Adapters.Test.Conn.conn/4 test cases for params that
  have a struct or function value.

* Add test that asserts Plug.Adapters.Test.Conn.conn/4 writes a
  message to stderr when the URI path does not start with forward
  slash.

* Add test that asserts Plug.Adapters.Test.Conn.push/3 sends
  message including path and headers.
---
 test/plug/adapters/test/conn_test.exs | 29 +++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/test/plug/adapters/test/conn_test.exs b/test/plug/adapters/test/conn_test.exs
index a3ad4fbb..2977aa5c 100644
--- a/test/plug/adapters/test/conn_test.exs
+++ b/test/plug/adapters/test/conn_test.exs
@@ -64,6 +64,20 @@ defmodule Plug.Adapters.Test.ConnTest do
     assert conn.query_params == %{"foo" => "bar"}
     assert conn.params == %{"foo" => "bar", "biz" => "baz"}
 
+    conn = conn(:post, "/", %{foo: &length/1})
+    assert %{"foo" => value} = conn.body_params
+    assert is_function(value)
+    assert conn.query_string == ""
+    assert conn.query_params == %{}
+    assert conn.params == %{"foo" => &length/1}
+
+    conn = conn(:post, "/", %{foo: %{__struct__: :mod}})
+    assert %{"foo" => value} = conn.body_params
+    assert is_struct(value)
+    assert conn.query_string == ""
+    assert conn.query_params == %{}
+    assert conn.params == %{"foo" => %{__struct__: :mod}}
+
     conn = conn(:post, "/", %{})
     assert conn.body_params == %{}
     assert conn.query_string == ""
@@ -158,9 +172,24 @@ defmodule Plug.Adapters.Test.ConnTest do
     assert child_conn.port == 4200
   end
 
+  test "conn/4 writes message to stderr when URI path does not start with forward slash" do
+    assert ExUnit.CaptureIO.capture_io(:stderr, fn ->
+             Plug.Adapters.Test.Conn.conn(%Plug.Conn{}, :get, "foo", [])
+           end) =~
+             ~s(the URI path used in plug tests must start with "/", got: "foo")
+  end
+
   test "use custom peer data" do
     peer_data = %{address: {127, 0, 0, 1}, port: 111_317}
     conn = conn(:get, "/") |> put_peer_data(peer_data)
     assert peer_data == Plug.Conn.get_peer_data(conn)
   end
+
+  test "push/3 sends message including path and headers" do
+    ref = make_ref()
+
+    Plug.Adapters.Test.Conn.push(%{owner: self(), ref: ref}, "/", [])
+
+    assert_receive {^ref, :push, {"/", []}}
+  end
 end