From f1048f75783165e5028e9f1c5d24e853d5b5b440 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Thu, 9 Mar 2023 14:16:48 +0100
Subject: [PATCH] Remove duplicated Fake class.

---
 .../impl/bugreport/BugReportPresenterTest.kt  |  2 ++
 .../impl/bugreport/FakeScreenshotHolder.kt    | 30 -------------------
 .../test/rageshake/FakeRageshakeDataStore.kt  |  1 -
 .../test/screenshot/FakeScreenshotHolder.kt   |  1 -
 4 files changed, 2 insertions(+), 32 deletions(-)
 delete mode 100644 features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeScreenshotHolder.kt

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 9869698703..93ed963543 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
@@ -24,6 +24,8 @@ import app.cash.turbine.test
 import com.google.common.truth.Truth.assertThat
 import io.element.android.features.rageshake.test.crash.A_CRASH_DATA
 import io.element.android.features.rageshake.test.crash.FakeCrashDataStore
+import io.element.android.features.rageshake.test.screenshot.A_SCREENSHOT_URI
+import io.element.android.features.rageshake.test.screenshot.FakeScreenshotHolder
 import io.element.android.libraries.architecture.Async
 import io.element.android.libraries.matrix.test.A_FAILURE_REASON
 import kotlinx.coroutines.ExperimentalCoroutinesApi
diff --git a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeScreenshotHolder.kt b/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeScreenshotHolder.kt
deleted file mode 100644
index 0ed7714287..0000000000
--- a/features/rageshake/impl/src/test/kotlin/io/element/android/features/rageshake/impl/bugreport/FakeScreenshotHolder.kt
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * Copyright (c) 2023 New Vector Ltd
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package io.element.android.features.rageshake.impl.bugreport
-
-import android.graphics.Bitmap
-import io.element.android.features.rageshake.api.screenshot.ScreenshotHolder
-
-const val A_SCREENSHOT_URI = "file://content/uri"
-
-class FakeScreenshotHolder(private val screenshotUri: String? = null) : ScreenshotHolder {
-    override fun writeBitmap(data: Bitmap) = Unit
-
-    override fun getFileUri() = screenshotUri
-
-    override fun reset() = Unit
-}
diff --git a/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/rageshake/FakeRageshakeDataStore.kt b/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/rageshake/FakeRageshakeDataStore.kt
index 7bf15e982a..a250e50361 100644
--- a/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/rageshake/FakeRageshakeDataStore.kt
+++ b/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/rageshake/FakeRageshakeDataStore.kt
@@ -22,7 +22,6 @@ import kotlinx.coroutines.flow.MutableStateFlow
 
 const val A_SENSITIVITY = 1f
 
-// TODO Remove this duplicated class when we will rework modules.
 class FakeRageshakeDataStore(
     isEnabled: Boolean = true,
     sensitivity: Float = A_SENSITIVITY,
diff --git a/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/screenshot/FakeScreenshotHolder.kt b/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/screenshot/FakeScreenshotHolder.kt
index bb4d25737a..5e45960b28 100644
--- a/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/screenshot/FakeScreenshotHolder.kt
+++ b/features/rageshake/test/src/main/kotlin/io/element/android/features/rageshake/test/screenshot/FakeScreenshotHolder.kt
@@ -21,7 +21,6 @@ import io.element.android.features.rageshake.api.screenshot.ScreenshotHolder
 
 const val A_SCREENSHOT_URI = "file://content/uri"
 
-// TODO Remove this duplicated class when we will rework modules.
 class FakeScreenshotHolder(private val screenshotUri: String? = null) : ScreenshotHolder {
     override fun writeBitmap(data: Bitmap) = Unit