Add the suggestions

fixity/storage_service.py

@@ -1,6 +1,7 @@
 import calendar
 import json
 from datetime import datetime
+from typing import Dict
 
 import requests
 from sqlalchemy.orm.exc import NoResultFound
@@ -23,7 +24,7 @@ class StorageServiceError(Exception):
     itself will not return, but the caller still needs access to it.
     """
 
-    def __init__(self, message, report=None):
+    def __init__(self, message: str, report: Dict[str, str] = None) -> None:
         self.report = report
         super().__init__(message)
 

tests/test_fixity.py

@@ -652,11 +652,8 @@ def test_main_handles_exception_if_environment_key_is_missing(
 
 @mock.patch("requests.get")
 def test_scanall_handles_exception_if_storage_service_is_not_connected(
-    _get: mock.Mock, monkeypatch: pytest.MonkeyPatch
+    _get: mock.Mock, environment: None
 ) -> None:
-    monkeypatch.setenv("STORAGE_SERVICE_URL", STORAGE_SERVICE_URL)
-    monkeypatch.setenv("STORAGE_SERVICE_USER", "")
-    monkeypatch.setenv("STORAGE_SERVICE_KEY", "")
     aip_id1 = str(uuid.uuid4())
     aip_id2 = str(uuid.uuid4())
     _get.side_effect = [
@@ -687,6 +684,6 @@ def test_scanall_handles_exception_if_storage_service_is_not_connected(
 
     fixity.main(["scanall"], stream=stream)
 
-    assert storage_service.StorageServiceError(  # type: ignore
+    assert storage_service.StorageServiceError(
         f'Storage service at "{STORAGE_SERVICE_URL}" failed authentication while requesting AIPs'
     )