flake8 pytest fixes for actblue and action builder

Author: bmos

test/test_actblue/test_actblue.py

@@ -1,6 +1,7 @@
 import unittest
 from unittest.mock import MagicMock
 
+import pytest
 import requests_mock
 
 from parsons import ActBlue, Table
@@ -87,7 +88,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()
@@ -100,7 +101,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"]
             )