Merge into main

Author: JoseAlcerreca

.github/workflows/blueprints.yaml

@@ -3,14 +3,16 @@ name: blueprints
 on:
   push:
     branches:
-      - dev-compose
+      - main
+      - views
   pull_request:
     branches:
-      - dev-compose
+      - main
+      - views
 
 jobs:
   build:
-    runs-on: macos-latest
+    runs-on: macos-latest # Needed for emulators
     timeout-minutes: 45
 
     steps:
@@ -48,132 +50,21 @@ jobs:
       - name: Robolectric Prod
         run: ./gradlew spotlessCheck assembleMockDebug assembleProdDebug testMockDebugUnitTest testProdDebugUnitTest --stacktrace
 
-#  test:
-#    needs: build
-#    runs-on: macOS-latest # enables hardware acceleration in the virtual machine
-#    timeout-minutes: 30
-#    strategy:
-#      matrix:
-#        api-level: [23, 29]
-#
-#    steps:
-#      - name: Checkout
-#        uses: actions/checkout@v2
-#
-#      - name: Copy CI gradle.properties
-#        run: mkdir -p ~/.gradle ; cp .github/ci-gradle.properties ~/.gradle/gradle.properties
-#
-#      - name: Set up JDK 11
-#        uses: actions/setup-java@v1
-#        with:
-#          java-version: 11
-#
-
-#
-#      - name: Upload test reports
-#        if: always()
-#        uses: actions/upload-artifact@v2
-#        with:
-#          name: test-reports
-#          path: app/build/reports/
-
-#
-#  gradleManagedVirtualDevicesTest:
-#    needs: build
-#    runs-on: macos-latest
-#    timeout-minutes: 30
-#
-#    steps:
-#      - name: Checkout
-#        uses: actions/checkout@v2
-#
-#      - name: Copy CI gradle.properties
-#        run: mkdir -p ~/.gradle ; cp .github/ci-gradle.properties ~/.gradle/gradle.properties
-#
-#      - name: Set up JDK 11
-#        uses: actions/setup-java@v1
-#        with:
-#          java-version: 11
-
       # Needed to accept licenses
       - name: Setup Android SDK
         uses: android-actions/setup-android@v2
 
-#      - name: Generate cache key for Gradle cache
-#        run: ./scripts/checksum.sh $SAMPLE_PATH checksum.txt
-#
-#      - uses: actions/cache@v2
-#        with:
-#          path: |
-#            ~/.gradle/caches/modules-*
-#            ~/.gradle/caches/jars-*
-#            ~/.gradle/caches/build-cache-*
-#          key: gradle-${{ hashFiles('checksum.txt') }}
-
-#      - name: Cache pixel2api30 system image
-#        uses: actions/cache@v2
-#        with:
-#          path: |
-#            ~/.android/gradle/avd/dev30_aosp-atd_x86_Pixel_2.avd.*
-#          key: pixel2api30
-#
-#      - name: Cache pixel2api27 system image
-#        uses: actions/cache@v2
-#        with:
-#          path: ~/.android/gradle/avd/dev27_aosp_x86_Pixel_2.*
-#          key: pixel2api27
-#
-#      - name: Cache nexus9api29 system image
-#        uses: actions/cache@v2
-#        with:
-#          path: ~/.android/gradle/avd/dev29_aosp_x86_Nexus_9.*
-#          key: nexus9api29
-
-#      - name: Run all tests pixel 2 api 27 ATD
-#        working-directory: ${{ env.SAMPLE_PATH }}
-#        run: ./gradlew pixel2api27atdMockDebugAndroidTest
-#
-#      - name: Run all tests nexus 9 api 29 ATD
-#        working-directory: ${{ env.SAMPLE_PATH }}
-#        run: ./gradlew nexus9api29atdMockDebugAndroidTest
-
       - name: Compile AndroidTests Mock
         run: ./gradlew compileMockDebugAndroidTestKotlin
 
       - name: Run all tests pixel 2 api 30 ATD
         working-directory: .
         run: ./gradlew -Pandroid.sdk.channel=3 -Pandroid.experimental.androidTest.numManagedDeviceShards=1 -Pandroid.testoptions.manageddevices.emulator.gpu="swiftshader_indirect" pixel2api30atdMockDebugAndroidTest
 
-      - name: Run all tests pixel 2 api 30 ATD again
-        working-directory: .
-        run: ./gradlew -Pandroid.sdk.channel=3 -Pandroid.experimental.androidTest.numManagedDeviceShards=1 -Pandroid.testoptions.manageddevices.emulator.gpu="swiftshader_indirect" pixel2api30atdMockDebugAndroidTest
-
-      - name: Run all tests pixel 2 api 30 non-atd
+      - name: Run all tests pixel 2 api 30 non-ATD
         working-directory: .
         run: ./gradlew -Pandroid.sdk.channel=3 -Pandroid.experimental.androidTest.numManagedDeviceShards=1 -Pandroid.testoptions.manageddevices.emulator.gpu="swiftshader_indirect" pixel2api30MockDebugAndroidTest
 
