Tryfox v2 candidate

README.md

@@ -30,12 +30,13 @@ Install the latest APK from [TryFox Releases](https://github.com/mozilla-mobile/
     -   `mozilla-central` (displayed as "central")
     -   `mozilla-beta` (displayed as "beta")
     -   `mozilla-release` (displayed as "release")
+    -   `autoland` (displayed as "autoland")
 -   **Search by Revision**: Enter a full revision hash to find associated builds for the selected project.
 -   **Automated CI Data Fetching**:
     1.  Queries Treeherder API for push details using the selected project and revision.
     2.  Displays relevant push comment from the revision (often a Bugzilla link).
     3.  Fetches all jobs associated with the push.
-    4.  Filters for relevant, signed, non-test build jobs (jobs containing "B" and "s", excluding "t" in their symbols).
+    4.  Filters for relevant Android APK build jobs, including signed and unsigned builds, while excluding test jobs.
     5.  For each job, retrieves its artifacts from Taskcluster.
 -   **Job and Artifact Display**:
     -   Lists build jobs with their app icon (e.g., Fenix, Focus), job name, job symbol, and Task ID.
@@ -106,7 +107,7 @@ Run Android instrumentation tests with a connected device or running emulator:
 -   **Retrofit** for type-safe HTTP requests to Treeherder and Taskcluster APIs.
 -   **Kotlinx Serialization** for efficient JSON parsing.
 -   **OkHttp** as the underlying HTTP client for Retrofit (with a `HttpLoggingInterceptor` for debugging network traffic).
--   **FileProvider** for securely sharing downloaded APKs with the system package installer.
+-   **PackageInstaller sessions** for installing downloaded APKs directly from TryFox.
 
 ## Requirements
 

app/build.gradle.kts

@@ -122,6 +122,7 @@ dependencies {
     // Additional Compose dependencies
     implementation(libs.androidx.compose.material.icons.extended)
     implementation(libs.androidx.lifecycle.viewmodel.compose)
+    implementation(libs.androidx.work.runtime.ktx)
 
     // DataStore
     implementation(libs.androidx.datastore.preferences)

app/src/androidTest/java/org/mozilla/tryfox/MainActivityDeeplinkTest.kt

@@ -82,7 +82,7 @@ class MainActivityDeeplinkTest {
     }
 
     @Test
-    fun testDeeplink_withAuthorEmail_populatesProfileScreen() {
+    fun testDeeplink_withAuthorEmail_populatesSearchScreen() {
         val email = "tthibaud@mozilla.com"
         val encodedEmail = "tthibaud%40mozilla.com"
         val deeplinkUri =
@@ -118,7 +118,7 @@ class MainActivityDeeplinkTest {
     }
 
     @Test
-    fun testTryfoxScheme_withAuthorEmail_populatesProfileScreen() {
+    fun testTryfoxScheme_withAuthorEmail_populatesSearchScreen() {
         val email = "tthibaud@mozilla.com"
         val encodedEmail = "tthibaud%40mozilla.com"
         val deeplinkUri =

app/src/androidTest/java/org/mozilla/tryfox/data/FakeApkDownloadCoordinator.kt

@@ -0,0 +1,17 @@
+package org.mozilla.tryfox.data
+
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.flowOf
+import org.mozilla.tryfox.download.ApkDownloadCoordinator
+import org.mozilla.tryfox.download.ApkDownloadRequest
+import org.mozilla.tryfox.download.model.PersistedDownloadState
+
+class FakeApkDownloadCoordinator : ApkDownloadCoordinator {
+    override val downloads = MutableStateFlow<Map<String, PersistedDownloadState>>(emptyMap())
+
+    override fun enqueue(request: ApkDownloadRequest): String = request.uniqueKey
+    override fun retry(request: ApkDownloadRequest): String = request.uniqueKey
+    override fun cancel(uniqueKey: String) = Unit
+    override fun observe(uniqueKey: String): Flow<PersistedDownloadState?> = flowOf(downloads.value[uniqueKey])
+}

app/src/androidTest/java/org/mozilla/tryfox/data/FakeCacheManager.kt

@@ -9,12 +9,15 @@ import java.io.File
 class FakeCacheManager : CacheManager {
     private val _cacheState = MutableStateFlow<CacheManagementState>(CacheManagementState.IdleEmpty)
     override val cacheState: StateFlow<CacheManagementState> = _cacheState
+    private val _cacheSizeBytes = MutableStateFlow(0L)
+    override val cacheSizeBytes: StateFlow<Long> = _cacheSizeBytes
 
     override suspend fun clearCache() {
         _cacheState.value = CacheManagementState.IdleEmpty
+        _cacheSizeBytes.value = 0L
     }
 
-    override fun checkCacheStatus() {
+    override suspend fun checkCacheStatus() {
         // No-op for now
     }
 

app/src/androidTest/java/org/mozilla/tryfox/data/FakeDownloadFileRepository.kt

@@ -16,7 +16,7 @@ class FakeDownloadFileRepository(
     override suspend fun downloadFile(
         downloadUrl: String,
         outputFile: File,
-        onProgress: (bytesDownloaded: Long, totalBytes: Long) -> Unit,
+        onProgress: suspend (bytesDownloaded: Long, totalBytes: Long) -> Unit,
     ): NetworkResult<File> {
         downloadFileCalled = true
 

app/src/androidTest/java/org/mozilla/tryfox/data/FakeIntentManager.kt

@@ -1,38 +1,14 @@
 package org.mozilla.tryfox.data
 
 import org.mozilla.tryfox.data.managers.IntentManager
-import java.io.File
-
 /**
  * A fake implementation of [IntentManager] for use in instrumented tests.
- * This class allows for verifying that the `installApk` method is called with the correct file.
  */
 class FakeIntentManager() : IntentManager {
 
     var wasUninstallApkCalled: Boolean = false
         private set
 
-    /**
-     * A boolean flag to indicate whether the `installApk` method was called.
-     */
-    val wasInstallApkCalled: Boolean
-        get() = installedFile != null
-
-    /**
-     * The file that was passed to the `installApk` method.
-     */
-    var installedFile: File? = null
-        private set
-
-    /**
-     * Overrides the `installApk` method to capture the file and set the `wasInstallApkCalled` flag.
-     *
-     * @param file The file to be "installed".
-     */
-    override fun installApk(file: File) {
-        installedFile = file
-    }
-
     override fun uninstallApk(packageName: String) {
         wasUninstallApkCalled = true
     }

app/src/androidTest/java/org/mozilla/tryfox/data/FakeUserDataRepository.kt

@@ -2,8 +2,12 @@ package org.mozilla.tryfox.data
 
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.MutableStateFlow
+import org.mozilla.tryfox.data.SearchHistory
+import org.mozilla.tryfox.data.SearchHistoryEntry
+import org.mozilla.tryfox.data.SearchHistoryQueryType
 import org.mozilla.tryfox.data.repositories.UserDataRepository
 import org.mozilla.tryfox.lan.LanReceiveIdentity
+import org.mozilla.tryfox.model.HomeScreenLayout
 
 /**
  * A fake implementation of [org.mozilla.tryfox.data.repositories.UserDataRepository] for testing purposes.
@@ -12,19 +16,37 @@ class FakeUserDataRepository : UserDataRepository {
 
     private val _lastSearchedEmailFlow = MutableStateFlow("")
     override val lastSearchedEmailFlow: Flow<String> = _lastSearchedEmailFlow
+    private val _searchHistoryFlow = MutableStateFlow<List<SearchHistoryEntry>>(emptyList())
+    override val searchHistoryFlow: Flow<List<SearchHistoryEntry>> = _searchHistoryFlow
     private val _lanReceiveIdentityFlow = MutableStateFlow<LanReceiveIdentity?>(null)
     override val lanReceiveIdentityFlow: Flow<LanReceiveIdentity?> = _lanReceiveIdentityFlow
+    private val _homeScreenLayoutFlow = MutableStateFlow(HomeScreenLayout.OneCardPerApp)
+    override val homeScreenLayoutFlow: Flow<HomeScreenLayout> = _homeScreenLayoutFlow
 
     override suspend fun saveLastSearchedEmail(email: String) {
-        _lastSearchedEmailFlow.value = email
+        recordSearch("try", email)
+    }
+
+    override suspend fun recordSearch(project: String, query: String, searchedAt: Long) {
+        val queryType = if ('@' in query) SearchHistoryQueryType.EMAIL else SearchHistoryQueryType.REVISION
+        _searchHistoryFlow.value = SearchHistory.record(
+            _searchHistoryFlow.value,
+            SearchHistoryEntry(project, query, queryType, searchedAt),
+        )
+        _lastSearchedEmailFlow.value = SearchHistory.latestEmail(_searchHistoryFlow.value)
     }
 
     override suspend fun saveLanReceiveIdentity(identity: LanReceiveIdentity) {
         _lanReceiveIdentityFlow.value = identity
     }
 
+    override suspend fun saveHomeScreenLayout(layout: HomeScreenLayout) {
+        _homeScreenLayoutFlow.value = layout
+    }
+
     // Helper method for tests to clear the stored email if needed
     fun clearLastSearchedEmail() {
         _lastSearchedEmailFlow.value = ""
+        _searchHistoryFlow.value = emptyList()
     }
 }