Skip to content

Commit 4574d0d

Browse files
ambrose40ambrose40
committed
MOPPAND-1716 CDOC2 settings fixes.
1 parent 964c01c commit 4574d0d

File tree

17 files changed

+105
-94
lines changed

17 files changed

+105
-94
lines changed

‎app/src/androidTest/kotlin/ee/ria/DigiDoc/domain/preferences/DataStoreTest.kt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import android.content.Context
2525
import android.content.SharedPreferences
2626
import androidx.preference.PreferenceManager
2727
import androidx.test.platform.app.InstrumentationRegistry
28+
import ee.ria.DigiDoc.common.Constant.Defaults.DEFAULT_UUID_VALUE
2829
import ee.ria.DigiDoc.common.preferences.EncryptedPreferences
2930
import ee.ria.DigiDoc.configuration.repository.ConfigurationRepository
3031
import ee.ria.DigiDoc.domain.model.methods.SigningMethod
@@ -36,7 +37,6 @@ import ee.ria.DigiDoc.libdigidoclib.init.Initialization
3637
import ee.ria.DigiDoc.libdigidoclib.init.LibdigidocLibraryLoader
3738
import ee.ria.DigiDoc.network.proxy.ProxySetting
3839
import ee.ria.DigiDoc.network.siva.SivaSetting
39-
import ee.ria.DigiDoc.utils.Constant.Defaults.DEFAULT_UUID_VALUE
4040
import kotlinx.coroutines.runBlocking
4141
import org.junit.Assert.assertEquals
4242
import org.junit.Assert.assertFalse
@@ -383,14 +383,14 @@ class DataStoreTest {
383383
fun dataStore_setCdocSetting_success() {
384384
dataStore.setCdocSetting(CDOCSetting.CDOC2)
385385

386-
val result = dataStore.getCdocSetting()
386+
val result = dataStore.getCdocSetting(false)
387387

388388
assertEquals(CDOCSetting.CDOC2, result)
389389
}
390390

391391
@Test
392392
fun dataStore_getCdocSetting_success() {
393-
val result = dataStore.getCdocSetting()
393+
val result = dataStore.getCdocSetting(false)
394394

395395
assertEquals(CDOCSetting.CDOC1, result)
396396
}

‎app/src/androidTest/kotlin/ee/ria/DigiDoc/viewmodel/DiagnosticsViewModelTest.kt‎

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import androidx.lifecycle.MutableLiveData
3131
import androidx.test.platform.app.InstrumentationRegistry
3232
import com.google.gson.Gson
3333
import ee.ria.DigiDoc.R
34+
import ee.ria.DigiDoc.common.Constant.Defaults.DEFAULT_UUID_VALUE
3435
import ee.ria.DigiDoc.configuration.ConfigurationProperty
3536
import ee.ria.DigiDoc.configuration.ConfigurationSignatureVerifierImpl
3637
import ee.ria.DigiDoc.configuration.loader.ConfigurationLoader
@@ -47,7 +48,6 @@ import ee.ria.DigiDoc.libdigidoclib.init.Initialization
4748
import ee.ria.DigiDoc.libdigidoclib.init.LibdigidocLibraryLoader
4849
import ee.ria.DigiDoc.network.proxy.ManualProxy
4950
import ee.ria.DigiDoc.network.proxy.ProxySetting
50-
import ee.ria.DigiDoc.utils.Constant.Defaults.DEFAULT_UUID_VALUE
5151
import ee.ria.DigiDoc.utilsLib.file.FileUtil
5252
import junit.framework.TestCase.assertEquals
5353
import junit.framework.TestCase.assertTrue
@@ -120,6 +120,7 @@ class DiagnosticsViewModelTest {
120120
fetch = "https://cdoc2.id.ee:8444",
121121
),
122122
),
123+
cdoc2Default = false,
123124
cdoc2UseKeyServer = false,
124125
cdoc2DefaultKeyServer = "00000000-0000-0000-0000-000000000000",
125126
)
@@ -289,7 +290,7 @@ class DiagnosticsViewModelTest {
289290
val diagnosticsFileName =
290291
"ria_digidoc_${viewModel.getAppVersion()}.${viewModel.getAppVersionCode()}_diagnostics.log"
291292
val diagnosticsFilePath: String = File(context.filesDir.path, "diagnostics").path
292-
val resultFile = viewModel.createDiagnosticsFile(context)
293+
val resultFile = viewModel.createDiagnosticsFile(context, configurationProvider)
293294

294295
assertEquals(File(diagnosticsFilePath, diagnosticsFileName).path, resultFile.path)
295296
assertTrue(resultFile.exists())
@@ -375,7 +376,7 @@ class DiagnosticsViewModelTest {
375376
dataStore.setCdocSetting(CDOCSetting.CDOC2)
376377
viewModel.updatedConfiguration = MutableLiveData(configurationProvider)
377378

378-
val isCdoc2Selected = viewModel.isCdoc2Selected()
379+
val isCdoc2Selected = viewModel.isCdoc2Selected(configurationProvider)
379380

380381
assertTrue(isCdoc2Selected)
381382
}
@@ -385,7 +386,7 @@ class DiagnosticsViewModelTest {
385386
dataStore.setCdocSetting(CDOCSetting.CDOC1)
386387
viewModel.updatedConfiguration = MutableLiveData(configurationProvider)
387388

388-
val isCdoc2Selected = viewModel.isCdoc2Selected()
389+
val isCdoc2Selected = viewModel.isCdoc2Selected(configurationProvider)
389390

390391
assertFalse(isCdoc2Selected)
391392
}
@@ -396,7 +397,7 @@ class DiagnosticsViewModelTest {
396397
dataStore.setUseOnlineEncryption(true)
397398
viewModel.updatedConfiguration = MutableLiveData(configurationProvider)
398399

399-
val isCdoc2KeyServerUsed = viewModel.isCdoc2KeyServerUsed()
400+
val isCdoc2KeyServerUsed = viewModel.isCdoc2KeyServerUsed(configurationProvider)
400401

401402
assertTrue(isCdoc2KeyServerUsed)
402403
}
@@ -407,7 +408,7 @@ class DiagnosticsViewModelTest {
407408
dataStore.setUseOnlineEncryption(false)
408409
viewModel.updatedConfiguration = MutableLiveData(configurationProvider)
409410

410-
val isCdoc2KeyServerUsed = viewModel.isCdoc2KeyServerUsed()
411+
val isCdoc2KeyServerUsed = viewModel.isCdoc2KeyServerUsed(configurationProvider)
411412

412413
assertFalse(isCdoc2KeyServerUsed)
413414
}
@@ -418,11 +419,12 @@ class DiagnosticsViewModelTest {
418419
dataStore.setCDOC2UUID(DEFAULT_UUID_VALUE)
419420
viewModel.updatedConfiguration = MutableLiveData(configurationProvider)
420421

421-
val cdoc2KeyServerUUID = viewModel.getCdoc2KeyServerUUID()
422+
val cdoc2KeyServerUUID = viewModel.getCdoc2KeyServerUUID(configurationProvider)
422423

423424
assertEquals(DEFAULT_UUID_VALUE, cdoc2KeyServerUUID)
424425
}
425426

427+
@Suppress("SameParameterValue")
426428
private fun createTempFileWithStringContent(
427429
filename: String,
428430
content: String,

‎app/src/androidTest/kotlin/ee/ria/DigiDoc/viewmodel/MobileIdViewModelTest.kt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ class MobileIdViewModelTest {
141141
fetch = "https://cdoc2.id.ee:8444",
142142
),
143143
),
144+
cdoc2Default = false,
144145
cdoc2UseKeyServer = false,
145146
cdoc2DefaultKeyServer = "00000000-0000-0000-0000-000000000000",
146147
)

