try to convert strings to integer when the API expects one (#51)

sometimes the API will return a value as a string, while expecting to
get the very same value as an integer, see
https://projects.theforeman.org/issues/27673

Author: evgeni

apypie/action.py

@@ -83,6 +83,12 @@ def _validate(self, params, values, data=None, files=None, path=None):
                             self._validate(param_description.params, item, path=self._add_to_path(path, param_description.name, num))
                     elif param_description.expected_type == 'hash':
                         self._validate(param_description.params, value, path=self._add_to_path(path, param_description.name))
+                if (param_description.expected_type == 'numeric' and isinstance(value, basestring)):
+                    try:
+                        value = int(value)
+                    except ValueError:
+                        # this will be caught in the next check
+                        pass
                 if ((param_description.expected_type == 'boolean' and not isinstance(value, bool) and not (isinstance(value, int) and value in [0, 1]))
                         or (param_description.expected_type == 'numeric' and not isinstance(value, int))
                         or (param_description.expected_type == 'string' and not isinstance(value, (basestring, int)))):

tests/test_action.py

@@ -138,6 +138,11 @@ def test_action_validate_valid_numeric(api):
     action.validate({'id': 1})
 
 
+def test_action_validate_valid_numeric_as_string(api):
+    action = api.resource('comments').action('archive')
+    action.validate({'id': '1'})
+
+
 def test_action_validate_invalid_string(resource):
     action = resource.action('create')
     with pytest.raises(ValueError) as excinfo: