Skip to content

Commit 4cc6703

Browse files
committed
Remove PIN Migration
This is no longer required
1 parent 0a8e038 commit 4cc6703

File tree

5 files changed

+1
-117
lines changed

5 files changed

+1
-117
lines changed

app/src/main/kotlin/com/darkrockstudios/app/securecamera/AppModule.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@ val appModule = module {
8181
factoryOf(::CreatePinUseCase)
8282
factoryOf(::PinSizeUseCase)
8383
factoryOf(::RemovePoisonPillIUseCase)
84-
factoryOf(::MigratePinHash)
8584
factoryOf(::InvalidateSessionUseCase)
8685
factoryOf(::AddDecoyPhotoUseCase)
8786

app/src/main/kotlin/com/darkrockstudios/app/securecamera/usecases/MigratePinHash.kt

Lines changed: 0 additions & 107 deletions
This file was deleted.

app/src/main/kotlin/com/darkrockstudios/app/securecamera/usecases/VerifyPinUseCase.kt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,9 @@ class VerifyPinUseCase(
1010
private val imageManager: SecureImageRepository,
1111
private val pinRepository: PinRepository,
1212
private val encryptionScheme: EncryptionScheme,
13-
private val migratePinHash: MigratePinHash,
1413
private val authorizePinUseCase: AuthorizePinUseCase,
1514
) {
1615
suspend fun verifyPin(pin: String): Boolean {
17-
migratePinHash.runMigration(pin)
18-
1916
if (pinRepository.hasPoisonPillPin() && pinRepository.verifyPoisonPillPin(pin)) {
2017
encryptionScheme.activatePoisonPill(oldPin = pinRepository.getHashedPin())
2118
imageManager.activatePoisonPill()

app/src/test/kotlin/com/darkrockstudios/app/securecamera/auth/AuthorizationManagerTest.kt renamed to app/src/test/kotlin/com/darkrockstudios/app/securecamera/auth/AuthorizationRepositoryTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ import kotlin.time.Duration.Companion.seconds
3030
import kotlin.time.Instant
3131

3232
@ExperimentalCoroutinesApi
33-
class AuthorizationManagerTest {
33+
class AuthorizationRepositoryTest {
3434

3535
private lateinit var context: Context
3636
private lateinit var preferencesManager: AppSettingsDataSource

app/src/test/kotlin/com/darkrockstudios/app/securecamera/usecases/VerifyPinUseCaseTest.kt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,27 +20,22 @@ class VerifyPinUseCaseTest {
2020
private lateinit var pinRepository: PinRepository
2121
private lateinit var verifyPinUseCase: VerifyPinUseCase
2222
private lateinit var encryptionScheme: EncryptionScheme
23-
private lateinit var migratePinHash: MigratePinHash
2423
private lateinit var authorizePinUseCase: AuthorizePinUseCase
2524

2625
@Before
2726
fun setup() {
2827
authManager = mockk()
2928
imageManager = mockk()
3029
pinRepository = mockk()
31-
migratePinHash = mockk()
3230
authorizePinUseCase = mockk()
3331
encryptionScheme = mockk(relaxed = true)
3432
verifyPinUseCase = VerifyPinUseCase(
3533
authManager = authManager,
3634
imageManager = imageManager,
3735
pinRepository = pinRepository,
3836
encryptionScheme = encryptionScheme,
39-
migratePinHash = migratePinHash,
4037
authorizePinUseCase = authorizePinUseCase,
4138
)
42-
43-
coEvery { migratePinHash.runMigration(any()) } just Runs
4439
}
4540

4641
@Test

0 commit comments

Comments
 (0)