Merge pull request #89 from mattmanley/update_create_task

Update create task method

Author: mattmanley

maproulette/api/task.py

@@ -36,6 +36,8 @@ def create_task(self, data):
         :param data: a JSON input containing task details
         :returns: the API response from the POST request
         """
+        if self.is_task_model(data):
+            data = TaskModel.to_dict(data)
         response = self.post(
             endpoint="/task",
             body=data)

tests/test_task_api.py

@@ -26,6 +26,18 @@ def test_get_task_history(self, mock_request, api_instance=api):
             f'{self.url}/task/{test_task_id}/history',
             params=None)
 
+    @patch('maproulette.api.maproulette_server.requests.Session.post')
+    def test_create_task(self, mock_request, api_instance=api):
+        geometries = test_geojson['features'][0]['geometry']
+        test_task_model = maproulette.TaskModel(name='test_task',
+                                                parent='12345',
+                                                geometries=geometries)
+        api_instance.create_task(test_task_model)
+        mock_request.assert_called_once_with(
+            f'{self.url}/task',
+            json=test_task_model.to_dict(),
+            params=None)
+
     @patch('maproulette.api.maproulette_server.requests.Session.post')
     def test_create_tasks(self, mock_request, api_instance=api):
         test_tasks = []
@@ -77,7 +89,7 @@ def test_create_cooperative_task(self, mock_request, api_instance=api):
         api_instance.create_task(test_task)
         mock_request.assert_called_once_with(
             f'{self.url}/task',
-            json=test_task,
+            json=test_task.to_dict(),
             params=None)
 
     @patch('maproulette.api.maproulette_server.requests.Session.put')

tox.ini

@@ -1,5 +1,5 @@
 [tox]
-envlist = py36
+minversion = 3.6.0
 
 [testenv]
 deps =