diff options
| author | SaiKiran <saikiranveeravarapu@gmail.com> | 2016-01-09 20:51:10 +0530 |
|---|---|---|
| committer | SaiKiran <saikiranveeravarapu@gmail.com> | 2016-01-23 11:44:15 +0530 |
| commit | 69b8cde5f12c1d11036d99ef69421f103ebed87d (patch) | |
| tree | 563097a1de03a9b3145eecc1ec3bbe4e37b9fa71 /openstackclient/tests/object/v1/test_object_all.py | |
| parent | 75caafa0965617023f8dfba3be5a53e6711eecce (diff) | |
| download | python-openstackclient-69b8cde5f12c1d11036d99ef69421f103ebed87d.tar.gz | |
Refactor abstract columns and datalist out in image and object
test cases
columns and datalist has been set in each test case in image
and object which is not necessary. This patch abstract it out
and remove all redundant code.
Change-Id: Ie6aa3fa27ab2a468c67da31209107517259631c2
Related-Bug: 1532384
Diffstat (limited to 'openstackclient/tests/object/v1/test_object_all.py')
| -rw-r--r-- | openstackclient/tests/object/v1/test_object_all.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/openstackclient/tests/object/v1/test_object_all.py b/openstackclient/tests/object/v1/test_object_all.py index 7a76ab76..a90e5b65 100644 --- a/openstackclient/tests/object/v1/test_object_all.py +++ b/openstackclient/tests/object/v1/test_object_all.py @@ -37,6 +37,8 @@ class TestObjectCreate(TestObjectAll): class TestObjectList(TestObjectAll): + columns = ('Name',) + def setUp(self): super(TestObjectList, self).setUp() @@ -69,8 +71,7 @@ class TestObjectList(TestObjectAll): # Lister.take_action() returns two tuples columns, data = self.cmd.take_action(parsed_args) - collist = ('Name',) - self.assertEqual(collist, columns) + self.assertEqual(self.columns, columns) datalist = [ (object_fakes.object_name_1, ), (object_fakes.object_name_2, ), @@ -104,8 +105,7 @@ class TestObjectList(TestObjectAll): # DisplayCommandBase.take_action() returns two tuples columns, data = self.cmd.take_action(parsed_args) - collist = ('Name',) - self.assertEqual(collist, columns) + self.assertEqual(self.columns, columns) datalist = ( (object_fakes.object_name_2, ), ) |
