Fix a bunch of lint issues.

ionelmc · ionelmc · commit b8e223a2c420 · 2023-12-13T19:32:14.000+02:00
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,8 +1,11 @@
 [build-system]
 requires = [
-    "setuptools>=30.3.0",
-    "setuptools_scm>=3.3.1",
+    "setuptools>=64",
+    "setuptools_scm>=8",
 ]
+build-backend = "setuptools.build_meta"
+
+[tool.setuptools_scm]
 
 [tool.ruff.per-file-ignores]
 "ci/*" = ["S"]
@@ -14,6 +17,11 @@ ignore = [
     "S101", # flake8-bandit assert
     "S308", # flake8-bandit suspicious-mark-safe-usage
     "E501", # pycodestyle line-too-long
+    "DTZ001",
+    "PT011",
+    "PT012",
+    "B004",
+    "S102",
 ]
 line-length = 140
 select = [
diff --git a/src/lazy_object_proxy/slots.py b/src/lazy_object_proxy/slots.py
@@ -98,8 +98,8 @@ def __wrapped__(self, __getattr__=object.__getattribute__, __setattr__=object.__
         except AttributeError:
             try:
                 factory = __getattr__(self, '__factory__')
-            except AttributeError:
-                raise ValueError("Proxy hasn't been initiated: __factory__ is missing.")
+            except AttributeError as exc:
+                raise ValueError("Proxy hasn't been initiated: __factory__ is missing.") from exc
             target = factory()
             __setattr__(self, '__target__', target)
             return target
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -30,7 +30,7 @@ class FakeModule:
             else:
                 raise RuntimeError('Unsupported param: %r.' % name)
 
-            Proxy
+            Proxy  # noqa: B018
 
         return FakeModule
 
@@ -63,7 +63,7 @@ class submod(lop_implementation):
         return lop_implementation
 
 
-@pytest.fixture(scope='function')
+@pytest.fixture
 def lop(request, lop_subclass):
     if request.node.get_closest_marker('xfail_subclass'):
         request.applymarker(
diff --git a/tests/test_async_py3.py b/tests/test_async_py3.py
@@ -249,7 +249,7 @@ async def foo():
     aw = coro.__await__()
     next(aw)
     with pytest.raises(ZeroDivisionError):
-        aw.throw(ZeroDivisionError, None, None)
+        aw.throw(ZeroDivisionError)
     assert N == 102
 
 
diff --git a/tests/test_lazy_object_proxy.py b/tests/test_lazy_object_proxy.py
@@ -771,7 +771,7 @@ def test_iteration(lop):
 
     wrapper = lop.Proxy(lambda: items)
 
-    result = [x for x in wrapper]
+    result = [x for x in wrapper]  # noqa: C416
 
     assert result == items
 
@@ -816,13 +816,11 @@ def function1(*args, **kwargs):
 
     function2 = lop.Proxy(lambda: function1)
 
-    table = dict()
-    table[function1] = True
+    table = {function1: True}
 
     assert table.get(function2)
 
-    table = dict()
-    table[function2] = True
+    table = {function2: True}
 
     assert table.get(function1)
 
@@ -1458,12 +1456,12 @@ def test_repr_doesnt_consume(lop):
 def test_derived_new(lop):
     class DerivedObjectProxy(lop.Proxy):
         def __new__(cls, wrapped):
-            instance = super(DerivedObjectProxy, cls).__new__(cls)
+            instance = super().__new__(cls)
             instance.__init__(wrapped)
             return instance
 
         def __init__(self, wrapped):
-            super(DerivedObjectProxy, self).__init__(wrapped)
+            super().__init__(wrapped)
 
     def function():
         return 123
@@ -1540,9 +1538,9 @@ class DerivedObjectProxy(lop.Proxy):
         def __getattr__(self, name):
             accessed.append(name)
             try:
-                __getattr__ = super(DerivedObjectProxy, self).__getattr__
+                __getattr__ = super().__getattr__
             except AttributeError as e:
-                raise RuntimeError(str(e))
+                raise RuntimeError(str(e)) from e
             return __getattr__(name)
 
     function.attribute = 1
@@ -1884,7 +1882,7 @@ class LazyProxy(lop.Proxy):
         name = None
 
         def __init__(self, func, **lazy_attr):
-            super(LazyProxy, self).__init__(func)
+            super().__init__(func)
             for attr, val in lazy_attr.items():
                 setattr(self, attr, val)
 
@@ -1904,7 +1902,7 @@ class Foo:
 
     class LazyProxy(lop.Proxy):
         def __init__(self, func, **lazy_attr):
-            super(LazyProxy, self).__init__(func)
+            super().__init__(func)
             for attr, val in lazy_attr.items():
                 object.__setattr__(self, attr, val)
 
@@ -1915,20 +1913,20 @@ def __init__(self, func, **lazy_attr):
 
 class FSPathMock:
     def __fspath__(self):
-        return '/tmp'
+        return '/foobar'
 
 
 @pytest.mark.skipif(not hasattr(os, 'fspath'), reason='No os.fspath support.')
 def test_fspath(lop):
-    assert os.fspath(lop.Proxy(lambda: '/tmp')) == '/tmp'
-    assert os.fspath(lop.Proxy(FSPathMock)) == '/tmp'
+    assert os.fspath(lop.Proxy(lambda: '/foobar')) == '/foobar'
+    assert os.fspath(lop.Proxy(FSPathMock)) == '/foobar'
     with pytest.raises(TypeError) as excinfo:
         os.fspath(lop.Proxy(lambda: None))
     assert '__fspath__() to return str or bytes, not NoneType' in excinfo.value.args[0]
 
 
 def test_fspath_method(lop):
-    assert lop.Proxy(FSPathMock).__fspath__() == '/tmp'
+    assert lop.Proxy(FSPathMock).__fspath__() == '/foobar'
 
 
 def test_resolved_new(lop):
