From b281ad06de2de331450a5e319e3ba497071d4197 Mon Sep 17 00:00:00 2001
From: Mark Felder <feld@feld.me>
Date: Tue, 13 Aug 2024 15:34:56 -0400
Subject: [PATCH] Revert "Custom mix task to retry failed tests once in CI
 pipeline"

This reverts commit ed2976b237b53a524247d564691e0a12d3231d68.
---
 .gitlab-ci.yml                          |  2 +-
 changelog.d/fix-test-failures.skip      |  0
 lib/mix/tasks/pleroma/test_runner.ex    | 25 -------------------------
 test/mix/tasks/pleroma/uploads_test.exs |  2 +-
 4 files changed, 2 insertions(+), 27 deletions(-)
 create mode 100644 changelog.d/fix-test-failures.skip
 delete mode 100644 lib/mix/tasks/pleroma/test_runner.ex

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index eaa9d3b25c..0c6631797a 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -133,7 +133,7 @@ unit-testing-1.13.4-otp-25:
   script: &testing_script
     - mix ecto.create
     - mix ecto.migrate
-    - mix pleroma.test_runner --cover --preload-modules
+    - mix test --cover --preload-modules
   coverage: '/^Line total: ([^ ]*%)$/'
   artifacts:
     reports:
diff --git a/changelog.d/fix-test-failures.skip b/changelog.d/fix-test-failures.skip
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/lib/mix/tasks/pleroma/test_runner.ex b/lib/mix/tasks/pleroma/test_runner.ex
deleted file mode 100644
index 69fefb0014..0000000000
--- a/lib/mix/tasks/pleroma/test_runner.ex
+++ /dev/null
@@ -1,25 +0,0 @@
-defmodule Mix.Tasks.Pleroma.TestRunner do
-  @shortdoc "Retries tests once if they fail"
-
-  use Mix.Task
-
-  def run(args \\ []) do
-    case System.cmd("mix", ["test"] ++ args, into: IO.stream(:stdio, :line)) do
-      {_, 0} ->
-        :ok
-
-      _ ->
-        retry(args)
-    end
-  end
-
-  def retry(args) do
-    case System.cmd("mix", ["test", "--failed"] ++ args, into: IO.stream(:stdio, :line)) do
-      {_, 0} ->
-        :ok
-
-      _ ->
-        exit(1)
-    end
-  end
-end
diff --git a/test/mix/tasks/pleroma/uploads_test.exs b/test/mix/tasks/pleroma/uploads_test.exs
index 22b6baf24c..f3d5aa64fd 100644
--- a/test/mix/tasks/pleroma/uploads_test.exs
+++ b/test/mix/tasks/pleroma/uploads_test.exs
@@ -58,7 +58,7 @@ test "nonexistent uploader" do
     end
   end
 
-  defp prep_uploads() do
+  defp prep_uploads do
     upload_dir = Config.get([Pleroma.Uploaders.Local, :uploads])
 
     if not File.exists?(upload_dir) || File.ls!(upload_dir) == [] do