Skip to content

Commit

Permalink
Added check for ints for caches and added a test.
Browse files Browse the repository at this point in the history
  • Loading branch information
ewongbb committed May 31, 2012
1 parent 5b22df9 commit 5166fd6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion master/buildbot/config.py
Expand Up @@ -365,7 +365,7 @@ def load_caches(self, filename, config_dict, errors):
for (x, y) in valPairs:
if (not isinstance(y, int)):
errors.addError(
"value for cache size '%s' must be an integer" % x)
"value for cache size '%s' must be an integer" % x)
self.caches.update(caches)

if 'buildCacheSize' in config_dict:
Expand Down
2 changes: 1 addition & 1 deletion master/buildbot/test/unit/test_config.py
Expand Up @@ -551,7 +551,7 @@ def test_load_caches_entries_test(self):
dict(caches=dict(foo="1")),
self.errors)
self.assertConfigError(self.errors,
"value for cache size 'foo' must be an integer")
"value for cache size 'foo' must be an integer")

def test_load_schedulers_defaults(self):
self.cfg.load_schedulers(self.filename, {}, self.errors)
Expand Down

0 comments on commit 5166fd6

Please sign in to comment.