[ BB2-1288 ] Code cleanup (Linting & Style) (#10)

* Update .py files with black & flake8

* Remove not needed name & verbose_name

Author: dtisza1

cms_bluebutton/cms_bluebutton.py

@@ -22,8 +22,6 @@
 
 
 class BlueButton:
-    name = "bb2"
-    verbose_name = "Blue Button 2.0 SDK Package"
 
     def __init__(self, config=DEFAULT_CONFIG_FILE_LOCATION):
         self.client_id = None

cms_bluebutton/tests/fixtures/token_response.py

@@ -1,2 +1,16 @@
-TOKEN_RESPONSE = {'access_token': 'sj27kLVRk2sQeLqSKFte5T5pgKM78U', 'expires_in': 36000, 'token_type': 'Bearer', 'scope': 'introspection patient/Coverage.read patient/ExplanationOfBenefit.read patient/Patient.read profile', 'refresh_token': 'QjqsforutTRf8IV8Z0GT7KcAZb4099', 'patient': '-19990000000001'}
-REFRESH_TOKEN_RESPONSE = {'access_token': 'sj27kLVRk2sQeLqSKFte5T5pgKM78U-refrehed', 'expires_in': 36000, 'token_type': 'Bearer', 'scope': 'introspection patient/Coverage.read patient/ExplanationOfBenefit.read patient/Patient.read profile', 'refresh_token': 'QjqsforutTRf8IV8Z0GT7KcAZb4099-refreshed', 'patient': '-19990000000001'}
+TOKEN_RESPONSE = {
+    "access_token": "sj27kLVRk2sQeLqSKFte5T5pgKM78U",
+    "expires_in": 36000,
+    "token_type": "Bearer",
+    "scope": "introspection patient/Coverage.read patient/ExplanationOfBenefit.read patient/Patient.read profile",
+    "refresh_token": "QjqsforutTRf8IV8Z0GT7KcAZb4099",
+    "patient": "-19990000000001",
+}
+REFRESH_TOKEN_RESPONSE = {
+    "access_token": "sj27kLVRk2sQeLqSKFte5T5pgKM78U-refrehed",
+    "expires_in": 36000,
+    "token_type": "Bearer",
+    "scope": "introspection patient/Coverage.read patient/ExplanationOfBenefit.read patient/Patient.read profile",
+    "refresh_token": "QjqsforutTRf8IV8Z0GT7KcAZb4099-refreshed",
+    "patient": "-19990000000001",
+}

cms_bluebutton/tests/test_configs.py

@@ -10,15 +10,13 @@ def test_invalid_file_extension():
     # Test extension not matching .json or .yaml
     with pytest.raises(
         ValueError,
-        match=r"Error: Configuration file extension"
-              " must be .json or .yaml for:.*",
+        match=r"Error: Configuration file extension" " must be .json or .yaml for:.*",
     ):
         BlueButton(config="file.xxx")
 
 
 def test_read_config_json_equals_yaml():
-    bb = BlueButton(config=CONFIGS_DIR +
-             "json/bluebutton-sample-config-valid-sbx.json")
+    bb = BlueButton(config=CONFIGS_DIR + "json/bluebutton-sample-config-valid-sbx.json")
 
     # Test dict's are equal for both JSON & YAML
     assert bb._read_config(
@@ -30,17 +28,17 @@ def test_read_config_json_equals_yaml():
 
 def test_valid_config():
     # valid config sbx
-    bb = BlueButton(config=CONFIGS_DIR +
-             "json/bluebutton-sample-config-valid-sbx.json")
+    bb = BlueButton(config=CONFIGS_DIR + "json/bluebutton-sample-config-valid-sbx.json")
     assert bb.base_url == "https://sandbox.bluebutton.cms.gov"
     assert bb.client_id == "<your BB2 client_id here>"
     assert bb.client_secret == "<your BB2 client_secret here>"
     assert bb.callback_url == "https://www.fake-sandbox.com/your/callback/here"
     assert bb.version == 1
 
     # valid config prod
-    bb = BlueButton(config=CONFIGS_DIR +
-             "json/bluebutton-sample-config-valid-prod.json")
+    bb = BlueButton(
+        config=CONFIGS_DIR + "json/bluebutton-sample-config-valid-prod.json"
+    )
     assert bb.base_url == "https://api.bluebutton.cms.gov"
     assert bb.client_id == "<your BB2 client_id here>"
     assert bb.client_secret == "<your BB2 client_secret here>"
@@ -80,8 +78,7 @@ def test_config_setting_missing():
     for setting, file_name in [
         ["client_id", "json/bluebutton-sample-config-missing-id.json"],
         ["client_secret", "json/bluebutton-sample-config-missing-secret.json"],
-        ["callback_url",
-         "json/bluebutton-sample-config-missing-callback.json"],
+        ["callback_url", "json/bluebutton-sample-config-missing-callback.json"],
     ]:
         with pytest.raises(
             ValueError,
@@ -92,6 +89,7 @@ def test_config_setting_missing():
 
 
 def test_config_version_missing_defaults_to_v2():
-    bb = BlueButton(config=CONFIGS_DIR +
-             "json/bluebutton-sample-config-missing-version.json")
+    bb = BlueButton(
+        config=CONFIGS_DIR + "json/bluebutton-sample-config-missing-version.json"
+    )
     assert bb.version == 2

cms_bluebutton/tests/test_fhir_request.py

@@ -61,12 +61,15 @@ def not_found_fhir_request_mock(*args, **kwargs):
 def generate_mock_config():
     return {
         "params": {},
-        "auth_token": AuthorizationToken({
-            "access_token": "fake_access_token",
-            "refresh_token": "fake_refresh_token",
-            "expires_at": datetime.datetime.now(datetime.timezone.utc) + datetime.timedelta(seconds=36000),
-            "patient": "-20140000010000",
-        }),
+        "auth_token": AuthorizationToken(
+            {
+                "access_token": "fake_access_token",
+                "refresh_token": "fake_refresh_token",
+                "expires_at": datetime.datetime.now(datetime.timezone.utc)
+                + datetime.timedelta(seconds=36000),
+                "patient": "-20140000010000",
+            }
+        ),
     }