test: separate non-paging tests

Author: jlwalke2

tests/unit/test_pageiterator.py

@@ -58,40 +58,6 @@ def side_effect(_, link, **kwargs):
     assert req.call_count >= math.ceil(call_count)
 
 
-def test_no_paging_required():
-    """If "next" link not present, current items should be included."""
-
-    items = [{"name": "a"}, {"name": "b"}, {"name": "c"}]
-    obj = RestObj(items=items, count=len(items))
-
-    with mock.patch("sasctl.core.request") as req:
-        # Mock appears to be *sometimes* shared with mock created in `paging` fixture
-        # above.  Depending on execution order, this can result in calls to the mock
-        # made by other tests to be counted here.  Explicitly reset to prevent this.
-        req.reset_mock()
-        try:
-            req.assert_not_called()
-        except AssertionError as e:
-            raise AssertionError(
-                f"method_calls={req.mock_calls}  call_args={req.call_args_list}"
-            )
-
-        pager = PageIterator(obj)
-
-        # Returned page of items should preserve item order
-        items = next(pager)
-        for idx, item in enumerate(items):
-            assert item.name == RestObj(items[idx]).name
-
-    # No req should have been made to retrieve additional data.
-    try:
-        req.assert_not_called()
-    except AssertionError as e:
-        raise AssertionError(
-            f"method_calls={req.mock_calls}  call_args={req.call_args_list}"
-        )
-
-
 def test_paging_required(paging):
     """Requests should be made to retrieve additional pages."""
     obj, items, _ = paging

tests/unit/test_pageiterator_no_paging.py

@@ -0,0 +1,26 @@
+from unittest import mock
+
+from sasctl.core import PageIterator, RestObj
+
+
+def test_no_paging_required():
+    """If "next" link not present, current items should be included."""
+
+    items = [{"name": "a"}, {"name": "b"}, {"name": "c"}]
+    obj = RestObj(items=items, count=len(items))
+
+    with mock.patch("sasctl.core.request") as req:
+        pager = PageIterator(obj)
+
+        # Returned page of items should preserve item order
+        items = next(pager)
+        for idx, item in enumerate(items):
+            assert item.name == RestObj(items[idx]).name
+
+    # No req should have been made to retrieve additional data.
+    try:
+        req.assert_not_called()
+    except AssertionError as e:
+        raise AssertionError(
+            f"method_calls={req.mock_calls}  call_args={req.call_args_list}"
+        )