Lint: purge some pylints

Author: Joseph-Edwards

src/libsemigroups_pybind11/detail/cxx_wrapper.py

@@ -59,7 +59,6 @@ def register_cxx_wrapped_type(cxx_type: pybind11_type, py_type: type) -> None:
 
 class CxxWrapper:
     # pylint: disable=missing-class-docstring
-    # pylint: disable=protected-access, no-member
     def __init__(
         self: Self,
         *args,

tests/cong_common.py

@@ -1,7 +1,4 @@
 # -*- coding: utf-8 -*-
-# pylint: disable=no-name-in-module, missing-function-docstring
-# pylint: disable=missing-class-docstring, invalid-name
-
 # Copyright (c) 2021-2024 J. D. Mitchell
 #
 # Distributed under the terms of the GPL license version 3.
@@ -13,16 +10,16 @@
 CongruenceCommon.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
-
 from libsemigroups_pybind11 import (
     Presentation,
     congruence_kind,
     presentation,
 )
 
 
-def check_congruence_common_return_policy(TestType):
+def check_congruence_common_return_policy(
+    TestType,
+):  # pylint: disable=missing-function-docstring, invalid-name
     p = Presentation("ab")
     presentation.add_rule(p, "abab", "a" * 6)
     presentation.add_rule(p, "ba", "ababbb")

tests/runner.py

@@ -1,7 +1,4 @@
 # -*- coding: utf-8 -*-
-# pylint: disable=no-name-in-module, missing-function-docstring
-# pylint: disable=missing-class-docstring, invalid-name
-
 # Copyright (c) 2021-2024 J. D. Mitchell
 #
 # Distributed under the terms of the GPL license version 3.
@@ -17,12 +14,14 @@
 
 from libsemigroups_pybind11 import ReportGuard
 
-n = 0
+N = 0
 
 
-def check_runner(x, t=timedelta(microseconds=1000)):
-    global n  # pylint: disable=global-statement
-    n = 0
+def check_runner(
+    x, t=timedelta(microseconds=1000)
+):  # pylint: disable=missing-function-docstring
+    global N  # pylint: disable=global-statement
+    N = 0
     ReportGuard(False)
 
     assert not x.stopped()
@@ -47,9 +46,9 @@ def check_runner(x, t=timedelta(microseconds=1000)):
         pass
 
     def func():
-        global n  # pylint: disable=global-statement
-        n += 1
-        return n >= 2
+        global N  # pylint: disable=global-statement
+        N += 1
+        return N >= 2
 
     x.run_until(func)
 

tests/test_action.py

@@ -11,8 +11,7 @@
 arising from action.*pp in libsemigroups.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
-# pylint: disable=redefined-outer-name, no-member
+# pylint: disable=missing-function-docstring
 
 import pytest
 
@@ -30,8 +29,8 @@
 from libsemigroups_pybind11.bmat8 import row_space_basis, col_space_basis
 
 
-@pytest.fixture
-def right_actions():
+@pytest.fixture(name="right_actions")
+def fixture_right_actions():
     seed = row_space_basis(
         BMat8([[1, 1, 1, 0], [1, 1, 0, 0], [0, 1, 0, 1], [0, 1, 0, 0]])
     )
@@ -142,8 +141,8 @@ def right_actions():
     return result
 
 
-@pytest.fixture
-def left_actions():
+@pytest.fixture(name="left_actions")
+def fixture_left_actions():
     seed = col_space_basis(
         BMat8([[1, 1, 1, 0], [1, 1, 0, 0], [0, 1, 0, 1], [0, 1, 0, 0]])
     )
@@ -205,8 +204,8 @@ def left_actions():
     return result
 
 
-@pytest.fixture
-def extreme_left_actions():
+@pytest.fixture(name="extreme_left_actions")
+def fixture_extreme_left_actions():
     result = []
     result.append(
         LeftAction(

tests/test_adapters.py

@@ -11,7 +11,7 @@
 arising from adapters.*pp in libsemigroups.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
+# pylint: disable=missing-function-docstring
 
 import pytest
 
@@ -25,25 +25,25 @@
 
 
 def test_adapters_017():
-    A, B, C = bmat8.random(), BMat8(0), bmat8.one()
-    right = ImageRightAction(point=A, element=A)
+    a, b, c = bmat8.random(), BMat8(0), bmat8.one()
+    right = ImageRightAction(point=a, element=a)
     # Point1, Point2, Element -> Point1 = Point2 ^ Element
-    B = right(A, C)
-    assert B == bmat8.row_space_basis(A)
+    b = right(a, c)
+    assert b == bmat8.row_space_basis(a)
 
-    B = right(C, A)
-    assert B == bmat8.row_space_basis(A)
+    b = right(c, a)
+    assert b == bmat8.row_space_basis(a)
 
-    left = ImageLeftAction(point=A, element=A)
-    B = left(A, C)
-    assert B == bmat8.col_space_basis(A)
+    left = ImageLeftAction(point=a, element=a)
+    b = left(a, c)
+    assert b == bmat8.col_space_basis(a)
 
-    B = left(C, A)
-    assert B == bmat8.col_space_basis(A)
+    b = left(c, a)
+    assert b == bmat8.col_space_basis(a)
 
-    B = right(A, A)
-    C = left(bmat8.transpose(A), bmat8.transpose(A))
-    assert B == bmat8.transpose(C)
+    b = right(a, a)
+    c = left(bmat8.transpose(a), bmat8.transpose(a))
+    assert b == bmat8.transpose(c)
 
 
 def test_image_right_action_pperm_pperm():

tests/test_bipart.py

@@ -10,7 +10,7 @@
 This module contains some tests for bipartitions.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
+# pylint: disable=missing-function-docstring, invalid-name
 
 from itertools import chain
 

tests/test_bmat8.py

@@ -11,7 +11,7 @@
 arising from bmat8.*pp in libsemigroups.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
+# pylint: disable=missing-function-docstring
 
 import pytest
 from libsemigroups_pybind11 import BMat8, LibsemigroupsError

tests/test_cong.py

@@ -10,7 +10,7 @@
 This file contains tests for Congruence from libsemigroups_pybind11.
 """
 
-# pylint: disable=no-name-in-module, missing-function-docstring, invalid-name
+# pylint: disable=missing-function-docstring
 
 from datetime import timedelta
 

tests/test_forest.py

@@ -6,7 +6,7 @@
 #
 # The full license is in the file LICENSE, distributed with this software.
 
-# pylint: disable=missing-function-docstring, redefined-outer-name
+# pylint: disable=missing-function-docstring
 
 """
 This module contains some tests for the libsemigroups_pybind11 functionality
@@ -18,13 +18,12 @@
 from libsemigroups_pybind11 import Forest, UNDEFINED
 
 
-@pytest.fixture
+@pytest.fixture(name="f")
 def forest_fixture():
     return Forest([UNDEFINED, 0, 1, 2, 3], [UNDEFINED, 0, 0, 0, 0])
 
 
-def test_forest_return_policy(forest_fixture):
-    f = forest_fixture
+def test_forest_return_policy(f):
     assert f.add_nodes(2) is f
     assert f.init(2) is f
     assert f.labels() is not f.labels()

tests/test_froidure_pin.py

@@ -1,6 +1,4 @@
 # -*- coding: utf-8 -*-
-# pylint: disable=no-name-in-module, missing-function-docstring
-# pylint: disable=missing-class-docstring, invalid-name, redefined-outer-name
 
 # Copyright (c) 2021-2024 J. D. Mitchell
 #
@@ -12,6 +10,8 @@
 This module contains some tests for FroidurePin
 """
 
+# pylint: disable=missing-function-docstring, invalid-name
+
 from datetime import timedelta
 import pytest
 
@@ -285,13 +285,13 @@ def check_froidure_pin_transf2(T):
     ]
 
 
-@pytest.fixture
-def checks_for_generators():
+@pytest.fixture(name="checks_for_generators")
+def fixture_checks_for_generators():
     return (check_constructors,)
 
 
-@pytest.fixture
-def checks_for_froidure_pin():
+@pytest.fixture(name="checks_for_froidure_pin")
+def fixture_checks_for_froidure_pin():
     return (
         check_settings,
         check_mem_compare,