remove failing test assertions

Author: SteffenDE

lib/mix/tasks/phx.gen.auth.ex

@@ -167,7 +167,8 @@ defmodule Mix.Tasks.Phx.Gen.Auth do
       router_scope: router_scope(context),
       web_path_prefix: web_path_prefix(schema),
       test_case_options: test_case_options(ecto_adapter),
-      live?: Keyword.fetch!(context.opts, :live)
+      live?: Keyword.fetch!(context.opts, :live),
+      datetime_module: datetime_module(schema)
     ]
 
     paths = Mix.Phoenix.generator_paths()
@@ -791,6 +792,10 @@ defmodule Mix.Tasks.Phx.Gen.Auth do
   defp test_case_options(Ecto.Adapters.Postgres), do: ", async: true"
   defp test_case_options(adapter) when is_atom(adapter), do: ""
 
+  defp datetime_module(%{timestamp_type: :naive_datetime}), do: NaiveDateTime
+  defp datetime_module(%{timestamp_type: :utc_datetime}), do: DateTime
+  defp datetime_module(%{timestamp_type: :utc_datetime_usec}), do: DateTime
+
   defp put_live_option(schema) do
     opts =
       case Keyword.fetch(schema.opts, :live) do

priv/templates/phx.gen.auth/context_functions.ex

@@ -82,8 +82,8 @@
   """
   def sudo_mode?(<%= schema.singular %>, minutes \\ -20)
 
-  def sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: ts}, minutes) when is_struct(ts, DateTime) do
-    DateTime.after?(ts, DateTime.utc_now() |> DateTime.add(minutes, :minute))
+  def sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: ts}, minutes) when is_struct(ts, <%= inspect datetime_module %>) do
+    <%= inspect datetime_module %>.after?(ts, <%= inspect datetime_module %>.utc_now() |> <%= inspect datetime_module %>.add(minutes, :minute))
   end
 
   def sudo_mode?(_<%= schema.singular %>, _minutes), do: false

priv/templates/phx.gen.auth/login_live_test.exs

@@ -27,7 +27,8 @@ defmodule <%= inspect context.web_module %>.<%= inspect Module.concat(schema.web
 
       assert html =~ "If your email is in our system"
 
-      assert <%= inspect schema.repo %>.get_by!(<%= inspect context.module %>.<%= inspect schema.alias %>Token, <%= schema.singular %>_id: <%= schema.singular %>.id).context == "login"
+      assert <%= inspect schema.repo %>.get_by!(<%= inspect context.module %>.<%= inspect schema.alias %>Token, <%= schema.singular %>_id: <%= schema.singular %>.id).context ==
+               "login"
     end
 
     test "does not disclose if <%= schema.singular %> is registered", %{conn: conn} do

priv/templates/phx.gen.auth/notifier.ex

@@ -49,7 +49,7 @@ defmodule <%= inspect context.module %>.<%= inspect schema.alias %>Notifier do
   end
 
   defp deliver_magic_link_instructions(<%= schema.singular %>, url) do
-    deliver(<%= schema.singular %>.email, "Log in", """
+    deliver(<%= schema.singular %>.email, "Log in instructions", """
 
     ==============================
 

priv/templates/phx.gen.auth/test_cases.exs

@@ -84,15 +84,15 @@
 
   describe "sudo_mode?/2" do
     test "validates the authenticated_at time" do
-      now = DateTime.utc_now()
+      now = <%= inspect datetime_module %>.utc_now()
 
-      assert <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: DateTime.utc_now()})
-      assert <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: DateTime.add(now, -19, :minute)})
-      refute <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: DateTime.add(now, -21, :minute)})
+      assert <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: <%= inspect datetime_module %>.utc_now()})
+      assert <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: <%= inspect datetime_module %>.add(now, -19, :minute)})
+      refute <%= inspect context.alias %>.sudo_mode?(%<%= inspect schema.alias %>{authenticated_at: <%= inspect datetime_module %>.add(now, -21, :minute)})
 
       # minute override
       refute <%= inspect context.alias %>.sudo_mode?(
-               %<%= inspect schema.alias %>{authenticated_at: DateTime.add(now, -11, :minute)},
+               %<%= inspect schema.alias %>{authenticated_at: <%= inspect datetime_module %>.add(now, -11, :minute)},
                -10
              )