fix test for Elixir < 1.15, format

SteffenDE · SteffenDE · commit c390cf51af83 · 2024-07-11T17:16:11.000+02:00
diff --git a/test/channel_test.exs b/test/channel_test.exs
@@ -108,33 +108,37 @@ defmodule Phoenix.LiveReloader.ChannelTest do
   end
 
   @endpoint MyApp.LogEndpoint
-  test "sends logs for web console only when enabled" do
-    System.delete_env("PLUG_EDITOR")
-
-    update_live_reload_env(@endpoint, fn conf ->
-      Keyword.drop(conf, [:web_console_logger])
-    end)
+  # web console logger relies on Logger.default_formatter/0
+  # which is only available since Elixir v1.15
+  if Version.match?(System.version(), ">= 1.15.0") do
+    test "sends logs for web console only when enabled" do
+      System.delete_env("PLUG_EDITOR")
+
+      update_live_reload_env(@endpoint, fn conf ->
+        Keyword.drop(conf, [:web_console_logger])
+      end)
 
-    {:ok, info, _socket} =
-      LiveReloader.Socket |> socket() |> subscribe_and_join(Channel, "phoenix:live_reload", %{})
+      {:ok, info, _socket} =
+        LiveReloader.Socket |> socket() |> subscribe_and_join(Channel, "phoenix:live_reload", %{})
 
-    assert info == %{}
-    Logger.info("hello")
+      assert info == %{}
+      Logger.info("hello")
 
-    refute_receive _
+      refute_receive _
 
-    update_live_reload_env(@endpoint, fn conf ->
-      Keyword.merge(conf, web_console_logger: true)
-    end)
+      update_live_reload_env(@endpoint, fn conf ->
+        Keyword.merge(conf, web_console_logger: true)
+      end)
 
-    {:ok, info, _socket} =
-      LiveReloader.Socket |> socket() |> subscribe_and_join(Channel, "phoenix:live_reload", %{})
+      {:ok, info, _socket} =
+        LiveReloader.Socket |> socket() |> subscribe_and_join(Channel, "phoenix:live_reload", %{})
 
-    assert info == %{}
+      assert info == %{}
 
-    Logger.info("hello again")
-    assert_receive %Phoenix.Socket.Message{event: "log", payload: %{msg: msg, level: "info"}}
-    assert msg =~ "hello again"
+      Logger.info("hello again")
+      assert_receive %Phoenix.Socket.Message{event: "log", payload: %{msg: msg, level: "info"}}
+      assert msg =~ "hello again"
+    end
   end
 
   test "sends editor_url and relative_path only when configurd" do
diff --git a/test/live_reloader_test.exs b/test/live_reloader_test.exs
@@ -141,12 +141,12 @@ defmodule Phoenix.LiveReloaderTest do
       |> Phoenix.LiveReloader.call(opts)
       |> send_resp(200, [
         "<html>",
-        '<bo',
+        ~c"<bo",
         [?d, ?y | ">"],
         "<h1>Phoenix</h1>",
         "</b",
         ?o,
-        'dy>',
+        ~c"dy>",
         "</html>"
       ])
 
@@ -161,7 +161,7 @@ defmodule Phoenix.LiveReloaderTest do
       |> Phoenix.LiveReloader.call([])
 
     assert to_string(conn.resp_body) =~
-      ~s[var targetWindow = "parent";\n]
+             ~s[var targetWindow = "parent";\n]
   end
 
   test "wrong window target defaults to top" do
@@ -171,6 +171,6 @@ defmodule Phoenix.LiveReloaderTest do
       |> Phoenix.LiveReloader.call([])
 
     assert to_string(conn.resp_body) =~
-      ~s[var targetWindow = "top";\n]
+             ~s[var targetWindow = "top";\n]
   end
 end
diff --git a/test/test_helper.exs b/test/test_helper.exs
@@ -34,14 +34,14 @@ Application.put_env(:phoenix_live_reload, MyApp.EndpointConfig,
 Application.put_env(:phoenix_live_reload, MyApp.EndpointParentWindow,
   pubsub_server: MyApp.PubSub,
   live_reload: [
-    target_window: :parent,
+    target_window: :parent
   ]
 )
 
 Application.put_env(:phoenix_live_reload, MyApp.EndpointWrongWindow,
   pubsub_server: MyApp.PubSub,
   live_reload: [
-    target_window: "other",
+    target_window: "other"
   ]
 )
 
@@ -54,7 +54,7 @@ Application.put_env(:phoenix_live_reload, MyApp.ReloadEndpoint,
       ~r"priv/gettext/.*(po)$",
       ~r{web/views/.*(ex)$},
       ~r{web/templates/.*(eex)$}
-      ],
+    ],
     notify: [
       live_view: [
         ~r{web/components.ex$},
@@ -64,9 +64,7 @@ Application.put_env(:phoenix_live_reload, MyApp.ReloadEndpoint,
   ]
 )
 
-Application.put_env(:phoenix_live_reload, MyApp.LogEndpoint,
-  pubsub_server: MyApp.PubSub
-)
+Application.put_env(:phoenix_live_reload, MyApp.LogEndpoint, pubsub_server: MyApp.PubSub)
 
 defmodule MyApp.Endpoint do
   use Phoenix.Endpoint, otp_app: :phoenix_live_reload

Original file line number	Diff line number	Diff line change
`@@ -34,14 +34,14 @@ Application.put_env(:phoenix_live_reload, MyApp.EndpointConfig,`
`34`	`34`	`Application.put_env(:phoenix_live_reload, MyApp.EndpointParentWindow,`
`35`	`35`	`pubsub_server: MyApp.PubSub,`
`36`	`36`	`live_reload: [`
`37`		`- target_window: :parent,`
	`37`	`+ target_window: :parent`
`38`	`38`	`]`
`39`	`39`	`)`
`40`	`40`
`41`	`41`	`Application.put_env(:phoenix_live_reload, MyApp.EndpointWrongWindow,`
`42`	`42`	`pubsub_server: MyApp.PubSub,`
`43`	`43`	`live_reload: [`
`44`		`- target_window: "other",`
	`44`	`+ target_window: "other"`
`45`	`45`	`]`
`46`	`46`	`)`
`47`	`47`
`@@ -54,7 +54,7 @@ Application.put_env(:phoenix_live_reload, MyApp.ReloadEndpoint,`
`54`	`54`	`~r"priv/gettext/.*(po)$",`
`55`	`55`	`~r{web/views/.*(ex)$},`
`56`	`56`	`~r{web/templates/.*(eex)$}`
`57`		`- ],`
	`57`	`+ ],`
`58`	`58`	`notify: [`
`59`	`59`	`live_view: [`
`60`	`60`	`~r{web/components.ex$},`
`@@ -64,9 +64,7 @@ Application.put_env(:phoenix_live_reload, MyApp.ReloadEndpoint,`
`64`	`64`	`]`
`65`	`65`	`)`
`66`	`66`
`67`		`-Application.put_env(:phoenix_live_reload, MyApp.LogEndpoint,`
`68`		`- pubsub_server: MyApp.PubSub`
`69`		`-)`
	`67`	`+Application.put_env(:phoenix_live_reload, MyApp.LogEndpoint, pubsub_server: MyApp.PubSub)`
`70`	`68`
`71`	`69`	`defmodule MyApp.Endpoint do`
`72`	`70`	`use Phoenix.Endpoint, otp_app: :phoenix_live_reload`