add 1 test case, now 55 test cases for apijson-get

zhangchunlin · zhangchunlin · commit f0c5026e9d84 · 2019-10-11T12:02:54.000+08:00
diff --git a/tests/test.py b/tests/test.py
@@ -743,6 +743,24 @@ def test_apijson_get():
     >>> print(d)
     {'code': 400, 'msg': "model does not have this column: 'nonexist'"}
 
+    >>> #query array with a nonexist column
+    >>> data ='''{
+    ...   "[]":{
+    ...     "@count":4,
+    ...     "@page":0,
+    ...     "user":{
+    ...             "@column":"id,username,nickname,email",
+    ...             "@order":"id-",
+    ...             "@role":"ADMIN",
+    ...             "nonexist":1
+    ...         }
+    ...     }
+    ... }'''
+    >>> r = handler.post('/apijson/get', data=data, pre_call=pre_call_as("admin"), middlewares=[])
+    >>> d = json_loads(r.data)
+    >>> print(d)
+    {'code': 400, 'msg': "non-existent column or not support item: 'nonexist'"}
+
     >>> #Association query: Two tables, one to one,ref path is absolute path
     >>> data ='''{
     ...     "moment":{},
diff --git a/uliweb_apijson/apijson/views.py b/uliweb_apijson/apijson/views.py
@@ -254,7 +254,7 @@ def _get_filter_condition(self,model,model_param,item,expr=False):
         elif hasattr(model,n):
             return getattr(model.c,n)==model_param[n]
         else:
-            raise UliwebError("not support item: '%s'"%(item))
+            raise UliwebError("non-existent column or not support item: '%s'"%(item))
 
     def head(self):
         try:
