Merge pull request #56 from PaulBoon/FixCallToraise_for_status_after_log

Fixing call raise_for_status_after_log(r)()

src/datastation/dataverse/banner_api.py

@@ -20,7 +20,7 @@ def list(self, dry_run: bool = False):
             print(f"Would have sent the following request: {url}")
             return
         r = requests.get(url, headers=headers, params={'unblock-key': self.unblock_key})
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r
 
     def add(self, msg: str, dismissible_by_user: bool = False, lang: str = 'en', dry_run: bool = False):
@@ -41,7 +41,7 @@ def add(self, msg: str, dismissible_by_user: bool = False, lang: str = 'en', dry
             print(json.dumps(banner, indent=4))
             return
         r = requests.post(url, headers=headers, params={'unblock-key': self.unblock_key}, json=banner)
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r
 
     def remove(self, banner_id: int, dry_run: bool = False):
@@ -52,5 +52,5 @@ def remove(self, banner_id: int, dry_run: bool = False):
             print(f"Would have sent the following request: {url}")
             return
         r = requests.delete(url, headers=headers, params={'unblock-key': self.unblock_key})
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r

src/datastation/dataverse/dataverse_api.py

@@ -50,7 +50,7 @@ def get_storage_size(self, dry_run=False):
             return None
         else:
             r = requests.get(url, headers=headers)
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r.json()['data']['message']
 
     def add_role_assignment(self, assignee, role, dry_run=False):

src/datastation/dataverse/file_api.py

@@ -20,5 +20,5 @@ def reingest(self, dry_run=False):
             print_dry_run_message(method='POST', url=url, headers=headers, params=params)
             return None
         r = requests.post(url, headers=headers, params=params)
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r

src/datastation/dataverse/metrics_api.py

@@ -14,5 +14,5 @@ def get_tree(self, dry_run: bool = False):
             print(f"Would have sent the following request: {url}")
             return
         r = requests.get(url)
-        raise_for_status_after_log(r)()
+        raise_for_status_after_log(r)
         return r.json()['data']