bugfix: extra fields are added when push results 35/35035/1
authorSerenaFeng <feng.xiaowei@zte.com.cn>
Fri, 19 May 2017 07:18:34 +0000 (15:18 +0800)
committerSerenaFeng <feng.xiaowei@zte.com.cn>
Fri, 19 May 2017 07:18:34 +0000 (15:18 +0800)
extra fields like 'carriers' & 'miss_checks' 'db_key'
are added when pushing result to TestDB

Change-Id: Ic8d0751bca14b3b3260df42c0f4f07456c9d5e05
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>
utils/test/testapi/opnfv_testapi/common/check.py
utils/test/testapi/opnfv_testapi/resources/handlers.py

index 4d9902c..67e8fbd 100644 (file)
@@ -61,7 +61,7 @@ def no_body(xstep):
 def miss_fields(xstep):
     @functools.wraps(xstep)
     def wrap(self, *args, **kwargs):
-        fields = kwargs.get('miss_fields')
+        fields = kwargs.pop('miss_fields', [])
         if fields:
             for miss in fields:
                 miss_data = self.json_args.get(miss)
@@ -75,7 +75,7 @@ def miss_fields(xstep):
 def carriers_exist(xstep):
     @functools.wraps(xstep)
     def wrap(self, *args, **kwargs):
-        carriers = kwargs.get('carriers')
+        carriers = kwargs.pop('carriers', {})
         if carriers:
             for table, query in carriers:
                 exist = yield self._eval_db_find_one(query(), table)
@@ -102,7 +102,7 @@ def new_not_exists(xstep):
 def updated_one_not_exist(xstep):
     @functools.wraps(xstep)
     def wrap(self, data, *args, **kwargs):
-        db_keys = kwargs.get('db_keys')
+        db_keys = kwargs.pop('db_keys', [])
         query = self._update_query(db_keys, data)
         if query:
             to_data = yield self._eval_db_find_one(query)
index 2426805..2fc31ca 100644 (file)
@@ -85,7 +85,8 @@ class GenericApiHandler(web.RequestHandler):
         """
         data = self.table_cls.from_dict(self.json_args)
         for k, v in kwargs.iteritems():
-            data.__setattr__(k, v)
+            if k != 'query':
+                data.__setattr__(k, v)
 
         if self.table != 'results':
             data.creation_date = datetime.now()