diff --git a/src/main/java/com/facebook/ads/sdk/APINodeList.java b/src/main/java/com/facebook/ads/sdk/APINodeList.java index 0ef02b41..9b56c716 100644 --- a/src/main/java/com/facebook/ads/sdk/APINodeList.java +++ b/src/main/java/com/facebook/ads/sdk/APINodeList.java @@ -25,10 +25,7 @@ import com.google.gson.JsonObject; import com.google.gson.JsonParser; import java.net.URL; -import java.util.List; import java.util.ArrayList; -import java.util.Map; -import java.util.HashMap; import java.util.Iterator; import java.util.Objects; import java.util.Spliterator; @@ -86,12 +83,7 @@ public APINodeList nextPage(int limit) throws APIException { } return (APINodeList) request.execute(); } - if (after == null) return null; - this.request.setOverrideUrl(null); - Map extraParams = new HashMap(); - if (limit > 0) extraParams.put("limit", limit); - extraParams.put("after", after); - return (APINodeList) request.execute(extraParams); + return null; } public void setCursors(String before, String after) { diff --git a/src/test/kotlin/com/facebook/ads/sdk/ApiNodeListTest.kt b/src/test/kotlin/com/facebook/ads/sdk/ApiNodeListTest.kt new file mode 100644 index 00000000..3fb954f3 --- /dev/null +++ b/src/test/kotlin/com/facebook/ads/sdk/ApiNodeListTest.kt @@ -0,0 +1,44 @@ +package com.facebook.ads.sdk + +import io.mockk.Runs +import io.mockk.every +import io.mockk.just +import io.mockk.mockk +import io.mockk.verify +import org.junit.Test +import kotlin.test.assertEquals +import kotlin.test.assertNull + +class ApiNodeListTest { + + @Test + fun testNextPageWhenNextPageAvailable() { + val request = mockk>() + val nextPage = mockk>() + every { request.setOverrideUrl("https://graph.facebook.api/NEXT") } just Runs + every { request.execute() } returns(nextPage) + val list = APINodeList(request, "raw value", "header") + list.setPaging("https://graph.facebook.api/PREVIOUS", "https://graph.facebook.api/NEXT") + list.setCursors("BEFORE_CURSOR", "AFTER_CURSOR") + + val result = list.nextPage() + + assertEquals(nextPage, result) + verify { + request.setOverrideUrl("https://graph.facebook.api/NEXT") + } + } + + @Test + fun testNextPageWhenAlreadyOnLastPage() { + val request = mockk>() + val list = APINodeList(request, "raw value", "header") + list.setPaging("https://graph.facebook.api/PREVIOUS", null) + list.setCursors("BEFORE_CURSOR", "AFTER_CURSOR") + + val result = list.nextPage() + + assertNull(result) + } + +} \ No newline at end of file