From 9f87878becc36b19110e151e802180aa5f7fb305 Mon Sep 17 00:00:00 2001 From: SerenaFeng Date: Tue, 14 Mar 2017 10:06:41 +0800 Subject: provide get_one_exceptions to unify get_xx process add refactor get_metric() Change-Id: I972c77e63a654eeb286c573d75efe842b2887c1d Signed-off-by: SerenaFeng --- qtip/api/controllers/common.py | 19 +++++++++++++++++++ qtip/api/controllers/metric.py | 12 +++--------- 2 files changed, 22 insertions(+), 9 deletions(-) create mode 100644 qtip/api/controllers/common.py diff --git a/qtip/api/controllers/common.py b/qtip/api/controllers/common.py new file mode 100644 index 00000000..6cabbc7f --- /dev/null +++ b/qtip/api/controllers/common.py @@ -0,0 +1,19 @@ +import httplib + +import connexion + +from qtip.base import error + + +def get_one_exceptions(resource): + def _decorator(func): + def _execute(name): + try: + return func(name), httplib.OK + except error.NotFoundError: + return connexion.problem( + httplib.NOT_FOUND, + '{} Not Found'.format(resource), + 'Requested {} `{}` not found.'.format(resource, name)) + return _execute + return _decorator diff --git a/qtip/api/controllers/metric.py b/qtip/api/controllers/metric.py index 86bf70f9..dd4c8ac6 100644 --- a/qtip/api/controllers/metric.py +++ b/qtip/api/controllers/metric.py @@ -9,9 +9,7 @@ import httplib -import connexion - -from qtip.base import error +from qtip.api.controllers import common from qtip.loader import metric @@ -20,13 +18,9 @@ def list_metrics(): return metric_list, httplib.OK +@common.get_one_exceptions(resource='metric') def get_metric(name): - try: metric_spec = metric.MetricSpec(name) return {'name': metric_spec.name, 'abspath': metric_spec.abspath, - 'content': metric_spec.content}, httplib.OK - except error.NotFoundError: - return connexion.problem(httplib.NOT_FOUND, - 'Metric Not Found', - 'Requested metric `' + name + '` not found.') + 'content': metric_spec.content} -- cgit 1.2.3-korg