Fix flickering test

modules/storages/spec/services/storages/group_folder_properties_sync_service_spec.rb:735
could fail because the order of the users returned by database is not
deterministic, making the expected ACL list order incorrect.

By sorting by id, the order is assured to be deterministic.

modules/storages/app/services/storages/group_folder_properties_sync_service.rb

@@ -228,6 +228,7 @@ def tokens_query(project:)
                      .where(oauth_client: @storage.oauth_client)
                      .where.not(id: @admin_tokens_query)
                      .includes(:user)
+                     .order(:id)
     # The user scope is required in all cases except one:
     #   when the project is public and non member has at least one storage permission
     #   then all non memebers should have access to the project folder