Cherry pick #3345 sort order blinks to master

lib/features/search/email/presentation/search_email_controller.dart

@@ -490,10 +490,7 @@ class SearchEmailController extends BaseController
       final lastEmail = listResultSearch.last;
 
       if (searchEmailFilter.value.sortOrderType.isScrollByPosition()) {
-        _updateSimpleSearchFilter(
-          positionOption: Some(listResultSearch.length),
-          beforeOption: const None()
-        );
+        _updateSimpleSearchFilter(positionOption: Some(listResultSearch.length));
       } else if (searchEmailFilter.value.sortOrderType == EmailSortOrderType.oldest) {
         _updateSimpleSearchFilter(startDateOption: optionOf(lastEmail.receivedAt));
       } else {

lib/features/thread/presentation/thread_controller.dart

@@ -566,6 +566,15 @@ class ThreadController extends BaseController with EmailActionController {
     }
   }
 
+  @visibleForTesting
+  EmailFilterCondition getFilterCondition({
+    PresentationEmail? oldestEmail,
+    MailboxId? mailboxIdSelected,
+  }) => _getFilterCondition(
+    oldestEmail: oldestEmail,
+    mailboxIdSelected: mailboxIdSelected,
+  );
+
   EmailFilterCondition _getFilterCondition({PresentationEmail? oldestEmail, MailboxId? mailboxIdSelected}) {
     switch(mailboxDashBoardController.filterMessageOption.value) {
       case FilterMessageOption.all:
@@ -656,6 +665,9 @@ class ThreadController extends BaseController with EmailActionController {
     }
   }
 
+  @visibleForTesting
+  Future<void> refreshChangeSearchEmail() => _refreshChangeSearchEmail();
+
   Future<void> _refreshChangeSearchEmail() async {
     await _refreshChangeListEmailCache();
 
@@ -666,7 +678,9 @@ class ThreadController extends BaseController with EmailActionController {
         _searchEmailFilter.sortOrderType.isScrollByPosition(),
         0,
       ),
-      beforeOption: const None(),
+      beforeOption: !_searchEmailFilter.sortOrderType.isScrollByPosition()
+          ? const None()
+          : null,
     );
     final searchViewState = await _searchEmailInteractor.execute(
       _session!,
@@ -936,6 +950,15 @@ class ThreadController extends BaseController with EmailActionController {
     searchController.clearTextSearch();
   }
 
+  @visibleForTesting
+  void searchEmail({
+    UnsignedInt? limit,
+    bool needRefreshSearchState = false,
+  }) => _searchEmail(
+    limit: limit,
+    needRefreshSearchState: needRefreshSearchState,
+  );
+
   void _searchEmail({UnsignedInt? limit, bool needRefreshSearchState = false}) {
     if (_session != null && _accountId != null) {
       if (!needRefreshSearchState && listEmailController.hasClients) {
@@ -1016,6 +1039,9 @@ class ThreadController extends BaseController with EmailActionController {
     }
   }
 
+  @visibleForTesting
+  void searchMoreEmails() => _searchMoreEmails();
+
   void _searchMoreEmails() {
     log('ThreadController::_searchMoreEmails:');
     if (canSearchMore && _session != null && _accountId != null) {