openstack/cinder
Code Issues Proposed changes

Merge "Override osapi_volume_listen_port for test_workers"

Browse Source
This commit is contained in:
Jenkins
2015-12-18 21:03:52 +00:00
committed by Gerrit Code Review
parent 9780e050fe 99bf66b5ae
commit 78bbd10f69
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
cinder/tests/unit/test_service.py
View File

@@ -278,17 +278,20 @@ class TestWSGIService(test.TestCase):
@mock.patch('oslo_service.wsgi.Server') @mock.patch('oslo_service.wsgi.Server')
def test_workers_set_default(self, wsgi_server): def test_workers_set_default(self, wsgi_server):
self.override_config('osapi_volume_listen_port', 0)
test_service = service.WSGIService("osapi_volume") test_service = service.WSGIService("osapi_volume")
self.assertEqual(processutils.get_worker_count(), test_service.workers) self.assertEqual(processutils.get_worker_count(), test_service.workers)
@mock.patch('oslo_service.wsgi.Server') @mock.patch('oslo_service.wsgi.Server')
def test_workers_set_good_user_setting(self, wsgi_server): def test_workers_set_good_user_setting(self, wsgi_server):
self.override_config('osapi_volume_listen_port', 0)
self.override_config('osapi_volume_workers', 8) self.override_config('osapi_volume_workers', 8)
test_service = service.WSGIService("osapi_volume") test_service = service.WSGIService("osapi_volume")
self.assertEqual(8, test_service.workers) self.assertEqual(8, test_service.workers)
@mock.patch('oslo_service.wsgi.Server') @mock.patch('oslo_service.wsgi.Server')
def test_workers_set_zero_user_setting(self, wsgi_server): def test_workers_set_zero_user_setting(self, wsgi_server):
self.override_config('osapi_volume_listen_port', 0)
self.override_config('osapi_volume_workers', 0) self.override_config('osapi_volume_workers', 0)
test_service = service.WSGIService("osapi_volume") test_service = service.WSGIService("osapi_volume")
# If a value less than 1 is used, defaults to number of procs available # If a value less than 1 is used, defaults to number of procs available