From 32ce22c744ca543b9b45465492a7f3e8fbd9fab1 Mon Sep 17 00:00:00 2001 From: TommyLike Date: Mon, 12 Jun 2017 11:19:46 +0800 Subject: [PATCH] Fix error in unit testcase 'ddt.data' decorator is wrongly used in 'test_volumes.py', correct it. Change-Id: Ib60c4cad33c071f52391fcfa227ed29482ad9445 --- cinderclient/tests/unit/v3/test_volumes.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cinderclient/tests/unit/v3/test_volumes.py b/cinderclient/tests/unit/v3/test_volumes.py index f6164a518..3a84bf34b 100644 --- a/cinderclient/tests/unit/v3/test_volumes.py +++ b/cinderclient/tests/unit/v3/test_volumes.py @@ -27,7 +27,7 @@ from six.moves.urllib import parse cs = fakes.FakeClient() -@ddt.data +@ddt.ddt class VolumesTest(utils.TestCase): def test_volume_manager_upload_to_image(self): @@ -107,7 +107,7 @@ class VolumesTest(utils.TestCase): @ddt.data(True, False) def test_get_pools_filter_by_name(self, detail): cs = fakes.FakeClient(api_version=api_versions.APIVersion('3.33')) - vol = cs.volumes.get_pools(detail, 'pool1') + vol = cs.volumes.get_pools(detail, {'name': 'pool1'}) request_url = '/scheduler-stats/get_pools?name=pool1' if detail: request_url = '/scheduler-stats/get_pools?detail=True&name=pool1'