Refactor next() -> nextBatch()

element-hq · Jan 24, 2023 · d0196d7 · d0196d7
1 parent ff229ff
commit d0196d7
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/RiotSwiftUI/Modules/Room/PollHistory/PollHistoryViewModel.swift b/RiotSwiftUI/Modules/Room/PollHistory/PollHistoryViewModel.swift
@@ -49,7 +49,7 @@ private extension PollHistoryViewModel {
         state.isLoading = true
 
         pollService
-            .next()
+            .nextBatch()
             .collect()
             .sink { [weak self] _ in
                 #warning("Handle errors")

diff --git a/RiotSwiftUI/Modules/Room/PollHistory/Service/Mock/MockPollHistoryService.swift b/RiotSwiftUI/Modules/Room/PollHistory/Service/Mock/MockPollHistoryService.swift
@@ -32,7 +32,7 @@ final class MockPollHistoryService: PollHistoryServiceProtocol {
         .setFailureType(to: Error.self)
         .eraseToAnyPublisher()
 
-    func next() -> AnyPublisher<TimelinePollDetails, Error> {
+    func nextBatch() -> AnyPublisher<TimelinePollDetails, Error> {
         nextPublisher
     }
 }

diff --git a/RiotSwiftUI/Modules/Room/PollHistory/Service/PollHistoryServiceProtocol.swift b/RiotSwiftUI/Modules/Room/PollHistory/Service/PollHistoryServiceProtocol.swift
@@ -1,4 +1,4 @@
-// 
+//
 // Copyright 2023 New Vector Ltd
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
@@ -18,13 +18,13 @@ import Combine
 
 protocol PollHistoryServiceProtocol {
     /// Returns a Publisher publishing the polls in the next batch.
-    /// Implementations should return the same publisher if `next()` is called again before the previous publisher completes.
-    func next() -> AnyPublisher<TimelinePollDetails, Error>
+    /// Implementations should return the same publisher if `nextBatch()` is called again before the previous publisher completes.
+    func nextBatch() -> AnyPublisher<TimelinePollDetails, Error>
 
-    /// Publishes updates for the polls previously pusblished by the `next()` publishers.
+    /// Publishes updates for the polls previously pusblished by the `nextBatch()` publishers.
     var updates: AnyPublisher<TimelinePollDetails, Never> { get }
 
     /// Publishes errors regarding poll aggregations.
-    /// Note: `next()` will continue to publish new polls even if some poll isn't being aggregated correctly.
+    /// Note: `nextBatch()` will continue to publish new polls even if some poll isn't being aggregated correctly.
     var pollErrors: AnyPublisher<Error, Never> { get }
 }