Merge current work on tests with updates

CHANGELOG.md

@@ -1,6 +1,20 @@
 Changelog
 =========
 
+## Version 5.4.6
+
+_2022-05-11_
+
+* Update dependencies to stable version.
+* Fix some tests.
+* Fix Gradle scripts.
+
+## Version 5.4.5
+
+_2022-04-09_
+
+* Update dependencies to stable version.
+
 ## Version 5.4.4
 
 _2022-03-26_

README.md

@@ -44,13 +44,13 @@ Then add the following dependencies in your app `build.gradle` or `build.gradle.
 
 **Groovy**
 ```groovy
-debugImplementation "com.infinum.dbinspector:dbinspector:5.4.4"
-releaseImplementation "com.infinum.dbinspector:dbinspector-no-op:5.4.4"
+debugImplementation "com.infinum.dbinspector:dbinspector:5.4.6"
+releaseImplementation "com.infinum.dbinspector:dbinspector-no-op:5.4.6"
 ```
 **KotlinDSL**
 ```kotlin
-debugImplementation("com.infinum.dbinspector:dbinspector:5.4.4")
-releaseImplementation("com.infinum.dbinspector:dbinspector-no-op:5.4.4")
+debugImplementation("com.infinum.dbinspector:dbinspector:5.4.6")
+releaseImplementation("com.infinum.dbinspector:dbinspector-no-op:5.4.6")
 ```
 
 ### Usage

config.gradle

@@ -7,7 +7,7 @@ ext {
     ]
     releaseConfig = [
             "group"      : "com.infinum.dbinspector",
-            "version"    : "5.4.4",
-            "versionCode": 5 * 100 * 100 + 4 * 100 + 4
+            "version"    : "5.4.6",
+            "versionCode": 5 * 100 * 100 + 4 * 100 + 6
     ]
 }

dbinspector-no-op/build.gradle

@@ -27,6 +27,7 @@ android {
         }
     }
 
+    namespace releaseConfig.group
     resourcePrefix "dbinspector_"
 
     compileOptions {
@@ -37,7 +38,8 @@ android {
     kotlinOptions {
         jvmTarget = JavaVersion.VERSION_1_8.toString()
         freeCompilerArgs += [
-                '-Xexplicit-api=strict'
+                '-Xexplicit-api=strict',
+                '-Xjvm-default=all'
         ]
     }
 

dbinspector-no-op/src/main/AndroidManifest.xml

@@ -1,7 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:tools="http://schemas.android.com/tools"
-    package="com.infinum.dbinspector">
+    xmlns:tools="http://schemas.android.com/tools">
 
     <application>
 

dbinspector/build.gradle

@@ -30,9 +30,10 @@ android {
         }
     }
 
+    namespace releaseConfig.group
     resourcePrefix "dbinspector_"
 
-    lintOptions {
+    lint {
         disable 'RtlEnabled', 'VectorPath', 'IconLocation'
     }
 
@@ -44,8 +45,9 @@ android {
     kotlinOptions {
         jvmTarget = JavaVersion.VERSION_1_8.toString()
         freeCompilerArgs += [
-                '-Xopt-in=kotlinx.coroutines.ExperimentalCoroutinesApi',
-                '-Xexplicit-api=strict'
+                '-opt-in=kotlinx.coroutines.ExperimentalCoroutinesApi',
+                '-Xexplicit-api=strict',
+                '-Xjvm-default=all'
         ]
     }
 
@@ -122,11 +124,11 @@ protobuf {
 }
 
 kover {
-    enabled = true
+    disabled = false
     coverageEngine.set(kotlinx.kover.api.CoverageEngine.INTELLIJ)
     intellijEngineVersion.set(libs.versions.intellij.get())
     jacocoEngineVersion.set(libs.versions.jacoco.get())
-    generateReportOnCheck.set(true)
+    generateReportOnCheck = true
 }
 
 tasks.withType(Test) {

dbinspector/src/main/AndroidManifest.xml

@@ -1,7 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:tools="http://schemas.android.com/tools"
-    package="com.infinum.dbinspector">
+    xmlns:tools="http://schemas.android.com/tools">
 
     <application>
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/data/sources/local/proto/history/HistoryDataStoreTest.kt

@@ -27,14 +27,13 @@ internal class HistoryDataStoreTest : BaseTest() {
 
     @Test
     fun `Data source exposes datastore`() {
-        launch {
+        blockingTest {
             val store: DataStore<HistoryEntity> = get()
             val source = HistoryDataStore(store)
 
-            runBlockingTest {
-                val actual = source.store()
-                assertEquals(store, actual)
-            }
+            val actual = source.store()
+
+            assertEquals(store, actual)
         }
     }
 
@@ -51,11 +50,9 @@ internal class HistoryDataStoreTest : BaseTest() {
             coEvery { firstOrNull() } returns null
         }
 
-        launch {
-            runBlockingTest {
-                val actual = source.current()
-                assertEquals(expected, actual)
-            }
+        blockingTest {
+            val actual = source.current()
+            assertEquals(expected, actual)
         }
     }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/data/sources/local/proto/history/HistorySerializerTest.kt

@@ -40,7 +40,7 @@ internal class HistorySerializerTest : BaseTest() {
         mockkStatic(HistoryEntity::class)
         every { HistoryEntity.parseFrom(mockk<InputStream>()) } returns mockk()
 
-        launch {
+        test {
             serializer.readFrom(any())
         }
 
@@ -56,7 +56,7 @@ internal class HistorySerializerTest : BaseTest() {
         val serializer = HistorySerializer()
         val stream: OutputStream = mockk()
 
-        launch {
+        test {
             serializer.writeTo(given, stream)
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/data/sources/local/proto/settings/SettingsDataStoreTest.kt

@@ -27,14 +27,12 @@ internal class SettingsDataStoreTest : BaseTest() {
 
     @Test
     fun `Data source exposes datastore`() {
-        launch {
+        blockingTest {
             val store: DataStore<SettingsEntity> = get()
             val source = SettingsDataStore(store)
 
-            runBlockingTest {
-                val actual = source.store()
-                assertEquals(store, actual)
-            }
+            val actual = source.store()
+            assertEquals(store, actual)
         }
     }
 
@@ -51,11 +49,9 @@ internal class SettingsDataStoreTest : BaseTest() {
             coEvery { firstOrNull() } returns null
         }
 
-        launch {
-            runBlockingTest {
-                val actual = source.current()
-                assertEquals(expected, actual)
-            }
+        blockingTest {
+            val actual = source.current()
+            assertEquals(expected, actual)
         }
     }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/data/sources/local/proto/settings/SettingsSerializerTest.kt

@@ -40,7 +40,7 @@ internal class SettingsSerializerTest : BaseTest() {
         mockkStatic(SettingsEntity::class)
         every { SettingsEntity.parseFrom(mockk<InputStream>()) } returns mockk()
 
-        launch {
+        test {
             serializer.readFrom(any())
         }
 
@@ -56,7 +56,7 @@ internal class SettingsSerializerTest : BaseTest() {
         val serializer = SettingsSerializer()
         val stream: OutputStream = mockk()
 
-        launch {
+        test {
             serializer.writeTo(given, stream)
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/data/sources/memory/connection/AndroidConnectionSourceTest.kt

@@ -42,7 +42,7 @@ internal class AndroidConnectionSourceTest : BaseTest() {
 
         val source = AndroidConnectionSource()
 
-        launch {
+        test {
             source.openConnection(given)
         }
 
@@ -69,7 +69,7 @@ internal class AndroidConnectionSourceTest : BaseTest() {
 
         source.connectionPool[given] = expected
 
-        launch {
+        test {
             source.openConnection(given)
         }
 
@@ -102,7 +102,7 @@ internal class AndroidConnectionSourceTest : BaseTest() {
 
         source.connectionPool[given] = expected
 
-        launch {
+        test {
             source.closeConnection(given)
         }
 
@@ -126,7 +126,7 @@ internal class AndroidConnectionSourceTest : BaseTest() {
 
         val source = AndroidConnectionSource()
 
-        launch {
+        test {
             source.closeConnection(given)
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/ConnectionRepositoryTest.kt

@@ -38,7 +38,7 @@ internal class ConnectionRepositoryTest : BaseTest() {
         coEvery { control.mapper.invoke(any()) } returns mockk()
         coEvery { control.converter.invoke(any()) } returns ""
 
-        launch {
+        test {
             repository.open(any())
         }
 
@@ -61,7 +61,7 @@ internal class ConnectionRepositoryTest : BaseTest() {
         coEvery { control.mapper.invoke(any()) } returns mockk()
         coEvery { control.converter.invoke(any()) } returns ""
 
-        launch {
+        test {
             repository.close(any())
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/control/converters/ConnectionConverterTest.kt

@@ -16,17 +16,16 @@ internal class ConnectionConverterTest : BaseTest() {
 
     @Test
     fun `Default converts to data model with same value`() =
-        launch {
+        test {
             val given = mockk<ConnectionParameters> {
                 every { databasePath } returns "test.db"
             }
             val expected = "test.db"
 
             val converter = ConnectionConverter()
 
-            val actual = test {
-                converter(given)
-            }
+            val actual = converter(given)
+
             assertEquals(expected, actual)
         }
 }

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/control/mappers/ConnectionMapperTest.kt

@@ -16,14 +16,13 @@ internal class ConnectionMapperTest : BaseTest() {
 
     @Test
     fun `Default local values maps to default domain values`() =
-        launch {
+        test {
             val given: SQLiteDatabase = mockk()
             val expected = DatabaseConnection(database = given)
 
             val mapper = ConnectionMapper()
-            val actual = test {
-                mapper(given)
-            }
+
+            val actual = mapper(given)
 
             assertEquals(expected, actual)
         }

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/interactors/CloseConnectionInteractorTest.kt

@@ -28,7 +28,7 @@ internal class CloseConnectionInteractorTest : BaseTest() {
 
         coEvery { source.closeConnection(any()) } returns mockk()
 
-        launch {
+        test {
             interactor.invoke(any())
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/interactors/OpenConnectionInteractorTest.kt

@@ -28,7 +28,7 @@ internal class OpenConnectionInteractorTest : BaseTest() {
 
         coEvery { source.openConnection(any()) } returns mockk()
 
-        launch {
+        test {
             interactor.invoke(any())
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/usecases/CloseConnectionUseCaseTest.kt

@@ -28,7 +28,7 @@ internal class CloseConnectionUseCaseTest : BaseTest() {
 
         coEvery { repository.close(any()) } returns Unit
 
-        launch {
+        test {
             useCase.invoke(any())
         }
 

dbinspector/src/test/kotlin/com/infinum/dbinspector/domain/connection/usecases/OpenConnectionUseCaseTest.kt

@@ -28,7 +28,7 @@ internal class OpenConnectionUseCaseTest : BaseTest() {
 
         coEvery { repository.open(any()) } returns mockk()
 
-        launch {
+        test {
             useCase.invoke(any())
         }