test: enable PT011

Author: bearomorphism

pyproject.toml

@@ -239,7 +239,6 @@ ignore = [
     "E501",
     "D1",
     "D415",
-    "PT011", # TODO(bearomorphism): enable this rule
     "PT022", # TODO(bearomorphism): enable this rule
 ]
 extend-safe-fixes = [

tests/commands/test_commit_command.py

@@ -248,8 +248,8 @@ def test_commit_when_customized_expected_raised(config, mocker: MockFixture):
 
 @pytest.mark.usefixtures("staging_is_clean")
 def test_commit_when_non_customized_expected_raised(config, mocker: MockFixture):
-    mocker.patch("questionary.prompt", side_effect=ValueError())
-    with pytest.raises(ValueError):
+    mocker.patch("questionary.prompt", side_effect=ValueError("error message"))
+    with pytest.raises(ValueError, match="error message"):
         commands.Commit(config, {})()
 
 

tests/test_exceptions.py

@@ -30,7 +30,7 @@ def test_from_str_with_invalid_values():
     """Test from_str with invalid values."""
     with pytest.raises(KeyError):
         ExitCode.from_str("invalid_name")
-    with pytest.raises(ValueError):
+    with pytest.raises(ValueError, match="is not a valid ExitCode"):
         ExitCode.from_str("999")  # Out of range decimal
     with pytest.raises(KeyError):
         ExitCode.from_str("")