Merge pull request #39 from OpenSRP/38-refactor-check-has-initialized-indicators

38 Refactor check has initialized indicators

Author: allan-on

gradle.properties

@@ -1,4 +1,4 @@
-VERSION_NAME=0.0.5-SNAPSHOT
+VERSION_NAME=0.0.6-SNAPSHOT
 VERSION_CODE=1
 GROUP=org.smartregister
 POM_SETTING_DESCRIPTION=OpenSRP Client Reporting Library

opensrp-reporting/src/main/java/org/smartregister/reporting/ReportingLibrary.java

@@ -187,18 +187,21 @@ private boolean hasInitializedIndicators(SQLiteDatabase sqLiteDatabase) {
         if (!appOnDebugMode && (!isAppUpdated() || isIndicatorsInitialized())) {
             return true;
         }
+        truncateIndicatorDefinitionTables(sqLiteDatabase);
+        return false;
+    }
 
+    public void truncateIndicatorDefinitionTables(SQLiteDatabase sqLiteDatabase) {
         if (sqLiteDatabase != null) {
             indicatorRepository.truncateTable(sqLiteDatabase);
             indicatorQueryRepository.truncateTable(sqLiteDatabase);
         } else {
             indicatorRepository.truncateTable();
             indicatorQueryRepository.truncateTable();
         }
-        return false;
     }
 
-    private void readConfigFile(String configFilePath, SQLiteDatabase sqLiteDatabase) {
+    public void readConfigFile(String configFilePath, SQLiteDatabase sqLiteDatabase) {
         initYamlIndicatorConfig();
         Iterable<Object> indicatorsFromFile = null;
         try {

opensrp-reporting/src/test/java/org/smartregister/reporting/ReportingLibraryTest.java

@@ -1,27 +1,45 @@
 package org.smartregister.reporting;
 
+import net.sqlcipher.database.SQLiteDatabase;
+
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.Mock;
+import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
-import org.powermock.modules.junit4.PowerMockRunner;
+import org.mockito.junit.MockitoJUnitRunner;
+import org.powermock.reflect.Whitebox;
 import org.smartregister.Context;
 import org.smartregister.commonregistry.CommonFtsObject;
+import org.smartregister.reporting.repository.IndicatorQueryRepository;
+import org.smartregister.reporting.repository.IndicatorRepository;
 import org.smartregister.repository.Repository;
 
 import static org.junit.Assert.assertNotNull;
 
-@RunWith(PowerMockRunner.class)
+@RunWith(MockitoJUnitRunner.class)
 public class ReportingLibraryTest {
 
     @Mock
     private Context context;
+
     @Mock
     private Repository repository;
+
+    @Mock
+    private IndicatorRepository indicatorRepository;
+
+    @Mock
+    private IndicatorQueryRepository indicatorQueryRepository;
+
+
     @Mock
     private CommonFtsObject commonFtsObject;
 
+    @Mock
+    private SQLiteDatabase sqliteDatabase;
+
     private int appVersion = 1;
     private int dbVersion = 1;
 
@@ -46,4 +64,18 @@ public void testThatAllRepositoriesAreInitialized() {
         assertNotNull(reportingLibrary.dailyIndicatorCountRepository());
 
     }
+
+    @Test
+    public void truncateIndicatorTablesWithDBInvokesRepositoryTruncate() {
+        ReportingLibrary.init(context, repository, commonFtsObject, appVersion, dbVersion);
+        ReportingLibrary reportingLibrary = ReportingLibrary.getInstance();
+        ReportingLibrary reportingLibrarySpy = Mockito.spy(reportingLibrary);
+        // Magic
+        Whitebox.setInternalState(reportingLibrarySpy, "indicatorRepository", indicatorRepository);
+        Whitebox.setInternalState(reportingLibrarySpy, "indicatorQueryRepository", indicatorQueryRepository);
+
+        reportingLibrarySpy.truncateIndicatorDefinitionTables(sqliteDatabase);
+        Mockito.verify(indicatorRepository, Mockito.times(1)).truncateTable(sqliteDatabase);
+        Mockito.verify(indicatorQueryRepository, Mockito.times(1)).truncateTable(sqliteDatabase);
+    }
 }