PR 4: Samples Endpoints

Author: pulk17

.pycodestylerc

@@ -2,4 +2,4 @@
 count = True
 max-line-length = 120
 exclude=test_diff.py,migrations,venv*,.venv*,parse.py,config.py
-ignore = E701
+ignore = E701,W503

mod_api/__init__.py

@@ -36,4 +36,5 @@
 # Route modules
 from mod_api.routes import auth as auth_routes  # noqa: E402, F401
 from mod_api.routes import runs as runs_routes  # noqa: E402, F401
+from mod_api.routes import samples as samples_routes  # noqa: E402, F401
 from mod_api.routes import system as system_routes  # noqa: E402, F401

mod_api/middleware/error_handler.py

@@ -151,12 +151,12 @@ def convert_api_errors_to_json(response):
             response.data = new_resp.data
             response.mimetype = new_resp.mimetype
             return response
-        if response.status_code == 404:
+        if response.status_code == 404 and not response.is_json:
             new_resp = make_error_response('not_found', 'Resource not found.', http_status=404)
             response.data = new_resp.data
             response.mimetype = new_resp.mimetype
             return response
-        if response.status_code == 405:
+        if response.status_code == 405 and not response.is_json:
             new_resp = make_error_response('method_not_allowed', 'Method not allowed.', http_status=405)
             response.data = new_resp.data
             response.mimetype = new_resp.mimetype

mod_api/routes/runs.py

@@ -235,7 +235,7 @@ def list_runs(
 
         if status_filter == 'queued':
             query = query.outerjoin(TestProgress).filter(
-                TestProgress.id is None)
+                TestProgress.id.is_(None))
         elif status_filter == 'running':
             query = query.join(
                 TestProgress,