lucaspalomodevelop/eventcally
1
0
Fork 0
mirror of https://github.com/lucaspalomodevelop/eventcally.git synced 2026-03-13 00:07:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

API loggt bereits behandelte Fehler #284

Browse Source
This commit is contained in:
Daniel Grams 2021-08-31 11:31:10 +02:00
parent c03cc7e17b
commit 3548525a00
2 changed files with 51 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
project/api/__init__.py
View File

@ -2,6 +2,8 @@ from apispec import APISpec
from apispec.exceptions import DuplicateComponentNameError
from apispec.ext.marshmallow import MarshmallowPlugin
from flask import url_for
from flask.globals import current_app
from flask.signals import got_request_exception
from flask_apispec.extension import FlaskApiSpec
from flask_babelex import gettext
from flask_marshmallow import Marshmallow
@ -73,11 +75,12 @@ class RestApi(Api):
self.fill_validation_data(err, data)
# Call default error handler that propagates error further
try:
super().handle_error(err)
except Exception:
if not schema:
raise
if code >= 500:
try:
super().handle_error(err)
except Exception:
if not schema:
raise
if data and "message" in data:
data["message"] = gettext(data["message"])

71
tests/api/test___init__.py
View File

@ -8,10 +8,12 @@ def test_handle_error_unique(app):
error = make_unique_violation()
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Unique Violation"
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Unique Violation"
def test_handle_error_checkViolation(app):
@ -19,10 +21,12 @@ def test_handle_error_checkViolation(app):
error = make_check_violation()
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Check Violation"
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Check Violation"
def test_handle_error_integrity(app):
@ -30,10 +34,12 @@ def test_handle_error_integrity(app):
error = make_integrity_error("custom")
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Integrity Error"
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 400
assert data["name"] == "Integrity Error"
def test_handle_error_httpException(app):
@ -41,9 +47,11 @@ def test_handle_error_httpException(app):
error = InternalServerError()
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 500
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 500
def test_handle_error_unprocessableEntity(app):
@ -56,11 +64,13 @@ def test_handle_error_unprocessableEntity(app):
error = UnprocessableEntity()
error.exc = validation_error
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 422
assert data["errors"][0]["field"] == "name"
assert data["errors"][0]["message"] == "Required"
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(error)
assert code == 422
assert data["errors"][0]["field"] == "name"
assert data["errors"][0]["message"] == "Required"
def test_handle_error_validationError(app):
@ -69,19 +79,24 @@ def test_handle_error_validationError(app):
args = {"name": ["Required"]}
validation_error = ValidationError(args)
api = RestApi(app)
(data, code) = api.handle_error(validation_error)
assert code == 422
assert data["errors"][0]["field"] == "name"
assert data["errors"][0]["message"] == "Required"
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
(data, code) = api.handle_error(validation_error)
assert code == 422
assert data["errors"][0]["field"] == "name"
assert data["errors"][0]["message"] == "Required"
def test_handle_error_unspecificRaises(app):
error = Exception()
api = RestApi(app)
with pytest.raises(Exception):
api.handle_error(error)
with app.app_context():
app.config["PROPAGATE_EXCEPTIONS"] = False
api = RestApi(app)
with pytest.raises(Exception):
api.handle_error(error)
def test_add_oauth2_scheme(app, utils):