Fix app not updating in background

Closes #38

Author: julieminer

app/src/main/java/com/melonhead/mangadexfollower/App.kt

@@ -54,8 +54,11 @@ class App: Application() {
                 appContext.isInForeground = true
                 appEventsRepository.postEvent(AppLifecycleEvent.AppForegrounded)
                 Clog.i("Creating background task")
-                val refreshWorkRequest = PeriodicWorkRequestBuilder<RefreshWorker>(15.minutes.toJavaDuration()).build()
-                WorkManager.getInstance(this@App).enqueueUniquePeriodicWork("refresh-task", ExistingPeriodicWorkPolicy.KEEP, refreshWorkRequest)
+                val refreshWorkRequest = PeriodicWorkRequestBuilder<RefreshWorker>(
+                    repeatInterval = 30.minutes.toJavaDuration(),
+                    flexTimeInterval = 15.minutes.toJavaDuration()
+                ).build()
+                WorkManager.getInstance(this@App).enqueueUniquePeriodicWork("refresh-task", ExistingPeriodicWorkPolicy.UPDATE, refreshWorkRequest)
             }
 
             override fun onStop(owner: LifecycleOwner) {

feature-authentication/src/main/java/com/melonhead/feature_authentication/di/FeatureAuthenticationModule.kt

@@ -24,6 +24,6 @@ val FeatureAuthenticationModule = module {
     includes(DataAuthenticationModule)
     includes(DataUserModule)
 
-    single<AuthRepository> { AuthRepositoryImpl(get(), get(), get(), get(), get(), get(), get(), get()) }
+    single<AuthRepository>(createdAtStart = true) { AuthRepositoryImpl(get(), get(), get(), get(), get(), get(), get(), get()) }
     single<LoginScreenResolver>(createdAtStart = true) { LoginScreenResolver() }
 }

lib-chapter-cache/src/main/java/com/melonhead/lib_chapter_cache/ChapterCache.kt

@@ -95,13 +95,14 @@ internal class ChapterCacheImpl(
                     val downloadJob = async {
                         try {
                             val result = httpClient.downloadFile(pageFile, page)
-                            if (!result) {
+                            if (result) {
+                                Clog.i("Finished downloading page $i for ${mangaForChapter.chosenTitle} chapter $chapterTitle - $page")
+                                true
+                            } else {
                                 pageFile.delete()
-                                Clog.w("Error downloading page $i for ${mangaForChapter.chosenTitle} chapter ${chapter.chapterTitle} - $page")
-                                return@async false
+                                Clog.w("Failed to download page $i for ${mangaForChapter.chosenTitle} chapter ${chapter.chapterTitle} - $page")
+                                false
                             }
-                            Clog.i("Finished downloading page $i for ${mangaForChapter.chosenTitle} chapter $chapterTitle - $page")
-                            true
                         } catch (e: Exception) {
                             Clog.w("Error downloading page $i for ${mangaForChapter.chosenTitle} chapter ${chapter.chapterTitle} - $page")
                             Clog.e("Error downloading page", e)

lib-networking/src/main/java/com/melonhead/lib_networking/extensions/HttpClientExtensions.kt

@@ -68,6 +68,12 @@ suspend fun HttpClient.downloadFile(outputFile: File, url: String): Boolean {
                 outputFile.appendBytes(bytes)
             }
         }
-        return@execute httpResponse.contentLength() == outputFile.length()
+        val sourceLength = httpResponse.contentLength()
+        val fileLength = outputFile.length()
+        if (sourceLength != fileLength) {
+            Clog.w("Downloaded file size and source file size don't match. Source = $sourceLength, file = $fileLength")
+            return@execute false
+        }
+        return@execute true
     }
 }