Remove unnecessary blank lines and handle 404 error when storing file result

- Remove multiple unnecessary blank lines in `main.py`
- Handle the 404 error when attempting to store a file result in `reporting.py`
This commit is contained in:
Jonathon Broughton
2024-08-04 17:47:14 +01:00
parent 46b6c0d68c
commit 7517682456
3 changed files with 6 additions and 1098 deletions
-1087
View File
File diff suppressed because it is too large Load Diff
+1 -6
View File
@@ -73,11 +73,6 @@ def automate_function(
automate_context, health_objects, function_inputs.density_level
)
colorise_densities(automate_context, health_objects)
# Wrap up the analysis by marking the run either successful or failed.
@@ -114,7 +109,7 @@ def automate_function(
file_name = write_pdf_to_temp(report)
print("------------------------------------------------")
print(f"| {commit_details["server_url"]} |")
print(f"| {commit_details['server_url']} |")
print("------------------------------------------------")
safe_store_file_result(automate_context, file_name)
+5 -5
View File
@@ -276,14 +276,14 @@ def safe_store_file_result(automate_context: AutomationContext, file_name: str):
# Attempt to store the file
automate_context.store_file_result(file_name)
except httpx.HTTPStatusError as e:
if e.response.status_code == 404:
if e.response.status_code != 404:
raise
else:
# Handle the 404 error
error_message = f"Unable to store file: {file_name}. Error: {str(e)}"
print(error_message) # For logging purposes
automate_context.mark_run_exception(error_message)
else:
raise
# automate_context.mark_run_exception(error_message)
finally:
# Restore the original URL
automate_context.automation_run_data.speckle_server_url = original_url