fix: resolve test failure and deprecation warnings

- Add sync point in router_server_test after send to handle_info,
  since lookup_path now reads ETS directly and needs the GenServer
  to have processed the message first
- Replace deprecated Igniter.Project.Config.modify_configuration_code
  with modify_config_code

Author: bcardarella

lib/mix/tasks/beacon.gen.site.ex

@@ -505,7 +505,7 @@ if Code.ensure_loaded?(Igniter) do
               {:ok,
                zipper
                |> Beacon.Igniter.move_to_variable!(:signing_salt)
-               |> Igniter.Project.Config.modify_configuration_code(
+               |> Igniter.Project.Config.modify_config_code(
                  [site_endpoint],
                  otp_app,
                  Sourceror.parse_string!("""
@@ -556,7 +556,7 @@ if Code.ensure_loaded?(Igniter) do
               {:ok,
                zipper
                |> Beacon.Igniter.move_to_variable!(:secret_key_base)
-               |> Igniter.Project.Config.modify_configuration_code(
+               |> Igniter.Project.Config.modify_config_code(
                  [site_endpoint],
                  otp_app,
                  Sourceror.parse_string!(config)

test/beacon/router_server_test.exs

@@ -96,6 +96,8 @@ defmodule Beacon.RouterServerTest do
 
     server = site |> Beacon.RouterServer.name() |> GenServer.whereis()
     send(server, {:page_loaded, page})
+    # Ensure the handle_info has been processed before reading ETS directly
+    RouterServer.dump_pages(site)
 
     assert %Beacon.Content.Page{id: ^page_id} = Beacon.RouterServer.lookup_page!(site, ["test", "router", "add"])
   end