Remove superfluous exception parens

Author: WhyNotHugo

docs/conf.py

@@ -76,7 +76,7 @@ def github_issue_role(name, rawtext, text, lineno, inliner, options=None, conten
     try:
         issue_num = int(text)
         if issue_num <= 0:
-            raise ValueError()
+            raise ValueError
     except ValueError:
         msg = inliner.reporter.error(f"Invalid GitHub issue: {text}", line=lineno)
         prb = inliner.problematic(rawtext, rawtext, msg)

pyproject.toml

@@ -8,6 +8,7 @@ select = [
     "UP",
     "C4",
     # "TID",
+    "RSE"
 ]
 target-version = "py37"
 

tests/storage/__init__.py

@@ -49,7 +49,7 @@ def get_storage_args(self):
 
         :param collection: The name of the collection to create and use.
         """
-        raise NotImplementedError()
+        raise NotImplementedError
 
     @pytest_asyncio.fixture
     async def s(self, get_storage_args):

tests/storage/servers/davical/__init__.py

@@ -23,7 +23,7 @@ def davical_args(self):
         elif self.storage_class.fileext == ".vcf":
             pytest.skip("No carddav")
         else:
-            raise RuntimeError()
+            raise RuntimeError
 
     @pytest.fixture
     def get_storage_args(self, davical_args, request):

tests/storage/servers/fastmail/__init__.py

@@ -24,7 +24,7 @@ async def inner(collection="test"):
             elif self.storage_class.fileext == ".vcf":
                 args["url"] = "https://carddav.fastmail.com/"
             else:
-                raise RuntimeError()
+                raise RuntimeError
 
             if collection is not None:
                 args = await slow_create_collection(

tests/storage/servers/icloud/__init__.py

@@ -22,7 +22,7 @@ async def inner(collection="test"):
             elif self.storage_class.fileext == ".vcf":
                 args["url"] = "https://contacts.icloud.com/"
             else:
-                raise RuntimeError()
+                raise RuntimeError
 
             if collection is not None:
                 args = slow_create_collection(self.storage_class, args, collection)

tests/storage/test_filesystem.py

@@ -93,7 +93,7 @@ async def test_too_long_uid(self, tmpdir):
     @pytest.mark.asyncio
     async def test_post_hook_inactive(self, tmpdir, monkeypatch):
         def check_call_mock(*args, **kwargs):
-            raise AssertionError()
+            raise AssertionError
 
         monkeypatch.setattr(subprocess, "call", check_call_mock)
 

tests/storage/test_http_with_singlefile.py

@@ -19,7 +19,7 @@ class CombinedStorage(Storage):
 
     def __init__(self, url, path, *, connector, **kwargs):
         if kwargs.get("collection", None) is not None:
-            raise ValueError()
+            raise ValueError
 
         super().__init__(**kwargs)
         self.url = url

tests/system/cli/test_discover.py

@@ -207,13 +207,13 @@ class TestStorage(Storage):
         def __init__(self, require_collection, **kw):
             if require_collection:
                 assert not kw.get("collection")
-                raise exceptions.CollectionRequired()
+                raise exceptions.CollectionRequired
 
         async def get(self, href: str):
-            raise NotImplementedError()
+            raise NotImplementedError
 
         async def list(self) -> List[tuple]:
-            raise NotImplementedError()
+            raise NotImplementedError
 
     from vdirsyncer.cli.utils import storage_names
 

tests/unit/cli/test_fetchparams.py

@@ -106,7 +106,7 @@ def test_failed_strategy(monkeypatch, value_cache):
 
     def strategy(x):
         calls.append(x)
-        raise KeyboardInterrupt()
+        raise KeyboardInterrupt
 
     monkeypatch.setitem(STRATEGIES, "mystrategy", strategy)