‎app/src/androidTest/kotlin/ee/ria/DigiDoc/viewmodel/SmartIdViewModelTest.kt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,7 @@ class SmartIdViewModelTest {
148148
fetch = "https://cdoc2.id.ee:8444",
149149
),
150150
),
151+
cdoc2Default = false,
151152
cdoc2UseKeyServer = false,
152153
cdoc2DefaultKeyServer = "00000000-0000-0000-0000-000000000000",
153154
)

‎app/src/main/kotlin/ee/ria/DigiDoc/domain/preferences/DataStore.kt‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ import androidx.core.content.edit
2828
import androidx.preference.PreferenceManager
2929
import ee.ria.DigiDoc.R
3030
import ee.ria.DigiDoc.common.Constant.Crypto.DECRYPT_METHOD_SETTING
31+
import ee.ria.DigiDoc.common.Constant.Defaults.DEFAULT_UUID_VALUE
3132
import ee.ria.DigiDoc.common.Constant.IS_CRASH_SENDING_ALWAYS_ENABLED
3233
import ee.ria.DigiDoc.common.Constant.KEY_LOCALE
3334
import ee.ria.DigiDoc.common.Constant.MyEID.IDENTIFICATION_METHOD_SETTING
@@ -43,7 +44,6 @@ import ee.ria.DigiDoc.domain.model.theme.ThemeSetting
4344
import ee.ria.DigiDoc.network.proxy.ManualProxy
4445
import ee.ria.DigiDoc.network.proxy.ProxySetting
4546
import ee.ria.DigiDoc.network.siva.SivaSetting
46-
import ee.ria.DigiDoc.utils.Constant.Defaults.DEFAULT_UUID_VALUE
4747
import ee.ria.DigiDoc.utils.snackbar.SnackBarManager.showMessage
4848
import ee.ria.DigiDoc.utilsLib.locale.LocaleUtil.getLocale
4949
import ee.ria.DigiDoc.utilsLib.logging.LoggingUtil.Companion.debugLog
@@ -341,10 +341,10 @@ class DataStore
341341
}
342342
}
343343

