Skip to content

Commit a902078

Browse files
committed
Fix config test failures
1 parent 8394078 commit a902078

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/benji/tests/test_config.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ class ConfigTestCase(TestCaseBase, TestCase):
3030
configuration:
3131
cephConfigFile: /etc/ceph/ceph.conf
3232
simultaneousReads: 10
33+
simultaneousWrites: 10
3334
newImageFeatures:
3435
- RBD_FEATURE_LAYERING
3536
- RBD_FEATURE_EXCLUSIVE_LOCK
@@ -57,6 +58,7 @@ class ConfigTestCase(TestCaseBase, TestCase):
5758
configuration:
5859
cephConfigFile: /etc/ceph/ceph.conf
5960
simultaneousReads: 10
61+
simultaneousWrites: 10
6062
newImageFeatures:
6163
- RBD_FEATURE_LAYERING
6264
- RBD_FEATURE_EXCLUSIVE_LOCK
@@ -84,6 +86,7 @@ class ConfigTestCase(TestCaseBase, TestCase):
8486
configuration:
8587
cephConfigFile: /etc/ceph/ceph.conf
8688
simultaneousReads: 10
89+
simultaneousWrites: 10
8790
newImageFeatures:
8891
- RBD_FEATURE_LAYERING
8992
- RBD_FEATURE_EXCLUSIVE_LOCK
@@ -143,8 +146,9 @@ def test_validation(self):
143146
'bandwidthWrite': 0,
144147
'consistencyCheckWrites': False,
145148
'path': '/var/tmp',
146-
'simultaneousReads': 1,
147-
'simultaneousWrites': 1
149+
'simultaneousReads': 3,
150+
'simultaneousWrites': 3,
151+
'simultaneousRemovals': 5,
148152
}, config.validate(module='benji.storage.file', config=module_configuration))
149153
module_configuration = {'asdasdas': 'dasdasd'}
150154
self.assertRaises(ConfigurationError,
@@ -167,7 +171,8 @@ def test_validation_io_rbd(self):
167171
'cephConfigFile': '/etc/ceph/ceph.conf',
168172
'clientIdentifier': 'admin',
169173
'newImageFeatures': ['RBD_FEATURE_LAYERING', 'RBD_FEATURE_EXCLUSIVE_LOCK'],
170-
'simultaneousReads': 10
174+
'simultaneousReads': 10,
175+
'simultaneousWrites': 10,
171176
}, config.validate(module='benji.io.rbd', config=module_configuration))
172177
module_configuration['newImageFeatures'] = ['ASASA', 'DDASAD']
173178
self.assertRaises(ConfigurationError,

0 commit comments

Comments
 (0)