Remove test route and capture unknown exceptions in sentry

thatgardnerone · thatgardnerone · commit 713f7492855b · 2024-11-20T20:18:56.000Z
diff --git a/app/__init__.py b/app/__init__.py
@@ -11,6 +11,7 @@
 
 
 def create_app(test_config=None):
+    # -- App Config --
     app = Flask(
         __name__,
         instance_relative_config=True,
@@ -56,9 +57,6 @@ def before_request():
         request.scheme = app.config["PREFERRED_URL_SCHEME"]
 
     # --- Routes ---
-    @app.route('/debug-sentry')
-    def trigger_error():
-        division_by_zero = 1 / 0
 
     # --- Controllers ---
     from app.http.controllers import dashboard_controller
diff --git a/app/http/controllers/dashboard_controller.py b/app/http/controllers/dashboard_controller.py
@@ -1,24 +1,23 @@
+import csv
+import os
+import tempfile
 import tracemalloc
 from time import time
-from typing import Union
 
-from flask import Blueprint, Response, jsonify, request
-from flask_inertia import lazy_include, render_inertia
-import sympy as sp
 import numpy as np
+import sympy as sp
+from flask import Blueprint, request
+from flask_inertia import lazy_include, render_inertia
 from picos import SolutionFailure
-import os
-import csv
-import tempfile
-
+from sentry_sdk import capture_exception
 from werkzeug.datastructures.file_storage import FileStorage
 
-import tests
 from app.models.safety_barrier import SafetyBarrier
 from app.models.stability import Stability
 
 bp = Blueprint("dashboard", __name__)
 
+
 def calculate_result() -> dict:
     """
     Calculate the result of the user's input.
@@ -50,6 +49,7 @@ def calculate_result() -> dict:
             "error": "An unknown error occurred.",
             "description": str(e),
         }
+        capture_exception(e)
 
     time_taken = time() - start_time
 
@@ -138,6 +138,7 @@ def index():
         },
     )
 
+
 def _parse_uploaded_files(data: dict) -> dict:
     """
     Parse the uploaded files and add them to the data dictionary.
@@ -173,6 +174,7 @@ def _parse_uploaded_files(data: dict) -> dict:
 
     return data
 
+
 def __load_mosek_license(_file: FileStorage) -> bool:
     """
     Read the MOSEK licence file into this request.
@@ -186,4 +188,4 @@ def __load_mosek_license(_file: FileStorage) -> bool:
 
         temp.close()
 
-    return True
+    return True
