Encapsulate logic related to resetting update attempts into a changeset function (#2368)

Just a tiny little change which caught my eye

Co-authored-by: Nate Shoemaker <[email protected]>

Author: joshk

lib/nerves_hub/devices.ex

@@ -1017,10 +1017,7 @@ defmodule NervesHub.Devices do
     _ = UpdateStats.log_update(device, previous_metadata)
 
     device
-    |> Ecto.Changeset.change()
-    |> Ecto.Changeset.put_change(:update_attempts, [])
-    |> Ecto.Changeset.put_change(:updates_blocked_until, nil)
-    |> Ecto.Changeset.put_change(:priority_updates, false)
+    |> Device.clear_updates_information_changeset()
     |> Repo.update()
   end
 

lib/nerves_hub/devices/device.ex

@@ -118,4 +118,12 @@ defmodule NervesHub.Devices.Device do
       end
     end)
   end
+
+  def clear_updates_information_changeset(%Device{} = device) do
+    device
+    |> change()
+    |> put_change(:update_attempts, [])
+    |> put_change(:updates_blocked_until, nil)
+    |> put_change(:priority_updates, false)
+  end
 end