Ensure file paths are sorted in status/commit changes

src/main/kotlin/com/jetpackduba/gitnuro/git/diff/GetCommitDiffEntriesUseCase.kt

@@ -1,5 +1,6 @@
 package com.jetpackduba.gitnuro.git.diff
 
+import com.jetpackduba.gitnuro.extensions.filePath
 import com.jetpackduba.gitnuro.extensions.fullData
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
@@ -36,6 +37,7 @@ class GetCommitDiffEntriesUseCase @Inject constructor() {
             .setNewTree(newTreeParser)
             .setOldTree(oldTreeParser)
             .call()
+            .sortedBy { it.filePath }
     }
 }
 

src/main/kotlin/com/jetpackduba/gitnuro/git/workspace/GetStagedUseCase.kt

@@ -23,6 +23,6 @@ class GetStagedUseCase @Inject constructor() {
                 added,
                 modified,
                 removed,
-            )
+            ).sortedBy { it.filePath }
         }
 }

src/main/kotlin/com/jetpackduba/gitnuro/git/workspace/GetUnstagedUseCase.kt

@@ -8,16 +8,13 @@ import javax.inject.Inject
 
 class GetUnstagedUseCase @Inject constructor() {
     suspend operator fun invoke(status: Status) = withContext(Dispatchers.IO) {
-        // TODO Test uninitialized modules after the refactor
+        val untracked = status.untracked.map {
-//        val uninitializedSubmodules = submodulesManager.uninitializedSubmodules(git)
-
-        val added = status.untracked.map {
             StatusEntry(it, StatusType.ADDED)
         }
         val modified = status.modified.map {
             StatusEntry(it, StatusType.MODIFIED)
         }
-        val removed = status.missing.map {
+        val missing = status.missing.map {
             StatusEntry(it, StatusType.REMOVED)
         }
         val conflicting = status.conflicting.map {
@@ -25,10 +22,10 @@ class GetUnstagedUseCase @Inject constructor() {
         }
 
         return@withContext flatListOf(
-            added,
+            untracked,
             modified,
-            removed,
+            missing,
             conflicting,
-        )
+        ).sortedBy { it.filePath }
     }
 }

src/main/kotlin/com/jetpackduba/gitnuro/updates/UpdatesRepository.kt

@@ -19,7 +19,7 @@ private val updateJson = Json {
 class UpdatesRepository @Inject constructor(
     private val updatesWebService: UpdatesService,
 ) {
-    fun hasUpdatesFlow() = flow<Update> {
+    fun hasUpdatesFlow() = flow {
         val latestReleaseJson = updatesWebService.release(AppConstants.VERSION_CHECK_URL)
 
         while(coroutineContext.isActive) {

src/main/kotlin/com/jetpackduba/gitnuro/viewmodels/StatusViewModel.kt

@@ -40,7 +40,6 @@ class StatusViewModel @Inject constructor(
     private val resetEntryUseCase: DiscardEntryUseCase,
     private val stageAllUseCase: StageAllUseCase,
     private val unstageAllUseCase: UnstageAllUseCase,
-    private val checkHasPreviousCommitsUseCase: CheckHasPreviousCommitsUseCase,
     private val getLastCommitMessageUseCase: GetLastCommitMessageUseCase,
     private val resetRepositoryStateUseCase: ResetRepositoryStateUseCase,
     private val continueRebaseUseCase: ContinueRebaseUseCase,
@@ -53,12 +52,10 @@ class StatusViewModel @Inject constructor(
     private val doCommitUseCase: DoCommitUseCase,
     private val loadAuthorUseCase: LoadAuthorUseCase,
     private val saveAuthorUseCase: SaveAuthorUseCase,
-    private val getRepositoryStateUseCase: GetRepositoryStateUseCase,
-    private val getRebaseAmendCommitIdUseCase: GetRebaseAmendCommitIdUseCase,
     private val sharedRepositoryStateManager: SharedRepositoryStateManager,
     private val getSpecificCommitMessageUseCase: GetSpecificCommitMessageUseCase,
-    private val tabScope: CoroutineScope,
+    tabScope: CoroutineScope,
-    private val appSettings: AppSettings,
+    appSettings: AppSettings,
 ) {
     private val _showSearchUnstaged = MutableStateFlow(false)
     val showSearchUnstaged: StateFlow<Boolean> = _showSearchUnstaged