refactoring: Use 'iterator' in place of 'autoPageIter' function name

sinch · Nov 29, 2023 · 4bef0ed · 4bef0ed
1 parent fb1abb3
commit 4bef0ed
Show file tree

Hide file tree

Showing 13 changed files with 13 additions and 18 deletions.
diff --git a/...est/java/com/sinch/sdk/domains/numbers/models/responses/ActiveNumberListResponseTest.java b/...est/java/com/sinch/sdk/domains/numbers/models/responses/ActiveNumberListResponseTest.java
@@ -86,7 +86,7 @@ void autoPageIter() {
     AtomicInteger count = new AtomicInteger();
     ActiveNumberListResponse response = new ActiveNumberListResponse(service, pages.get(0));
     response
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(
             value -> {
               Assertions.assertThat(value)

diff --git a/.../java/com/sinch/sdk/domains/numbers/models/responses/AvailableNumberListResponseTest.java b/.../java/com/sinch/sdk/domains/numbers/models/responses/AvailableNumberListResponseTest.java
@@ -41,7 +41,7 @@ class AvailableNumberListResponseTest {
   void autoPageIter() {
     AtomicInteger count = new AtomicInteger();
     response
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(
             value -> {
               Assertions.assertThat(value)

diff --git a/.../java/com/sinch/sdk/domains/numbers/models/responses/AvailableRegionListResponseTest.java b/.../java/com/sinch/sdk/domains/numbers/models/responses/AvailableRegionListResponseTest.java
@@ -33,7 +33,7 @@ class AvailableRegionListResponseTest {
   void autoPageIter() {
     AtomicInteger count = new AtomicInteger();
     response
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(
             value -> {
               Assertions.assertThat(value)

diff --git a/client/src/test/java/com/sinch/sdk/domains/sms/adapters/BatchesServiceTest.java b/client/src/test/java/com/sinch/sdk/domains/sms/adapters/BatchesServiceTest.java
@@ -379,7 +379,7 @@ void list() throws ApiException {
 
     BatchesListResponse response = service.list(null);
 
-    Iterator<Batch<?>> iterator = response.autoPageIter();
+    Iterator<Batch<?>> iterator = response.iterator();
     Batch<?> batch = iterator.next();
     Assertions.assertThat(iterator.hasNext()).isEqualTo(true);
     Assertions.assertThat(batch)

diff --git a/client/src/test/java/com/sinch/sdk/domains/sms/adapters/DeliveryReportsServiceTest.java b/client/src/test/java/com/sinch/sdk/domains/sms/adapters/DeliveryReportsServiceTest.java
@@ -171,7 +171,7 @@ void list() throws ApiException {
 
     DeliveryReportsListResponse response = service.list(null);
 
-    Iterator<DeliveryReportRecipient> iterator = response.autoPageIter();
+    Iterator<DeliveryReportRecipient> iterator = response.iterator();
     DeliveryReportRecipient item = iterator.next();
     Assertions.assertThat(iterator.hasNext()).isEqualTo(true);
 

diff --git a/client/src/test/java/com/sinch/sdk/domains/sms/adapters/GroupsServiceTest.java b/client/src/test/java/com/sinch/sdk/domains/sms/adapters/GroupsServiceTest.java
@@ -88,7 +88,7 @@ void list() throws ApiException {
 
     GroupsListResponse response = service.list(null);
 
-    Iterator<Group> iterator = response.autoPageIter();
+    Iterator<Group> iterator = response.iterator();
     Group item = iterator.next();
     Assertions.assertThat(iterator.hasNext()).isEqualTo(true);
 

diff --git a/client/src/test/java/com/sinch/sdk/domains/sms/adapters/InboundsServiceTest.java b/client/src/test/java/com/sinch/sdk/domains/sms/adapters/InboundsServiceTest.java
@@ -95,7 +95,7 @@ void list() throws ApiException {
 
     InboundsListResponse response = service.list(null);
 
-    Iterator<Inbound<?>> iterator = response.autoPageIter();
+    Iterator<Inbound<?>> iterator = response.iterator();
     Inbound<?> item = iterator.next();
     Assertions.assertThat(iterator.hasNext()).isEqualTo(true);
 

diff --git a/core/src/main/com/sinch/sdk/core/models/pagination/ListResponse.java b/core/src/main/com/sinch/sdk/core/models/pagination/ListResponse.java
@@ -46,7 +46,7 @@ public abstract class ListResponse<T> {
    *
    * @return Iterator onto items
    */
-  public Iterator<T> autoPageIter() {
+  public Iterator<T> iterator() {
     return new ItemsIterator<>(this);
   }
 

diff --git a/sample-app/src/main/java/com/sinch/sample/numbers/active/List.java b/sample-app/src/main/java/com/sinch/sample/numbers/active/List.java
@@ -38,7 +38,7 @@ public void run() {
                     .build());
     AtomicInteger page = new AtomicInteger(1);
     response
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(
             value ->
                 LOGGER.info(

diff --git a/sample-app/src/main/java/com/sinch/sample/sms/batches/List.java b/sample-app/src/main/java/com/sinch/sample/sms/batches/List.java
@@ -36,7 +36,7 @@ public void run() {
                 // .setClientReference("a client reference")
                 // .setStartDate(Instant.now())
                 .build())
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(f -> LOGGER.info(f.toString()));
   }
 }
diff --git a/sample-app/src/main/java/com/sinch/sample/sms/deliveryReports/List.java b/sample-app/src/main/java/com/sinch/sample/sms/deliveryReports/List.java
@@ -36,7 +36,7 @@ public void run() {
                     Arrays.asList(
                         DeliveryReportErrorCode.from(0), DeliveryReportErrorCode.DISPATCHED))
                 .build())
-        .autoPageIter()
+        .iterator()
         .forEachRemaining(f -> LOGGER.info(f.toString()));
   }
 }
diff --git a/sample-app/src/main/java/com/sinch/sample/sms/groups/List.java b/sample-app/src/main/java/com/sinch/sample/sms/groups/List.java
@@ -26,6 +26,6 @@ public void run() {
 
     GroupsListResponse response = client.sms().groups().list();
 
-    response.autoPageIter().forEachRemaining(f -> LOGGER.info(f.toString()));
+    response.iterator().forEachRemaining(f -> LOGGER.info(f.toString()));
   }
 }
diff --git a/sample-app/src/main/java/com/sinch/sample/sms/inbounds/List.java b/sample-app/src/main/java/com/sinch/sample/sms/inbounds/List.java
@@ -24,11 +24,6 @@ public void run() {
     LOGGER.info("List inbounds");
 
     LOGGER.info("Response:");
-    client
-        .sms()
-        .inbounds()
-        .list(null)
-        .autoPageIter()
-        .forEachRemaining(f -> LOGGER.info(f.toString()));
+    client.sms().inbounds().list(null).iterator().forEachRemaining(f -> LOGGER.info(f.toString()));
   }
 }