From c243dd0adea0bcda6cc41330fce7c3abb14933b4 Mon Sep 17 00:00:00 2001
From: Aurélien Bompard <aurelien@bompard.org>
Date: Mar 16 2021 12:45:33 +0000
Subject: Fix 9f355d9ef, option name is not intuitive


Since the big refactor in python-freeipa, option names are prefixed with
`o_`. It's not very intuitive but that's what we have for now.

Signed-off-by: Aurélien Bompard <aurelien@bompard.org>

---

diff --git a/noggin/tests/unit/utility/test_pagination.py b/noggin/tests/unit/utility/test_pagination.py
index 327a8dc..384ec81 100644
--- a/noggin/tests/unit/utility/test_pagination.py
+++ b/noggin/tests/unit/utility/test_pagination.py
@@ -90,7 +90,7 @@ def test_groups_page_nopaging(client, logged_in_dummy_user, mocker):
     page = BeautifulSoup(result.data, 'html.parser')
     groups = page.select("ul.list-group li")
     assert len(groups) == 2
-    ipa.group_find.assert_called_with(fasgroup=True, all=True, sizelimit=0)
+    ipa.group_find.assert_called_with(fasgroup=True, all=True, o_sizelimit=0)
     ipa.batch.assert_not_called()
 
 
@@ -116,7 +116,7 @@ def test_empty_result(mocker, app):
     ipa.group_find.return_value = {"result": []}
     with app.test_request_context("/?page_size=10"):
         result = paginated_find(ipa, Group)
-    ipa.group_find.assert_called_once_with(pkey_only=True, sizelimit=0)
+    ipa.group_find.assert_called_once_with(pkey_only=True, o_sizelimit=0)
     ipa.batch.assert_not_called()
     assert len(result.items) == 0
 
diff --git a/noggin/utility/pagination.py b/noggin/utility/pagination.py
index 462edc4..751a999 100644
--- a/noggin/utility/pagination.py
+++ b/noggin/utility/pagination.py
@@ -47,7 +47,7 @@ class PagedResult:
 
 
 def paginated_find(ipa, representation, *args, **kwargs):
-    kwargs.setdefault("sizelimit", 0)
+    kwargs.setdefault("o_sizelimit", 0)
     pkey_name = representation.get_ipa_pkey()
     object_name = representation.ipa_object
     find_method = getattr(ipa, f"{object_name}_find")