summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorarati.mahimane <arati.mahimane@rackspace.com>2014-04-23 10:39:34 -0500
committerarati.mahimane <arati.mahimane@rackspace.com>2014-04-29 10:34:36 -0500
commit1a918b5bba6c61510846796e90128d4e72f6aeca (patch)
treeebb0d6975bf9c2f5ee8ffdcf33095c00f3bd45fd
parent8e6077e270fc5be3e8d6da8ea9980e6471c1b271 (diff)
downloadwsme-1a918b5bba6c61510846796e90128d4e72f6aeca.tar.gz
Fix error: variable 'kw' referenced before assignment
Change-Id: Ife95d74bf645687dc4595b16434c88ee874c2f95 Closes-bug: #1311742
-rw-r--r--tests/rest/test_args.py20
-rw-r--r--wsme/rest/args.py1
2 files changed, 21 insertions, 0 deletions
diff --git a/tests/rest/test_args.py b/tests/rest/test_args.py
new file mode 100644
index 0000000..4ae3246
--- /dev/null
+++ b/tests/rest/test_args.py
@@ -0,0 +1,20 @@
+import mock
+import unittest
+
+from wsme import exc
+from wsme.rest import args
+from wsme.rest import json
+
+
+class TestArgs(unittest.TestCase):
+
+ def test_args_from_body(self):
+
+ funcdef = mock.MagicMock()
+ body = mock.MagicMock()
+ mimetype = "application/json"
+ funcdef.ignore_extra_args = True
+ json.parse = mock.MagicMock()
+ json.parse.side_effect = (exc.UnknownArgument(""))
+ resp = args.args_from_body(funcdef, body, mimetype)
+ self.assertEqual(resp, ((), {}))
diff --git a/wsme/rest/args.py b/wsme/rest/args.py
index 9b8f2e1..0c4f6cf 100644
--- a/wsme/rest/args.py
+++ b/wsme/rest/args.py
@@ -232,6 +232,7 @@ def args_from_body(funcdef, body, mimetype):
except UnknownArgument:
if not funcdef.ignore_extra_args:
raise
+ kw = {}
return (), kw