Fix race condition in tests

Nebo15 · May 14, 2024 · fdb13e5 · fdb13e5
1 parent 7a52b45
commit fdb13e5
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/lib/logger_json/formatters/basic.ex b/lib/logger_json/formatters/basic.ex
@@ -40,7 +40,7 @@ defmodule LoggerJSON.Formatters.Basic do
     line =
       %{
         time: utc_time(meta),
-        severity: Atom.to_string(level) |> String.downcase(),
+        severity: Atom.to_string(level),
         message: encode(message, redactors),
         metadata: encode(take_metadata(meta, metadata_selector), redactors)
       }

diff --git a/test/logger_json/ecto_test.exs b/test/logger_json/ecto_test.exs
@@ -3,6 +3,11 @@ defmodule LoggerJSON.EctoTest do
   import LoggerJSON.Ecto
   require Logger
 
+  setup do
+    formatter = {LoggerJSON.Formatters.Basic, metadata: :all}
+    :logger.update_handler_config(:default, :formatter, formatter)
+  end
+
   describe "telemetry_logging_handler/4" do
     test "logs ecto queries received via telemetry event" do
       log =
@@ -43,8 +48,7 @@ defmodule LoggerJSON.EctoTest do
                    "queue_time_μs" => 106,
                    "repo" => "Repo"
                  }
-               },
-               "severity" => "info"
+               }
              } = decode_or_print_error(log)
     end
 

diff --git a/test/logger_json/plug_test.exs b/test/logger_json/plug_test.exs
@@ -3,6 +3,11 @@ defmodule LoggerJSON.PlugTest do
   import LoggerJSON.Plug
   require Logger
 
+  setup do
+    formatter = {LoggerJSON.Formatters.Basic, metadata: :all}
+    :logger.update_handler_config(:default, :formatter, formatter)
+  end
+
   describe "telemetry_logging_handler/4" do
     test "logs request latency and metadata" do
       conn = Plug.Test.conn(:get, "/")
@@ -25,8 +30,7 @@ defmodule LoggerJSON.PlugTest do
                "request" => %{
                  "client" => %{"ip" => "127.0.0.1", "user_agent" => nil},
                  "connection" => %{"method" => "GET", "path" => "/", "protocol" => "HTTP/1.1", "status" => nil}
-               },
-               "severity" => "info"
+               }
              } = decode_or_print_error(log)
     end