More decorator tests and we no longer let things through the argspec loop that aren't functions or methods

Author: apiguy

flask_classy.py

@@ -308,6 +308,9 @@ def get_true_argspec(method):
         inner_method = cell.cell_contents
         if inner_method is method:
             continue
+        if not inspect.isfunction(inner_method) \
+            and not inspect.ismethod(inner_method):
+            continue
         true_argspec = get_true_argspec(inner_method)
         if true_argspec:
             return true_argspec

test_classy/test_decorators.py

@@ -48,8 +48,12 @@ def test_recursive_with_route_with_parameter():
 
 
 def test_params_decorator():
-    resp = client.get('/decorated/params_decorator/')
+    resp = client.get('/decorated/params_decorator_method/')
     eq_(b"Params Decorator", resp.data)
 
+def test_params_decorator_delete():
+    resp = client.delete('/decorated/1234')
+    eq_(b"Params Decorator Delete 1234", resp.data)
+
 
 

test_classy/view_classes.py

@@ -1,6 +1,11 @@
 from flask_classy import FlaskView, route
 from functools import wraps
 
+VALUE1 = "value1"
+
+def get_value():
+    return VALUE1
+
 class BasicView(FlaskView):
 
     def index(self):
@@ -215,9 +220,14 @@ def post(self):
         return "Post"
 
     @params_decorator("oneval", "anotherval")
-    def params_decorator(self):
+    def params_decorator_method(self):
         return "Params Decorator"
 
+    @params_decorator(get_value(), "value")
+    def delete(self, obj_id):
+        return "Params Decorator Delete " + obj_id
+
+
     @more_recursive(None)
     def get_some(self):
         return "Get Some"