-      - name: Run all tests pixel 2 api 30 non-atd again
-        working-directory: .
-        run: ./gradlew -Pandroid.sdk.channel=3 -Pandroid.experimental.androidTest.numManagedDeviceShards=1 -Pandroid.testoptions.manageddevices.emulator.gpu="swiftshader_indirect" pixel2api30MockDebugAndroidTest
-
-      - name: Run instrumentation tests API 30
-        uses: reactivecircus/android-emulator-runner@v2
-        with:
-          api-level: 30
-          arch: x86_64
-          disable-animations: true
-          script: ./gradlew app:cMDAT --stacktrace
-
-      - name: Run instrumentation tests API 30 ATD
-        uses: reactivecircus/android-emulator-runner@v2
-        with:
-          api-level: 30
-          arch: x86
-          channel: canary
-          target: aosp_atd
-          disable-animations: true
-          script: ./gradlew app:cMDAT --stacktrace
-
       - name: Upload build reports
         if: always()
         uses: actions/upload-artifact@v2
@@ -187,10 +78,3 @@ jobs:
         with:
           name: outputs
           path: app/build/outputs/
-
-#      - name: Upload test reports
-#        if: always()
-#        uses: actions/upload-artifact@v2
-#        with:
-#          name: test-reports
-#          path: ./app/build/reports/androidTests/managedDevice/

app/src/sharedTest/java/com/example/android/architecture/blueprints/todoapp/MainCoroutineRule.kt

@@ -16,45 +16,35 @@
 
 package com.example.android.architecture.blueprints.todoapp
 
-import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.ExperimentalCoroutinesApi
-import kotlinx.coroutines.test.TestCoroutineScope
+import kotlinx.coroutines.test.TestDispatcher
+import kotlinx.coroutines.test.UnconfinedTestDispatcher
 import kotlinx.coroutines.test.resetMain
 import kotlinx.coroutines.test.setMain
 import org.junit.rules.TestWatcher
 import org.junit.runner.Description
-import kotlin.coroutines.ContinuationInterceptor
 
 /**
- * Sets the main coroutines dispatcher to a [TestCoroutineScope] for unit testing. A
- * [TestCoroutineScope] provides control over the execution of coroutines.
+ * Sets the main coroutines dispatcher to a [TestDispatcher] for unit testing.
  *
  * Declare it as a JUnit Rule:
  *
  * ```
  * @get:Rule
- * var mainCoroutineRule = MainCoroutineRule()
+ * val mainCoroutineRule = MainCoroutineRule()
  * ```
  *
- * Use it directly as a [TestCoroutineScope]:
- *
- * ```
- * mainCoroutineRule.pauseDispatcher()
- * ...
- * mainCoroutineRule.resumeDispatcher()
- * ...
- * mainCoroutineRule.runBlockingTest { }
- * ...
- *
- * ```
+ * Then, use `runTest` to execute your tests.
  */
 @ExperimentalCoroutinesApi
