bugfix: post/put for models with a CamelCase name

Author: dakra

atomx/models.py

@@ -71,6 +71,11 @@ def __dir__(self):
     def __repr__(self):
         return '{}({})'.format(self.__class__.__name__, pprint.pformat(self.json))
 
+    @property
+    def _resource_name(self):
+        from atomx.utils import model_name_to_rest
+        return model_name_to_rest(self.__class__.__name__)
+
     @property
     def _dirty_json(self):
         return {k: self._attributes[k] for k in self._dirty}
@@ -91,7 +96,7 @@ def create(self, session=None):
         session = session or self.session
         if not session:
             raise NoSessionError
-        res = session.post(self.__class__.__name__, json=self.json)
+        res = session.post(self._resource_name, json=self.json)
         self.__init__(session=session, **res)
         return self
 
@@ -110,7 +115,7 @@ def save(self, session=None):
         session = session or self.session
         if not session:
             raise NoSessionError
-        res = session.put(self.__class__.__name__, self.id, json=self._dirty_json)
+        res = session.put(self._resource_name, self.id, json=self._dirty_json)
         self.__init__(session=session, **res)
         return self
 
@@ -138,7 +143,7 @@ def reload(self, session=None):
         if not hasattr(self, 'id'):
             raise ModelNotFoundError("Can't reload without 'id' parameter. "
                                      "Forgot to save() first?")
-        res = session.get(self.__class__.__name__ + '/' + str(self.id))
+        res = session.get(self._resource_name + '/' + str(self.id))
         self.__init__(session=session, **res.json)
         return self
 

atomx/utils.py

@@ -46,3 +46,23 @@ def get_attribute_model_name(attribute):
     :return: name of the model or False
     """
     return get_model_name(re.sub('(_filter|_include|_exclude)$', '', attribute))
+
+
+def model_name_to_rest(name):
+    """Gets a name of a :mod:`atomx.models` and transforms it in the
+    resource name for the atomx api.
+    E.g.::
+
+        >>> assert model_name_to_rest('ConversionPixels') ==  'conversion-pixels'
+        >>> assert model_name_to_rest('OperatingSystem') == 'operating-system'
+        >>> assert model_name_to_rest('Advertiser') == 'Advertiser'
+
+    :param str name: Name of the model to convert
+    :return: :class:`str` with the transformed name.
+    """
+    r = name[0]
+    for i in range(1, len(name)):
+        if name[i].isupper():
+            r += '-'
+        r += name[i]
+    return r.lower()