From b80d1575e6d5268e3ee69c14adcc8e2ad9dcfa88 Mon Sep 17 00:00:00 2001
From: Lance Hudson <lance.hudson@cbsinteractive.com>
Date: Thu, 11 Jun 2020 17:56:31 -0400
Subject: [PATCH] Update tests with new per_page params

---
 octodns/provider/cloudflare.py            |  3 ++-
 tests/test_octodns_provider_cloudflare.py | 13 ++++++++-----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/octodns/provider/cloudflare.py b/octodns/provider/cloudflare.py
index 96febf4..96c9f5e 100644
--- a/octodns/provider/cloudflare.py
+++ b/octodns/provider/cloudflare.py
@@ -251,7 +251,8 @@ class CloudflareProvider(BaseProvider):
             path = '/zones/{}/dns_records'.format(zone_id)
             page = 1
             while page:
-                resp = self._try_request('GET', path, params={'page': page, 'per_page': 100})
+                resp = self._try_request('GET', path, params={'page': page,
+                                         'per_page': 100})
                 records += resp['result']
                 info = resp['result_info']
                 if info['count'] > 0 and info['count'] == info['per_page']:
diff --git a/tests/test_octodns_provider_cloudflare.py b/tests/test_octodns_provider_cloudflare.py
index 08608ea..735d95c 100644
--- a/tests/test_octodns_provider_cloudflare.py
+++ b/tests/test_octodns_provider_cloudflare.py
@@ -426,7 +426,7 @@ class TestCloudflareProvider(TestCase):
         # get the list of zones, create a zone, add some records, update
         # something, and delete something
         provider._request.assert_has_calls([
-            call('GET', '/zones', params={'page': 1}),
+            call('GET', '/zones', params={'page': 1, 'per_page': 50}),
             call('POST', '/zones', data={
                 'jump_start': False,
                 'name': 'unit.tests'
@@ -531,7 +531,7 @@ class TestCloudflareProvider(TestCase):
 
         # Get zones, create zone, create a record, delete a record
         provider._request.assert_has_calls([
-            call('GET', '/zones', params={'page': 1}),
+            call('GET', '/zones', params={'page': 1, 'per_page': 50}),
             call('POST', '/zones', data={
                 'jump_start': False,
                 'name': 'unit.tests'
@@ -1302,7 +1302,8 @@ class TestCloudflareProvider(TestCase):
         provider._request.side_effect = [result]
         self.assertEquals([], provider.zone_records(zone))
         provider._request.assert_has_calls([call('GET', '/zones',
-                                           params={'page': 1})])
+                                           params={'page': 1,
+                                                   'per_page': 50})])
 
         # One retry required
         provider._zones = None
@@ -1313,7 +1314,8 @@ class TestCloudflareProvider(TestCase):
         ]
         self.assertEquals([], provider.zone_records(zone))
         provider._request.assert_has_calls([call('GET', '/zones',
-                                           params={'page': 1})])
+                                           params={'page': 1,
+                                                   'per_page': 50})])
 
         # Two retries required
         provider._zones = None
@@ -1325,7 +1327,8 @@ class TestCloudflareProvider(TestCase):
         ]
         self.assertEquals([], provider.zone_records(zone))
         provider._request.assert_has_calls([call('GET', '/zones',
-                                           params={'page': 1})])
+                                           params={'page': 1,
+                                                   'per_page': 50})])
 
         # # Exhaust our retries
         provider._zones = None