Merge pull request android#16 from googlesamples/todo-mvp-loaders_fixRepoLocalCache

David González · David González · commit abf5944d3d3a · 2016-03-31T12:06:04.000+01:00
Adds unit test from todo-mvp
diff --git a/todoapp/app/src/test/java/com/example/android/architecture/blueprints/todoapp/data/source/TasksRepositoryTest.java b/todoapp/app/src/test/java/com/example/android/architecture/blueprints/todoapp/data/source/TasksRepositoryTest.java
@@ -16,6 +16,17 @@
 
 package com.example.android.architecture.blueprints.todoapp.data.source;
 
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.nullValue;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertThat;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.Matchers.any;
+import static org.mockito.Matchers.eq;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
 import android.content.Context;
 
 import com.example.android.architecture.blueprints.todoapp.data.Task;
@@ -31,15 +42,6 @@
 
 import java.util.List;
 
-import static org.hamcrest.CoreMatchers.is;
-import static org.hamcrest.CoreMatchers.nullValue;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertThat;
-import static org.junit.Assert.assertTrue;
-import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
-
 /**
  * Unit tests for the implementation of the in-memory repository with cache.
  */
@@ -308,6 +310,21 @@ public void getTaskWithBothDataSourcesUnavailable_firesOnDataUnavailable() {
         assertThat(task, is(nullValue()));
     }
 
+    @Test
+    public void getTasks_refreshesLocalDataSource() {
+        // Mark cache as dirty to force a reload of data from remote data source.
+        mTasksRepository.refreshTasks();
+
+        // Make the remote data source return data
+        setTasksAvailable(mTasksRemoteDataSource, TASKS);
+
+        // When calling getTasks in the repository
+        mTasksRepository.getTasks();
+
+        // Verify that the data fetched from the remote data source was saved in local.
+        verify(mTasksLocalDataSource, times(TASKS.size())).saveTask(any(Task.class));
+    }
+
     /**
     * Convenience method that issues two calls to the tasks repository
     */
diff --git a/todoapp/build.gradle b/todoapp/build.gradle
@@ -3,7 +3,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:2.0.0-beta6'
+        classpath 'com.android.tools.build:gradle:2.1.0-alpha4'
 
         // NOTE: Do not place your application dependencies here; they belong
         // in the individual module build.gradle files

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@ buildscript {`
`3`	`3`	`jcenter()`
`4`	`4`	`}`
`5`	`5`	`dependencies {`
`6`		`- classpath 'com.android.tools.build:gradle:2.0.0-beta6'`
	`6`	`+ classpath 'com.android.tools.build:gradle:2.1.0-alpha4'`
`7`	`7`
`8`	`8`	`// NOTE: Do not place your application dependencies here; they belong`
`9`	`9`	`// in the individual module build.gradle files`