Fix limit handling to not send value of -1

Updated all list commands that use limit to not send a value of -1.
Added tests for all of these cases.

Change-Id: I70ca546c939afdae2cc5df6ad3e155bb44310a4b
Closes-Bug: 1730755

mistralclient/api/v2/action_executions.py

@@ -76,7 +76,7 @@ class ActionExecutionManager(base.ResourceManager):
 
         qparams = {}
 
-        if limit:
+        if limit and limit > 0:
             qparams['limit'] = limit
 
         query_string = ("?%s" % urlparse.urlencode(list(qparams.items()))

mistralclient/api/v2/actions.py

@@ -74,7 +74,7 @@ class ActionManager(base.ResourceManager):
         if marker:
             qparams['marker'] = marker
 
-        if limit:
+        if limit and limit > 0:
             qparams['limit'] = limit
 
         if sort_keys:

mistralclient/api/v2/executions.py

@@ -82,7 +82,7 @@ class ExecutionManager(base.ResourceManager):
         if marker:
             qparams['marker'] = marker
 
-        if limit:
+        if limit and limit > 0:
             qparams['limit'] = limit
 
         if sort_keys:

mistralclient/api/v2/tasks.py

@@ -42,7 +42,7 @@ class TaskManager(base.ResourceManager):
         if marker:
             qparams['marker'] = marker
 
-        if limit:
+        if limit and limit > 0:
             qparams['limit'] = limit
 
         if sort_keys:

mistralclient/api/v2/workflows.py

@@ -82,7 +82,7 @@ class WorkflowManager(base.ResourceManager):
         if marker:
             qparams['marker'] = marker
 
-        if limit:
+        if limit and limit > 0:
             qparams['limit'] = limit
 
         if sort_keys:

mistralclient/tests/unit/v2/test_action_executions.py

@@ -108,6 +108,20 @@ class TestActionExecutions(base.BaseClientV2Test):
         # Make sure that limit is passed to the server correctly.
         self.assertEqual(['1'], last_request.qs['limit'])
 
+    def test_list_with_no_limit(self):
+        self.requests_mock.get(
+            self.TEST_URL + URL_TEMPLATE,
+            json={'action_executions': [ACTION_EXEC]}
+        )
+
+        action_execution_list = self.action_executions.list(limit=-1)
+
+        self.assertEqual(1, len(action_execution_list))
+
+        last_request = self.requests_mock.last_request
+
+        self.assertNotIn('limit', last_request.qs)
+
     def test_get(self):
         url = self.TEST_URL + URL_TEMPLATE_ID % ACTION_EXEC['id']
         self.requests_mock.get(url, json=ACTION_EXEC)

mistralclient/tests/unit/v2/test_actions.py

@@ -182,6 +182,18 @@ class TestActionsV2(base.BaseClientV2Test):
         self.assertEqual(['created_at'], last_request.qs['sort_keys'])
         self.assertEqual(['asc'], last_request.qs['sort_dirs'])
 
+    def test_list_with_no_limit(self):
+        self.requests_mock.get(self.TEST_URL + URL_TEMPLATE,
+                               json={'actions': [ACTION]})
+
+        action_list = self.actions.list(limit=-1)
+
+        self.assertEqual(1, len(action_list))
+
+        last_request = self.requests_mock.last_request
+
+        self.assertNotIn('limit', last_request.qs)
+
     def test_get(self):
         self.requests_mock.get(self.TEST_URL + URL_TEMPLATE_NAME % 'action',
                                json=ACTION)

mistralclient/tests/unit/v2/test_executions.py

@@ -195,6 +195,18 @@ class TestExecutionsV2(base.BaseClientV2Test):
         self.assertEqual(['created_at'], last_request.qs['sort_keys'])
         self.assertEqual(['asc'], last_request.qs['sort_dirs'])
 
+    def test_list_with_no_limit(self):
+        self.requests_mock.get(self.TEST_URL + URL_TEMPLATE,
+                               json={'executions': [EXEC]})
+
+        execution_list = self.executions.list(limit=-1)
+
+        self.assertEqual(1, len(execution_list))
+
+        last_request = self.requests_mock.last_request
+
+        self.assertNotIn('limit', last_request.qs)
+
     def test_get(self):
         url = self.TEST_URL + URL_TEMPLATE_ID % EXEC['id']
         self.requests_mock.get(url, json=EXEC)

mistralclient/tests/unit/v2/test_tasks.py

@@ -59,6 +59,18 @@ class TestTasksV2(base.BaseClientV2Test):
         self.tasks.list(fields=["id,name"])
         self.assertTrue(self.requests_mock.called_once)
 
+    def test_list_with_no_limit(self):
+        self.requests_mock.get(self.TEST_URL + URL_TEMPLATE,
+                               json={'tasks': [TASK]})
+
+        task_list = self.tasks.list(limit=-1)
+
+        self.assertEqual(1, len(task_list))
+
+        last_request = self.requests_mock.last_request
+
+        self.assertNotIn('limit', last_request.qs)
+
     def test_get(self):
         url = self.TEST_URL + URL_TEMPLATE_ID % TASK['id']
         self.requests_mock.get(url, json=TASK)

mistralclient/tests/unit/v2/test_workflows.py

@@ -168,6 +168,18 @@ class TestWorkflowsV2(base.BaseClientV2Test):
         self.assertEqual(['created_at'], last_request.qs['sort_keys'])
         self.assertEqual(['asc'], last_request.qs['sort_dirs'])
 
+    def test_list_with_no_limit(self):
+        self.requests_mock.get(self.TEST_URL + URL_TEMPLATE,
+                               json={'workflows': [WORKFLOW]})
+
+        workflows_list = self.workflows.list(limit=-1)
+
+        self.assertEqual(1, len(workflows_list))
+
+        last_request = self.requests_mock.last_request
+
+        self.assertNotIn('limit', last_request.qs)
+
     def test_get(self):
         url = self.TEST_URL + URL_TEMPLATE_NAME % 'wf'
         self.requests_mock.get(url, json=WORKFLOW)