-class MainCoroutineRule : TestWatcher(), TestCoroutineScope by TestCoroutineScope() {
+class MainCoroutineRule(
+    val testDispatcher: TestDispatcher = UnconfinedTestDispatcher()
+) : TestWatcher() {
 
     override fun starting(description: Description?) {
         super.starting(description)
-        Dispatchers.setMain(this.coroutineContext[ContinuationInterceptor] as CoroutineDispatcher)
+        Dispatchers.setMain(testDispatcher)
     }
 
     override fun finished(description: Description?) {

app/src/sharedTest/java/com/example/android/architecture/blueprints/todoapp/data/source/local/TasksDaoTest.kt

@@ -24,7 +24,7 @@ import androidx.test.filters.SmallTest
 import com.example.android.architecture.blueprints.todoapp.MainCoroutineRule
 import com.example.android.architecture.blueprints.todoapp.data.Task
 import kotlinx.coroutines.ExperimentalCoroutinesApi
-import kotlinx.coroutines.test.runBlockingTest
+import kotlinx.coroutines.test.runTest
 import org.hamcrest.CoreMatchers.`is`
 import org.hamcrest.CoreMatchers.notNullValue
 import org.hamcrest.MatcherAssert.assertThat
@@ -44,11 +44,11 @@ class TasksDaoTest {
     // Set the main coroutines dispatcher for unit testing.
     @ExperimentalCoroutinesApi
     @get:Rule
-    var mainCoroutineRule = MainCoroutineRule()
+    val mainCoroutineRule = MainCoroutineRule()
 
     // Executes each task synchronously using Architecture Components.
     @get:Rule
-    var instantExecutorRule = InstantTaskExecutorRule()
+    val instantExecutorRule = InstantTaskExecutorRule()
 
     @Before
     fun initDb() {
@@ -64,7 +64,7 @@ class TasksDaoTest {
     fun closeDb() = database.close()
 
     @Test
-    fun insertTaskAndGetById() = runBlockingTest {
+    fun insertTaskAndGetById() = runTest {
         // GIVEN - insert a task
         val task = Task("title", "description")
         database.taskDao().insertTask(task)
@@ -81,7 +81,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun insertTaskReplacesOnConflict() = runBlockingTest {
+    fun insertTaskReplacesOnConflict() = runTest {
         // Given that a task is inserted
         val task = Task("title", "description")
         database.taskDao().insertTask(task)
@@ -99,7 +99,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun insertTaskAndGetTasks() = runBlockingTest {
+    fun insertTaskAndGetTasks() = runTest {
         // GIVEN - insert a task
         val task = Task("title", "description")
         database.taskDao().insertTask(task)
@@ -116,7 +116,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun updateTaskAndGetById() = runBlockingTest {
+    fun updateTaskAndGetById() = runTest {
         // When inserting a task
         val originalTask = Task("title", "description")
         database.taskDao().insertTask(originalTask)
@@ -134,7 +134,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun updateCompletedAndGetById() = runBlockingTest {
+    fun updateCompletedAndGetById() = runTest {
         // When inserting a task
         val task = Task("title", "description", true)
         database.taskDao().insertTask(task)
@@ -151,7 +151,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun deleteTaskByIdAndGettingTasks() = runBlockingTest {
+    fun deleteTaskByIdAndGettingTasks() = runTest {
         // Given a task inserted
         val task = Task("title", "description")
         database.taskDao().insertTask(task)
@@ -165,7 +165,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun deleteTasksAndGettingTasks() = runBlockingTest {
+    fun deleteTasksAndGettingTasks() = runTest {
         // Given a task inserted
         database.taskDao().insertTask(Task("title", "description"))
 
@@ -178,7 +178,7 @@ class TasksDaoTest {
     }
 
     @Test
-    fun deleteCompletedTasksAndGettingTasks() = runBlockingTest {
+    fun deleteCompletedTasksAndGettingTasks() = runTest {
         // Given a completed task inserted
         database.taskDao().insertTask(Task("completed", "task", true))
 

app/src/sharedTest/java/com/example/android/architecture/blueprints/todoapp/data/source/local/TasksLocalDataSourceTest.kt

@@ -27,7 +27,7 @@ import com.example.android.architecture.blueprints.todoapp.data.source.TasksData
 import com.example.android.architecture.blueprints.todoapp.data.succeeded
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.ExperimentalCoroutinesApi
-import kotlinx.coroutines.test.runBlockingTest
+import kotlinx.coroutines.test.runTest
 import org.hamcrest.CoreMatchers.`is`
 import org.hamcrest.MatcherAssert.assertThat
 import org.junit.After
@@ -50,11 +50,11 @@ class TasksLocalDataSourceTest {
     // Set the main coroutines dispatcher for unit testing.
     @ExperimentalCoroutinesApi
     @get:Rule
-    var mainCoroutineRule = MainCoroutineRule()
+    val mainCoroutineRule = MainCoroutineRule()
 
     // Executes each task synchronously using Architecture Components.
     @get:Rule
-    var instantExecutorRule = InstantTaskExecutorRule()
+    val instantExecutorRule = InstantTaskExecutorRule()
 
     @Before
     fun setup() {
@@ -75,7 +75,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun saveTask_retrievesTask() = runBlockingTest {
+    fun saveTask_retrievesTask() = runTest {
         // GIVEN - a new task saved in the database
         val newTask = Task("title", "description", true)
         localDataSource.saveTask(newTask)
@@ -92,7 +92,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun completeTask_retrievedTaskIsComplete() = runBlockingTest {
+    fun completeTask_retrievedTaskIsComplete() = runTest {
         // Given a new task in the persistent repository
         val newTask = Task("title")
         localDataSource.saveTask(newTask)
@@ -109,7 +109,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun activateTask_retrievedTaskIsActive() = runBlockingTest {
+    fun activateTask_retrievedTaskIsActive() = runTest {
         // Given a new completed task in the persistent repository
         val newTask = Task("Some title", "Some description", true)
         localDataSource.saveTask(newTask)
@@ -127,7 +127,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun clearCompletedTask_taskNotRetrievable() = runBlockingTest {
+    fun clearCompletedTask_taskNotRetrievable() = runTest {
         // Given 2 new completed tasks and 1 active task in the persistent repository
         val newTask1 = Task("title")
         val newTask2 = Task("title2")
@@ -153,7 +153,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun deleteAllTasks_emptyListOfRetrievedTask() = runBlockingTest {
+    fun deleteAllTasks_emptyListOfRetrievedTask() = runTest {
         // Given a new task in the persistent repository and a mocked callback
         val newTask = Task("title")
 
@@ -168,7 +168,7 @@ class TasksLocalDataSourceTest {
     }
 
     @Test
-    fun getTasks_retrieveSavedTasks() = runBlockingTest {
+    fun getTasks_retrieveSavedTasks() = runTest {
         // Given 2 new tasks in the persistent repository
         val newTask1 = Task("title")
         val newTask2 = Task("title")