From f930796f572d1ebcfcfe24467aaf6915ff69f92f Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Thu, 23 May 2024 10:55:22 +0200
Subject: [PATCH] Move FakeBugReporterMode to FakeBugReporter.Mode

---
 .../impl/bugreport/BugReportPresenterTest.kt  |  6 +++---
 .../impl/bugreport/FakeBugReporter.kt         | 20 +++++++++----------
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt b/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt
index 69617f30f8..9b07b1c942 100644
--- a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt
+++ b/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt
@@ -144,7 +144,7 @@ class BugReportPresenterTest {
     @Test
     fun `present - send success`() = runTest {
         val presenter = createPresenter(
-            FakeBugReporter(mode = FakeBugReporterMode.Success),
+            FakeBugReporter(mode = FakeBugReporter.Mode.Success),
             FakeCrashDataStore(crashData = A_CRASH_DATA, appHasCrashed = true),
             FakeScreenshotHolder(screenshotUri = A_SCREENSHOT_URI),
         )
@@ -170,7 +170,7 @@ class BugReportPresenterTest {
     @Test
     fun `present - send failure`() = runTest {
         val presenter = createPresenter(
-            FakeBugReporter(mode = FakeBugReporterMode.Failure),
+            FakeBugReporter(mode = FakeBugReporter.Mode.Failure),
             FakeCrashDataStore(crashData = A_CRASH_DATA, appHasCrashed = true),
             FakeScreenshotHolder(screenshotUri = A_SCREENSHOT_URI),
         )
@@ -219,7 +219,7 @@ class BugReportPresenterTest {
     @Test
     fun `present - send cancel`() = runTest {
         val presenter = createPresenter(
-            FakeBugReporter(mode = FakeBugReporterMode.Cancel),
+            FakeBugReporter(mode = FakeBugReporter.Mode.Cancel),
             FakeCrashDataStore(crashData = A_CRASH_DATA, appHasCrashed = true),
             FakeScreenshotHolder(screenshotUri = A_SCREENSHOT_URI),
         )
diff --git a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeBugReporter.kt b/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeBugReporter.kt
index cce2d5d144..a4d4f83da9 100644
--- a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeBugReporter.kt
+++ b/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeBugReporter.kt
@@ -22,7 +22,13 @@ import io.element.android.libraries.matrix.test.A_FAILURE_REASON
 import kotlinx.coroutines.delay
 import java.io.File
 
-class FakeBugReporter(val mode: FakeBugReporterMode = FakeBugReporterMode.Success) : BugReporter {
+class FakeBugReporter(val mode: Mode = Mode.Success) : BugReporter {
+    enum class Mode {
+        Success,
+        Failure,
+        Cancel
+    }
+
     override suspend fun sendBugReport(
         withDevicesLogs: Boolean,
         withCrashLogs: Boolean,
@@ -37,12 +43,12 @@ class FakeBugReporter(val mode: FakeBugReporterMode = FakeBugReporterMode.Succes
         listener?.onProgress(50)
         delay(100)
         when (mode) {
-            FakeBugReporterMode.Success -> Unit
-            FakeBugReporterMode.Failure -> {
+            Mode.Success -> Unit
+            Mode.Failure -> {
                 listener?.onUploadFailed(A_FAILURE_REASON)
                 return
             }
-            FakeBugReporterMode.Cancel -> {
+            Mode.Cancel -> {
                 listener?.onUploadCancelled()
                 return
             }
@@ -64,9 +70,3 @@ class FakeBugReporter(val mode: FakeBugReporterMode = FakeBugReporterMode.Succes
         // No op
     }
 }
-
-enum class FakeBugReporterMode {
-    Success,
-    Failure,
-    Cancel
-}