Merge pull request #125 from labd/fix/categories-fix

Fix/categories fix

Author: Ardjan-Aalberts

src/commercetools/testing/categories.py

@@ -32,8 +32,9 @@ def _create_from_draft(
             created_at=datetime.datetime.now(datetime.timezone.utc),
             last_modified_at=datetime.datetime.now(datetime.timezone.utc),
             ancestors=[],
-            order_hint="",
+            order_hint=draft.order_hint,
             custom=utils.create_from_draft(draft.custom),
+            parent=draft.parent,
         )
 
 
@@ -50,6 +51,12 @@ def urls(self):
             ("^$", "POST", self.create),
             ("^key=(?P<key>[^/]+)$", "GET", self.get_by_key),
             ("^key=(?P<key>[^/]+)$", "POST", self.update_by_key),
+            ("^key=(?P<key>[^/]+)$", "DELETE", self.delete_by_key),
             ("^(?P<id>[^/]+)$", "GET", self.get_by_id),
             ("^(?P<id>[^/]+)$", "POST", self.update_by_id),
+            ("^(?P<id>[^/]+)$", "DELETE", self.delete_by_id),
         ]
+
+    _actions = {
+        "setDescription": utils.update_attribute("description", "description"),
+    }

tests/platform/test_service_categories.py

@@ -77,23 +77,40 @@ def test_category_update(old_client):
     It doesn't test the actual update itself.
     TODO: See if this is worth testing since we're using a mocking backend
     """
+    parent_category = old_client.categories.create(
+        models.CategoryDraft(
+            key="parent-test-category",
+            slug=models.LocalizedString(nl="nl-slug-parent"),
+            name=models.LocalizedString(nl="parent-category"),
+        )
+    )
+
     category = old_client.categories.create(
         models.CategoryDraft(
             key="test-category",
             slug=models.LocalizedString(nl="nl-slug"),
             name=models.LocalizedString(nl="category"),
+            parent=models.CategoryResourceIdentifier(id=parent_category.id),
+            order_hint="0.00001",
+            description=models.LocalizedString(nl="description-nl"),
         )
     )
     assert category.key == "test-category"
+    assert category.order_hint == "0.00001"
+    assert getattr(category.parent, "id") == parent_category.id
 
     category = old_client.categories.update_by_id(
         id=category.id,
         version=category.version,
         actions=[
-            models.CategoryChangeSlugAction(slug=models.LocalizedString(nl="nl-slug2"))
+            models.CategoryChangeSlugAction(slug=models.LocalizedString(nl="nl-slug2")),
+            models.CategorySetDescriptionAction(
+                description=models.LocalizedString(nl="updated-description-nl")
+            ),
         ],
     )
     assert category.key == "test-category"
+    assert category.description == models.LocalizedString(nl="updated-description-nl")
 
     category = old_client.categories.update_by_key(
         key="test-category",