Merge pull request #455 from adamtheturtle/useless-suppression

Check for useless suppression in pylint

Author: adamtheturtle

pylintrc

@@ -55,7 +55,9 @@ confidence=
 # either give multiple identifier separated by comma (,) or put this option
 # multiple time (only on the command line, not in the configuration file where
 # it should appear only once). See also the "--disable" option for examples.
-enable=spelling
+enable=
+    spelling,
+    useless-suppression,
 
 # Disable the message, report, category or checker with the given id(s). You
 # can either give multiple identifiers separated by comma (,) or put this

src/mock_vws/_mock_web_services_api.py

@@ -573,7 +573,7 @@ def get_duplicates(
         self,
         request: _RequestObjectProxy,  # pylint: disable=unused-argument
         context: _Context,  # pylint: disable=unused-argument
-        target: Target,  # pylint: disable=unused-argument
+        target: Target,
     ) -> str:
         """
         Get targets which may be considered duplicates of a given target.

tests/mock_vws/conftest.py

@@ -55,7 +55,7 @@ def _delete_all_targets(database_keys: VuforiaDatabaseKeys) -> None:
 
 @pytest.fixture()
 def target_id(
-    png_rgb_success: io.BytesIO,  # pylint: disable=redefined-outer-name
+    png_rgb_success: io.BytesIO,
     vuforia_database_keys: VuforiaDatabaseKeys,  # noqa: E501 pylint: disable=redefined-outer-name
 ) -> str:
     """