diff --git a/lib/skate/application.ex b/lib/skate/application.ex index e2b36435d..9ceefc0be 100644 --- a/lib/skate/application.ex +++ b/lib/skate/application.ex @@ -29,11 +29,7 @@ defmodule Skate.Application do {Oban, Application.fetch_env!(:skate, Oban)} ] - link = Supervisor.start_link(children, strategy: :one_for_all, name: Skate.Supervisor) - - Skate.RemoveUsersWithoutEmail.run() - - link + Supervisor.start_link(children, strategy: :one_for_all, name: Skate.Supervisor) end # Tell Phoenix to update the endpoint configuration diff --git a/lib/skate/remove_users_without_email.ex b/lib/skate/remove_users_without_email.ex deleted file mode 100644 index 7e0cd903c..000000000 --- a/lib/skate/remove_users_without_email.ex +++ /dev/null @@ -1,18 +0,0 @@ -defmodule Skate.RemoveUsersWithoutEmail do - @moduledoc """ - Removes users from the database who don't have an email associated - """ - - import Ecto.Query - alias Skate.Repo - alias Skate.Settings.Db.User - - def run() do - Repo.delete_all( - from(user in User, - where: is_nil(user.email) or user.email == "", - select: user.id - ) - ) - end -end diff --git a/test/skate/remove_users_without_email_test.exs b/test/skate/remove_users_without_email_test.exs deleted file mode 100644 index ad5148bf0..000000000 --- a/test/skate/remove_users_without_email_test.exs +++ /dev/null @@ -1,20 +0,0 @@ -defmodule Skate.RemoveUsersWithoutEmailTest do - use Skate.DataCase - import Skate.Factory - alias Skate.Settings.Db.User - - describe "run/0" do - test "deletes only the users with missing email addresses" do - %{id: user_nil_id} = Skate.Repo.insert!(build(:user, %{email: nil})) - - %{id: user_blank_id} = Skate.Repo.insert!(build(:user, %{email: ""})) - - %{id: user_good_id} = Skate.Repo.insert!(build(:user)) - Skate.RemoveUsersWithoutEmail.run() - - assert nil === Skate.Repo.get(User, user_nil_id) - assert nil === Skate.Repo.get(User, user_blank_id) - assert %{id: ^user_good_id} = Skate.Repo.get(User, user_good_id) - end - end -end