Add ruff flake8-pytest-style rules

parsons/box/box.py

@@ -239,8 +239,7 @@ def upload_table_to_folder_id(
 
         if format not in self.ALLOWED_FILE_FORMATS:
             raise ValueError(
-                f"Format argument to upload_table() must be in one "
-                f'of {self.ALLOWED_FILE_FORMATS}; found "{format}"'
+                f'Format argument to upload_table() must be in one of {self.ALLOWED_FILE_FORMATS}; found "{format}"'
             )
 
         # Create a temp directory in which we will let Parsons create a
@@ -318,8 +317,7 @@ def get_table_by_file_id(self, file_id, format="csv") -> Table:
         """
         if format not in self.ALLOWED_FILE_FORMATS:
             raise ValueError(
-                f"Format argument to upload_table() must be in one "
-                f'of {self.ALLOWED_FILE_FORMATS}; found "{format}"'
+                f'Format argument to upload_table() must be in one of {self.ALLOWED_FILE_FORMATS}; found "{format}"'
             )
 
         # Temp file will be around as long as enclosing process is running,

parsons/capitol_canary/capitol_canary.py

@@ -259,14 +259,12 @@ def create_advocate(
 
         if (sms_optin or sms_optout) and not phone:
             raise ValueError(
-                "When opting an advocate in or out of SMS messages, you must specify a valid "
-                "phone and one or more campaigns"
+                "When opting an advocate in or out of SMS messages, you must specify a valid phone and one or more campaigns"
             )
 
         if (email_optin or email_optout) and not email:
             raise ValueError(
-                "When opting an advocate in or out of email messages, you must specify a valid "
-                "email address and one or more campaigns"
+                "When opting an advocate in or out of email messages, you must specify a valid email address and one or more campaigns"
             )
 
         # Align our arguments with the expected parameters for the API
@@ -353,14 +351,12 @@ def update_advocate(
         # Validate the passed in arguments
         if (sms_optin or sms_optout) and not (phone and campaigns):
             raise ValueError(
-                "When opting an advocate in or out of SMS messages, you must specify a valid "
-                "phone and one or more campaigns"
+                "When opting an advocate in or out of SMS messages, you must specify a valid phone and one or more campaigns"
             )
 
         if (email_optin or email_optout) and not (email and campaigns):
             raise ValueError(
-                "When opting an advocate in or out of email messages, you must specify a valid "
-                "email address and one or more campaigns"
+                "When opting an advocate in or out of email messages, you must specify a valid email address and one or more campaigns"
             )
 
         # Align our arguments with the expected parameters for the API

parsons/donorbox/donorbox.py

@@ -220,6 +220,6 @@ def _date_format_helper(self, date_string):
             except ValueError:
                 continue
         raise ValueError(
-            f"The date you supplied, {date_string}, is not a valid Donorbox format."
+            f"The date you supplied, {date_string}, is not a valid Donorbox format. "
             "Try the following formats: YYYY-mm-dd YYYY/mm/dd YYYYmmdd dd-mm-YYYY"
         )

parsons/nation_builder/nation_builder.py

@@ -37,7 +37,7 @@ def __init__(self, slug: Optional[str] = None, access_token: Optional[str] = Non
     @classmethod
     def get_uri(cls, slug: Optional[str]) -> str:
         if slug is None:
-            raise ValueError("slug can't None")
+            raise ValueError("slug can't be None")
 
         if not isinstance(slug, str):
             raise ValueError("slug must be an str")
@@ -50,7 +50,7 @@ def get_uri(cls, slug: Optional[str]) -> str:
     @classmethod
     def get_auth_headers(cls, access_token: Optional[str]) -> dict[str, str]:
         if access_token is None:
-            raise ValueError("access_token can't None")
+            raise ValueError("access_token can't be None")
 
         if not isinstance(access_token, str):
             raise ValueError("access_token must be an str")
@@ -133,7 +133,7 @@ def update_person(self, person_id: str, person: dict[str, Any]) -> dict[str, Any
             A person object with the updated data.
         """
         if person_id is None:
-            raise ValueError("person_id can't None")
+            raise ValueError("person_id can't be None")
 
         if not isinstance(person_id, str):
             raise ValueError("person_id must be a str")

parsons/ngpvan/people.py

@@ -387,17 +387,15 @@ def _valid_search(
             and None in [firstName, lastName, addressLine1, zipOrPostalCode]
             and None in [email]
         ):
-            raise ValueError(
-                """
-                             Person find must include the following minimum
-                             combinations to conduct a search.
-                                - first_name, last_name, email
-                                - first_name, last_name, phone
-                                - first_name, last_name, zip, dob
-                                - first_name, last_name, street_number, street_name, zip
-                                - email
-                            """
-            )
+            error_msg = """
+            Person find must include the following minimum combinations to conduct a search.
+            - first_name, last_name, email
+            - first_name, last_name, phone
+            - first_name, last_name, zip, dob
+            - first_name, last_name, street_number, street_name, zip
+            - email
+            """
+            raise ValueError(error_msg)
 
         return True
 

parsons/ngpvan/scores.py

@@ -103,8 +103,7 @@ def update_score_status(self, score_update_id, status):
 
         if status not in ["pending approval", "approved", "disapproved", "canceled"]:
             raise ValueError(
-                """Valid inputs for status are, 'pending approval',
-                             'approved','disapproved','canceled'"""
+                """Valid inputs for status are, 'pending approval','approved','disapproved','canceled'"""
             )
 
         else:

parsons/notifications/sendmail.py

@@ -191,10 +191,7 @@ def _validate_email_string(self, str):
         self.log.debug(f"Validating email {str}...")
         realname, email_addr = parseaddr(str)
 
-        if not email_addr:
-            raise ValueError("Invalid email address.")
-
-        if not validate_email(email_addr):
+        if not email_addr or not validate_email(email_addr):
             raise ValueError("Invalid email address.")
 
         return True

parsons/targetsmart/targetsmart_api.py

@@ -288,7 +288,7 @@ def district(
             raise ValueError("Search type 'address' requires 'address' argument")
 
         elif search_type not in ["zip", "point", "address"]:
-            raise KeyError("Invalid 'search_type' provided. ")
+            raise KeyError("Invalid 'search_type' provided.")
 
         else:
             pass

parsons/targetsmart/targetsmart_automation.py

@@ -213,8 +213,7 @@ def config_status(self, job_name):
                 logger.info(f"Match job {job_name} configuration error.")
                 #  To Do: Lift up the configuration error.
                 raise ValueError(
-                    "Job configuration failed. If you provided an email"
-                    "address, you will be sent more details."
+                    "Job configuration failed. If you provided an email address, you will be sent more details."
                 )
 
             else:

pyproject.toml

@@ -104,6 +104,7 @@ select = [
     "SIM", # flake8-simplify (SIM)
     "PTH", # flake8-use-pathlib (PTH)
     "ISC", # flake8-implicit-str-concat (ISC)
+    "PT", # flake8-pytest-style (PT)
 ]
 ignore = [
     "E501", # line-too-long (E501)

test/test_actblue/test_actblue.py

@@ -2,6 +2,7 @@
 from pathlib import Path
 from unittest.mock import MagicMock
 
+import pytest
 import requests_mock
 
 from parsons import ActBlue, Table
@@ -90,7 +91,7 @@ def test_error_on_complete_without_download_url(self, m):
 
         m.get(f"{TEST_URI}/csvs/{TEST_ID}", json=mocked_get_response_no_url)
 
-        with self.assertRaises(ValueError):
+        with pytest.raises(ValueError, match="CSV generation failed"):
             self.ab.get_download_url(csv_id=TEST_ID)
 
     @requests_mock.Mocker()
@@ -103,7 +104,7 @@ def test_error_on_unexpected_status(self, m):
 
         m.get(f"{TEST_URI}/csvs/{TEST_ID}", json=mocked_get_response_no_url)
 
-        with self.assertRaises(ValueError):
+        with pytest.raises(ValueError, match="CSV generation failed"):
             self.ab.get_download_url(csv_id=TEST_ID)
 
     @requests_mock.Mocker()

test/test_action_builder/test_action_builder.py

@@ -473,6 +473,7 @@ def test_upsert_connection_missing_identifiers(self, m):
             self.bldr.upsert_connection(self.fake_entity_id)
         with pytest.raises(ValueError, match="Must provide exactly two identifiers"):
             self.bldr.upsert_connection([self.fake_entity_id])
+        with pytest.raises(ValueError, match="Must provide exactly two identifiers"):
             self.bldr.upsert_connection(
                 [self.fake_entity_id, "fake-entity-id-2", "fake-entity-id-3"]
             )

test/test_action_kit.py

@@ -26,20 +26,20 @@ def tearDown(self):
     @mock.patch.dict(os.environ, ENV_PARAMETERS)
     def test_from_environ(self):
         actionkit = ActionKit()
-        self.assertEqual(actionkit.domain, "env_domain")
-        self.assertEqual(actionkit.username, "env_username")
-        self.assertEqual(actionkit.password, "env_password")
+        assert actionkit.domain == "env_domain"
+        assert actionkit.username == "env_username"
+        assert actionkit.password == "env_password"
 
     def test_base_endpoint(self):
         # Test the endpoint
         url = self.actionkit._base_endpoint("user")
-        self.assertEqual(url, "https://domain.actionkit.com/rest/v1/user/")
+        assert url == "https://domain.actionkit.com/rest/v1/user/"
 
         url = self.actionkit._base_endpoint("user", 1234)
-        self.assertEqual(url, "https://domain.actionkit.com/rest/v1/user/1234/")
+        assert url == "https://domain.actionkit.com/rest/v1/user/1234/"
 
         url = self.actionkit._base_endpoint("user", "1234")
-        self.assertEqual(url, "https://domain.actionkit.com/rest/v1/user/1234/")
+        assert url == "https://domain.actionkit.com/rest/v1/user/1234/"
 
     def test_delete_actionfield(self):
         # Test delete actionfield
@@ -495,7 +495,7 @@ def test_paginated_get(self):
         resp_mock.get.side_effect = [first_mock, second_mock]
         self.actionkit.conn = resp_mock
         results = self.actionkit.paginated_get("user", 150, order_by="created_at")
-        self.assertEqual(results.num_rows, 150)
+        assert results.num_rows == 150
         calls = [
             unittest.mock.call(
                 "https://domain.actionkit.com/rest/v1/user/",
@@ -523,9 +523,9 @@ def test_paginated_get_custom_limit(self):
         resp_mock.get.side_effect = [first_mock, second_mock]
         self.actionkit.conn = resp_mock
         results = self.actionkit.paginated_get_custom_limit("user", 150, "value", 102)
-        self.assertEqual(results.num_rows, 102)
-        self.assertEqual(results.column_data("value")[0], 0)
-        self.assertEqual(results.column_data("value")[-1], 101)
+        assert results.num_rows == 102
+        assert results.column_data("value")[0] == 0
+        assert results.column_data("value")[-1] == 101
         calls = [
             unittest.mock.call(
                 "https://domain.actionkit.com/rest/v1/user/",
@@ -785,16 +785,16 @@ def test_bulk_upload_table(self):
             ),
             "fake_page",
         )
-        self.assertEqual(resp_mock.post.call_count, 2)
+        assert resp_mock.post.call_count == 2
         name, args, kwargs = resp_mock.method_calls[1]
-        self.assertEqual(
-            kwargs["data"],
-            {"page": "fake_page", "autocreate_user_fields": 0, "user_fields_only": 0},
-        )
+        assert kwargs["data"] == {
+            "page": "fake_page",
+            "autocreate_user_fields": 0,
+            "user_fields_only": 0,
+        }
         upload_data = kwargs["files"]["upload"].read()
-        self.assertEqual(
-            upload_data.decode(),
-            "user_id,user_customfield1,action_foo\r\n5,yes,123 Main St\r\n",
+        assert (
+            upload_data.decode() == "user_id,user_customfield1,action_foo\r\n5,yes,123 Main St\r\n"
         )
 
     def test_bulk_upload_table_userfields(self):
@@ -804,32 +804,30 @@ def test_bulk_upload_table_userfields(self):
         self.actionkit.bulk_upload_table(
             Table([("user_id", "user_customfield1"), (5, "yes")]), "fake_page"
         )
-        self.assertEqual(resp_mock.post.call_count, 2)
+        assert resp_mock.post.call_count == 2
         name, args, kwargs = resp_mock.method_calls[1]
-        self.assertEqual(
-            kwargs["data"],
-            {"page": "fake_page", "autocreate_user_fields": 0, "user_fields_only": 1},
-        )
-        self.assertEqual(
-            kwargs["files"]["upload"].read().decode(),
-            "user_id,user_customfield1\r\n5,yes\r\n",
-        )
+        assert kwargs["data"] == {
+            "page": "fake_page",
+            "autocreate_user_fields": 0,
+            "user_fields_only": 1,
+        }
+        assert kwargs["files"]["upload"].read().decode() == "user_id,user_customfield1\r\n5,yes\r\n"
 
     def test_table_split(self):
         test1 = Table([("x", "y", "z"), ("a", "b", ""), ("1", "", "3"), ("4", "", "6")])
         tables = self.actionkit._split_tables_no_empties(test1, True, [])
-        self.assertEqual(len(tables), 2)
+        assert len(tables) == 2
         assert_matching_tables(tables[0], Table([("x", "y"), ("a", "b")]))
         assert_matching_tables(tables[1], Table([("x", "z"), ("1", "3"), ("4", "6")]))
 
         test2 = Table([("x", "y", "z"), ("a", "b", "c"), ("1", "2", "3"), ("4", "5", "6")])
         tables2 = self.actionkit._split_tables_no_empties(test2, True, [])
-        self.assertEqual(len(tables2), 1)
+        assert len(tables2) == 1
         assert_matching_tables(tables2[0], test2)
 
         test3 = Table([("x", "y", "z"), ("a", "b", ""), ("1", "2", "3"), ("4", "5", "6")])
         tables3 = self.actionkit._split_tables_no_empties(test3, False, ["z"])
-        self.assertEqual(len(tables3), 2)
+        assert len(tables3) == 2
         assert_matching_tables(tables3[0], Table([("x", "y"), ("a", "b")]))
         assert_matching_tables(
             tables3[1], Table([("x", "y", "z"), ("1", "2", "3"), ("4", "5", "6")])