2 files changed, 127 insertions, 1 deletions
diff --git a/openstackclient/tests/unit/image/v2/fakes.py b/openstackclient/tests/unit/image/v2/fakes.py
index 2decd122..f2015450 100644
--- a/openstackclient/tests/unit/image/v2/fakes.py
+++ b/openstackclient/tests/unit/image/v2/fakes.py
@@ -52,6 +52,9 @@ class FakeImagev2Client:
         self.management_url = kwargs['endpoint']
         self.version = 2.0
 
+        self.tasks = mock.Mock()
+        self.tasks.resource_class = fakes.FakeResource(None, {})
+
 
 class TestImagev2(utils.TestCommand):
 
@@ -176,10 +179,26 @@ def create_one_task(attrs=None):
         # https://github.com/openstack/glance/blob/24.0.0/glance/api/v2/tasks.py#L186-L190
         'type': 'import',
         'updated_at': '2016-06-29T16:13:07Z',
-
     }
 
     # Overwrite default attributes if there are some attributes set
     task_info.update(attrs)
 
     return task.Task(**task_info)
+
+
+def create_tasks(attrs=None, count=2):
+    """Create multiple fake tasks.
+
+    :param attrs: A dictionary with all attributes of Task
+    :type attrs: dict
+    :param count: The number of tasks to be faked
+    :type count: int
+    :return: A list of fake Task objects
+    :rtype: list
+    """
+    tasks = []
+    for n in range(0, count):
+        tasks.append(create_one_task(attrs))
+
+    return tasks
diff --git a/openstackclient/tests/unit/image/v2/test_task.py b/openstackclient/tests/unit/image/v2/test_task.py
index 90fa11d8..e077e2b1 100644
--- a/openstackclient/tests/unit/image/v2/test_task.py
+++ b/openstackclient/tests/unit/image/v2/test_task.py
@@ -78,3 +78,110 @@ class TestTaskShow(TestTask):
 
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(self.data, data)
+
+
+class TestTaskList(TestTask):
+
+    tasks = image_fakes.create_tasks()
+
+    columns = (
+        'ID',
+        'Type',
+        'Status',
+        'Owner',
+    )
+    datalist = [
+        (
+            task.id,
+            task.type,
+            task.status,
+            task.owner_id,
+        )
+        for task in tasks
+    ]
+
+    def setUp(self):
+        super().setUp()
+
+        self.client.tasks.side_effect = [self.tasks, []]
+
+        # Get the command object to test
+        self.cmd = task.ListTask(self.app, None)
+
+    def test_task_list_no_options(self):
+        arglist = []
+        verifylist = [
+            ('sort_key', None),
+            ('sort_dir', None),
+            ('limit', None),
+            ('marker', None),
+            ('type', None),
+            ('status', None),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with()
+
+        self.assertEqual(self.columns, columns)
+        self.assertCountEqual(self.datalist, data)
+
+    def test_task_list_sort_key_option(self):
+        arglist = ['--sort-key', 'created_at']
+        verifylist = [('sort_key', 'created_at')]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with(
+            sort_key=parsed_args.sort_key,
+        )
+
+        self.assertEqual(self.columns, columns)
+        self.assertCountEqual(self.datalist, data)
+
+    def test_task_list_sort_dir_option(self):
+        arglist = ['--sort-dir', 'desc']
+        verifylist = [('sort_dir', 'desc')]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with(
+            sort_dir=parsed_args.sort_dir,
+        )
+
+    def test_task_list_pagination_options(self):
+        arglist = ['--limit', '1', '--marker', self.tasks[0].id]
+        verifylist = [('limit', 1), ('marker', self.tasks[0].id)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with(
+            limit=parsed_args.limit,
+            marker=parsed_args.marker,
+        )
+
+    def test_task_list_type_option(self):
+        arglist = ['--type', self.tasks[0].type]
+        verifylist = [('type', self.tasks[0].type)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with(
+            type=self.tasks[0].type,
+        )
+
+    def test_task_list_status_option(self):
+        arglist = ['--status', self.tasks[0].status]
+        verifylist = [('status', self.tasks[0].status)]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        self.cmd.take_action(parsed_args)
+
+        self.client.tasks.assert_called_with(
+            status=self.tasks[0].status,
+        )