Fix pylint no-self-use and arguments-differ in test/unit/pagination

Author: autoreleasefool

test/unit/pagination/box_object_collection_test_base.py

@@ -13,8 +13,9 @@
 class BoxObjectCollectionTestBase(object):
     NUM_ENTRIES = 25
 
+    @staticmethod
     @pytest.fixture()
-    def translator(self):
+    def translator():
         return Translator()
 
     @pytest.fixture()
@@ -51,7 +52,8 @@ def _object_collection_instance(self, session, limit, return_full_pages=False, s
         """
         raise NotImplementedError
 
-    def _assert_items_dict_and_objects_same(self, expected_items_dict, returned_item_objects):
+    @staticmethod
+    def _assert_items_dict_and_objects_same(expected_items_dict, returned_item_objects):
         """
         A fixture very specific to this test class. Asserts that the list of items in dictionary form are the
         same (at least in name, and in quantity) as a list of BaseObjects.

test/unit/pagination/test_marker_based_object_collection.py

@@ -20,14 +20,15 @@ class TestMarkerBasedObjectCollection(BoxObjectCollectionTestBase):
 
     NO_NEXT_MARKER = object()
 
+    @staticmethod
     @pytest.fixture(params=['', None, NO_NEXT_MARKER])
-    def next_marker_value_for_last_page(self, request):
+    def next_marker_value_for_last_page(request):
         return request.param
 
     @pytest.fixture()
     def mock_items_response(self, entries, next_marker_value_for_last_page):
         """Baseclass override."""
-        # pylint:disable=redefined-outer-name
+        # pylint:disable=redefined-outer-name,arguments-differ
         def get_response(limit, marker):
             mock_box_response = Mock(BoxResponse)
             mock_network_response = Mock(DefaultNetworkResponse)