Browse Source

Move FakeBugReporterMode to FakeBugReporter.Mode

pull/2900/head
Benoit Marty 4 months ago committed by Benoit Marty
parent
commit
f930796f57
  1. 6
      features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt
  2. 20
      features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeBugReporter.kt

6
features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/BugReportPresenterTest.kt

@ -144,7 +144,7 @@ class BugReportPresenterTest { @@ -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 { @@ -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 { @@ -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),
)

20
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 @@ -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 @@ -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 @@ -64,9 +70,3 @@ class FakeBugReporter(val mode: FakeBugReporterMode = FakeBugReporterMode.Succes
// No op
}
}
enum class FakeBugReporterMode {
Success,
Failure,
Cancel
}

Loading…
Cancel
Save