fix: send initial kwargs for all nextlink calls (#36)

ali-cetin-4ss · web-flow · commit 6bfe1ee72aa5 · 2023-08-25T13:16:29.000+02:00
diff --git a/fourinsight/api/authenticate.py b/fourinsight/api/authenticate.py
@@ -223,6 +223,7 @@ def get_pages(self, url, **kwargs):
                 url = None
             else:
                 url = response.json().get("@odata.nextLink")
+                kwargs = {}
             yield response
 
 
diff --git a/tests/test_authenticator.py b/tests/test_authenticator.py
@@ -552,9 +552,9 @@ def get_side_effect(*args, **kwargs):
             mock_get.assert_has_calls(
                 [
                     call("foo/bar/baz", baz="foobar"),
-                    call("first/nextlink", baz="foobar"),
-                    call("second/nextlink", baz="foobar"),
-                    call("third/nextlink", baz="foobar"),
+                    call("first/nextlink"),
+                    call("second/nextlink"),
+                    call("third/nextlink"),
                 ]
             )
 
@@ -605,9 +605,9 @@ def get_side_effect(*args, **kwargs):
             mock_get.assert_has_calls(
                 [
                     call("foo/bar/baz", baz="foobar"),
-                    call("first/nextlink", baz="foobar"),
-                    call("second/nextlink", baz="foobar"),
-                    call("third/nextlink", baz="foobar"),
+                    call("first/nextlink"),
+                    call("second/nextlink"),
+                    call("third/nextlink"),
                 ]
             )
 

Original file line number	Diff line number	Diff line change
`@@ -552,9 +552,9 @@ def get_side_effect(args, *kwargs):`
`552`	`552`	`mock_get.assert_has_calls(`
`553`	`553`	`[`
`554`	`554`	`call("foo/bar/baz", baz="foobar"),`
`555`		`- call("first/nextlink", baz="foobar"),`
`556`		`- call("second/nextlink", baz="foobar"),`
`557`		`- call("third/nextlink", baz="foobar"),`
	`555`	`+ call("first/nextlink"),`
	`556`	`+ call("second/nextlink"),`
	`557`	`+ call("third/nextlink"),`
`558`	`558`	`]`
`559`	`559`	`)`
`560`	`560`
`@@ -605,9 +605,9 @@ def get_side_effect(args, *kwargs):`
`605`	`605`	`mock_get.assert_has_calls(`
`606`	`606`	`[`
`607`	`607`	`call("foo/bar/baz", baz="foobar"),`
`608`		`- call("first/nextlink", baz="foobar"),`
`609`		`- call("second/nextlink", baz="foobar"),`
`610`		`- call("third/nextlink", baz="foobar"),`
	`608`	`+ call("first/nextlink"),`
	`609`	`+ call("second/nextlink"),`
	`610`	`+ call("third/nextlink"),`
`611`	`611`	`]`
`612`	`612`	`)`
`613`	`613`