activate periodic sync worker when startup

Author: johnnzhou

app/build.gradle

@@ -83,10 +83,9 @@ dependencies {
 
     def appCenterSdkVersion = '4.1.0'
     coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:1.2.2'
-//    implementation "androidx.work:work-runtime:2.5.0"
     implementation 'androidx.work:work-runtime-ktx:2.8.1'
-    implementation("androidx.hilt:hilt-work:1.0.0")
-    kapt("androidx.hilt:hilt-compiler:1.0.0")
+    implementation "androidx.hilt:hilt-work:1.0.0"
+    kapt "androidx.hilt:hilt-compiler:1.0.0"
 
     implementation fileTree(dir: 'libs', include: ['*.jar'])
     implementation 'androidx.appcompat:appcompat:1.3.0'
@@ -110,7 +109,7 @@ dependencies {
     implementation "androidx.compose.material3:material3:1.0.1"
     implementation "androidx.compose.material3:material3-window-size-class:1.0.1"
 
-    implementation "androidx.lifecycle:lifecycle-runtime-compose:2.6.0-alpha03"
+    implementation "androidx.lifecycle:lifecycle-runtime-compose:2.6.1"
 
     // start up runtime
     implementation "androidx.startup:startup-runtime:1.1.1"
@@ -133,13 +132,9 @@ dependencies {
 
 
     // third-party lib
-//    implementation "com.squareup.moshi:moshi:1.14.0"
-//    kapt "com.squareup.moshi:moshi-kotlin-codegen:1.14.0"
     implementation 'com.squareup.retrofit2:retrofit:2.9.0'
-//    implementation 'com.squareup.retrofit2:converter-moshi:2.9.0'
     implementation 'com.squareup.retrofit2:converter-scalars:2.9.0'
 
-    // TODO: remove okhttp in the future
     implementation "com.squareup.okhttp3:okhttp:$okhttp_version"
     implementation 'com.github.kongzue.DialogX:DialogX:0.0.43.beta13'
 

app/src/main/AndroidManifest.xml

@@ -29,6 +29,10 @@
         </intent>
     </queries>
 
+        <meta-data
+            android:name="com.google.android.gms.version"
+            android:value="@integer/google_play_services_version" />
+
     <application
         android:allowBackup="true"
         android:icon="@mipmap/icon"
@@ -50,29 +54,17 @@
                 android:resource="@xml/filepaths" />
         </provider>
 
-        <!-- If you want to disable android.startup completely. -->
-<!--        <provider-->
-<!--            android:name="androidx.startup.InitializationProvider"-->
-<!--            android:authorities="${applicationId}.androidx-startup"-->
-<!--            android:exported="false"-->
-<!--            tools:node="remove">-->
-<!--        </provider>-->
-
         <provider
             android:name="androidx.startup.InitializationProvider"
             android:authorities="${applicationId}.androidx-startup"
             android:exported="false"
             tools:node="merge">
             <meta-data
-                android:name="com.lcl.lclmeasurementtool.sync.SyncInitializer"
+                android:name="androidx.work.WorkManagerInitializer"
                 android:value="androidx.startup"
                 tools:node="remove" />
         </provider>
 
-        <meta-data
-            android:name="com.google.android.gms.version"
-            android:value="@integer/google_play_services_version" />
-
         <activity
             android:name=".MainActivity2"
             android:exported="true"

app/src/main/java/com/lcl/lclmeasurementtool/LCLApplication.kt

@@ -1,8 +1,10 @@
 package com.lcl.lclmeasurementtool
 
 import android.app.Application
+import android.util.Log
 import androidx.hilt.work.HiltWorkerFactory
 import androidx.work.Configuration
+import com.lcl.lclmeasurementtool.sync.Sync
 import dagger.hilt.android.HiltAndroidApp
 import javax.inject.Inject
 
@@ -14,14 +16,14 @@ class LCLApplication : Application(), Configuration.Provider {
     @Inject lateinit var workerFactory: HiltWorkerFactory
 
     override fun getWorkManagerConfiguration() = Configuration.Builder()
+                                                    .setMinimumLoggingLevel(Log.DEBUG)
                                                     .setWorkerFactory(workerFactory)
                                                     .build()
 
     override fun onCreate() {
         super.onCreate()
 
-        // TODO: uncomment the sync code below
         // Initialize Sync; the system responsible for keeping data in the app up to date.
-//        Sync.initialize(context = this)
+        Sync.initialize(context = this)
     }
 }

app/src/main/java/com/lcl/lclmeasurementtool/MainActivity2.kt

@@ -45,7 +45,7 @@ class MainActivity2 : ComponentActivity() {
     @Inject
     lateinit var simStateMonitor: SimStateMonitor
 
-    val viewModel: MainActivityViewModel by viewModels()
+    private val viewModel: MainActivityViewModel by viewModels()
 
     @OptIn(ExperimentalMaterial3WindowSizeClassApi::class)
     override fun onCreate(savedInstanceState: Bundle?) {
@@ -104,8 +104,8 @@ class MainActivity2 : ComponentActivity() {
         setContent {
             if (uiState == MainActivityUiState.Login) {
                 viewModel.setDeviceId(UUID.randomUUID().toString())
-//                Login(viewModel = viewModel)
-//                return@setContent
+                Login(viewModel = viewModel)
+                return@setContent
             }
             LCLApp(windowSizeClass = calculateWindowSizeClass(activity = this), networkMonitor, simStateMonitor)
         }

app/src/main/java/com/lcl/lclmeasurementtool/sync/SyncInitializer.kt

@@ -1,6 +1,7 @@
 package com.lcl.lclmeasurementtool.sync
 
 import android.content.Context
+import android.util.Log
 import androidx.startup.AppInitializer
 import androidx.startup.Initializer
 import androidx.work.ExistingPeriodicWorkPolicy
@@ -13,6 +14,7 @@ object Sync {
     // automatic initialization with Androidx Startup. It is called from the app module's
     // Application.onCreate() and should be only done once.
     fun initialize(context: Context) {
+        Log.d("Sync", "Sync.initialize called")
         AppInitializer.getInstance(context)
             .initializeComponent(SyncInitializer::class.java)
     }
@@ -22,13 +24,12 @@ internal const val SyncWorkName = "LCLDataReportSync"
 
 class SyncInitializer: Initializer<Sync> {
     override fun create(context: Context): Sync {
-        WorkManager.getInstance(context).apply {
-            enqueueUniquePeriodicWork(
+        WorkManager.getInstance(context).enqueueUniquePeriodicWork(
                 SyncWorkName,
                 ExistingPeriodicWorkPolicy.CANCEL_AND_REENQUEUE,
                 UploadWorker.periodicSyncWork()
             )
-        }
+        Log.d("SyncInitializer", "SyncInitializer initialize here!")
         return Sync
     }
 

app/src/main/java/com/lcl/lclmeasurementtool/sync/SyncUtil.kt

@@ -5,6 +5,7 @@ import retrofit2.HttpException
 import kotlin.coroutines.cancellation.CancellationException
 
 private suspend fun <T> suspendRunCatching(block: suspend () -> T): Result<T> = try {
+    Log.d("suspendRunCatching", "block ${block.javaClass.name} run successfully")
     Result.success(block())
 } catch (cancellationException: CancellationException) {
     throw cancellationException

app/src/main/java/com/lcl/lclmeasurementtool/sync/UploadWorker.kt

@@ -1,6 +1,7 @@
 package com.lcl.lclmeasurementtool.sync
 
 import android.content.Context
+import android.util.Log
 import androidx.hilt.work.HiltWorker
 import androidx.work.*
 import com.lcl.lclmeasurementtool.datastore.Dispatcher
@@ -18,7 +19,7 @@ import java.util.concurrent.TimeUnit
 
 @HiltWorker
 class UploadWorker @AssistedInject constructor(
-    @Assisted context: Context,
+    @Assisted private val context: Context,
     @Assisted workerParameters: WorkerParameters,
     private val signalStrengthRepository: SignalStrengthRepository,
     private val connectivityRepository: ConnectivityRepository,
@@ -33,22 +34,27 @@ class UploadWorker @AssistedInject constructor(
                     async { connectivityRepository.sync() }
                 ).all { it }
                 if (syncSuccessfully) {
+                    Log.d(TAG, "upload successfully")
                     return@withContext Result.success()
                 } else {
+                    Log.d(TAG, "upload failed. some sync work failed")
                     return@withContext Result.failure()
                 }
             } catch (e: Exception) {
+                Log.d(TAG, "upload failed. exception is $e")
                 Result.failure()
             }
         }
 
     companion object {
         fun periodicSyncWork() =
-            PeriodicWorkRequest.Builder(UploadWorker::class.java, 8, TimeUnit.HOURS)
-            .setExpedited(OutOfQuotaPolicy.RUN_AS_NON_EXPEDITED_WORK_REQUEST)
+            PeriodicWorkRequest.Builder(UploadWorker::class.java, 15, TimeUnit.MINUTES)
             .setConstraints(Constraints(requiredNetworkType = NetworkType.CONNECTED))
+            .setInitialDelay(5, TimeUnit.MINUTES)
             .setBackoffCriteria(BackoffPolicy.EXPONENTIAL, Duration.ofMinutes(30))
             .build()
+
+        const val TAG = "UploadWorker"
     }
 
 }

app/src/main/java/com/lcl/lclmeasurementtool/ui/HistoryScreen.kt

@@ -1,31 +1,22 @@
 package com.lcl.lclmeasurementtool.ui
 
 import androidx.compose.foundation.layout.Box
-import androidx.compose.foundation.layout.BoxWithConstraints
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.lazy.LazyColumn
-import androidx.compose.foundation.lazy.items
 import androidx.compose.material3.*
 import androidx.compose.material3.TabRowDefaults.tabIndicatorOffset
 import androidx.compose.runtime.*
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
-import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.style.TextAlign
-import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import androidx.hilt.navigation.compose.hiltViewModel
-import androidx.lifecycle.compose.ExperimentalLifecycleComposeApi
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
-import com.lcl.lclmeasurementtool.model.datamodel.ConnectivityReportModel
-import com.lcl.lclmeasurementtool.model.datamodel.SignalStrengthReportModel
 import com.lcl.lclmeasurementtool.model.viewmodels.ConnectivityUiState
 import com.lcl.lclmeasurementtool.model.viewmodels.ConnectivityViewModel
 import com.lcl.lclmeasurementtool.model.viewmodels.SignalStrengthUiState
 import com.lcl.lclmeasurementtool.model.viewmodels.SignalStrengthViewModel
-import dagger.hilt.android.lifecycle.HiltViewModel
-import kotlinx.coroutines.flow.collect
 
 @Composable
 fun HistoryRoute(
@@ -55,7 +46,6 @@ fun HistoryScreen(
     }
 }
 
-@OptIn(ExperimentalLifecycleComposeApi::class)
 @Composable
 fun LCLTabContents(
     selectedTabIndex: Int,

app/src/main/java/com/lcl/lclmeasurementtool/ui/HomeScreen.kt

@@ -26,7 +26,6 @@ import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
-import androidx.lifecycle.compose.ExperimentalLifecycleComposeApi
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
 import com.lcl.lclmeasurementtool.ConnectivityTestResult
 import com.lcl.lclmeasurementtool.MainActivityViewModel
@@ -41,7 +40,6 @@ fun HomeRoute(isOffline: Boolean, mainActivityViewModel: MainActivityViewModel)
     HomeScreen(isOffline = isOffline, mainActivityViewModel = mainActivityViewModel)
 }
 
-@OptIn(ExperimentalLifecycleComposeApi::class)
 @Composable
 fun HomeScreen(modifier: Modifier = Modifier, isOffline: Boolean, mainActivityViewModel: MainActivityViewModel) {
 

app/src/main/java/com/lcl/lclmeasurementtool/ui/LCLApplication.kt

@@ -12,7 +12,6 @@ import androidx.compose.ui.graphics.vector.ImageVector
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
 import androidx.hilt.navigation.compose.hiltViewModel
-import androidx.lifecycle.compose.ExperimentalLifecycleComposeApi
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
 import androidx.navigation.NavDestination
 import androidx.navigation.NavDestination.Companion.hierarchy
@@ -27,9 +26,7 @@ import com.lcl.lclmeasurementtool.ui.navigation.historyGraph
 import com.lcl.lclmeasurementtool.ui.navigation.homeNavigationRoute
 import com.lcl.lclmeasurementtool.ui.navigation.homeScreen
 
-@OptIn(ExperimentalLayoutApi::class, ExperimentalMaterial3Api::class,
-    ExperimentalLifecycleComposeApi::class
-)
+@OptIn(ExperimentalLayoutApi::class, ExperimentalMaterial3Api::class)
 @Composable
 fun LCLApp(
     windowSizeClass: WindowSizeClass,