344-
fun getUseEncryption(): Boolean =
344+
fun getUseEncryption(defaultValue: Boolean): Boolean =
345345
preferences.getBoolean(
346346
resources.getString(ee.ria.DigiDoc.cryptolib.R.string.crypto_settings_use_cdoc2_encryption),
347-
false,
347+
defaultValue,
348348
)
349349

350350
fun setUseEncryption(useEncryption: Boolean) {
@@ -436,8 +436,8 @@ class DataStore
436436
}
437437
}
438438

439-
fun getCdocSetting(): CDOCSetting {
440-
if (getUseEncryption()) {
439+
fun getCdocSetting(defaultValue: Boolean): CDOCSetting {
440+
if (getUseEncryption(defaultValue)) {
441441
return CDOCSetting.CDOC2
442442
}
443443
return CDOCSetting.CDOC1

‎app/src/main/kotlin/ee/ria/DigiDoc/fragment/screen/DiagnosticsScreen.kt‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ fun DiagnosticsScreen(
256256
onClickItem = {
257257
try {
258258
val diagnosticsFile =
259-
diagnosticsViewModel.createDiagnosticsFile(context)
259+
diagnosticsViewModel.createDiagnosticsFile(context, currentConfiguration)
260260
actionFile = diagnosticsFile
261261
val saveIntent =
262262
Intent.createChooser(
@@ -518,7 +518,7 @@ fun DiagnosticsScreen(
518518
testTagsAsResourceId = true
519519
}.testTag("mainDiagnosticsCdoc2Default"),
520520
stringResource(id = R.string.main_diagnostics_cdoc2_default_title),
521-
diagnosticsViewModel.isCdoc2Selected().toString(),
521+
diagnosticsViewModel.isCdoc2Selected(currentConfiguration).toString(),
522522
)
523523
DiagnosticsText(
524524
modifier =
@@ -527,7 +527,7 @@ fun DiagnosticsScreen(
527527
testTagsAsResourceId = true
528528
}.testTag("mainDiagnosticsCdoc2UseKeyserver"),
529529
stringResource(id = R.string.main_diagnostics_cdoc2_use_keyserver_title),
530-
diagnosticsViewModel.isCdoc2KeyServerUsed().toString(),
530+
diagnosticsViewModel.isCdoc2KeyServerUsed(currentConfiguration).toString(),
531531
)
532532
DiagnosticsText(
533533
modifier =
@@ -536,7 +536,7 @@ fun DiagnosticsScreen(
536536
testTagsAsResourceId = true
537537
}.testTag("mainDiagnosticsCdoc2DefaultKeyserver"),
538538
stringResource(id = R.string.main_diagnostics_cdoc2_default_keyserver_title),
539-
diagnosticsViewModel.getCdoc2KeyServerUUID(),
539+
diagnosticsViewModel.getCdoc2KeyServerUUID(currentConfiguration),
540540
)
541541
HorizontalDivider(
542542
modifier =

0 commit comments

Comments
 (0)