From 4e4a209ec319d20f87d4b7cce6f1a6050b5c8a9e Mon Sep 17 00:00:00 2001 From: Thomas Adamcik Date: Sat, 1 Sep 2012 12:15:08 +0200 Subject: [PATCH] Fix existing settings tests that did_you_mean broke. --- mopidy/utils/settings.py | 3 +++ tests/utils/settings_test.py | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/mopidy/utils/settings.py b/mopidy/utils/settings.py index 4de7d1cf..34126907 100644 --- a/mopidy/utils/settings.py +++ b/mopidy/utils/settings.py @@ -199,6 +199,9 @@ def mask_value_if_secret(key, value): def did_you_mean(setting, defaults): """Suggest most likely setting based on levenshtein.""" + if not defaults: + return None + candidates = [(levenshtein(setting, d), d) for d in defaults] candidates.sort() diff --git a/tests/utils/settings_test.py b/tests/utils/settings_test.py index 55e1156b..c129b9b5 100644 --- a/tests/utils/settings_test.py +++ b/tests/utils/settings_test.py @@ -23,7 +23,7 @@ class ValidateSettingsTest(unittest.TestCase): result = validate_settings(self.defaults, {'MPD_SERVER_HOSTNMAE': '127.0.0.1'}) self.assertEqual(result['MPD_SERVER_HOSTNMAE'], - u'Unknown setting. Is it misspelled?') + u'Unknown setting. Did you mean MPD_SERVER_HOSTNAME?') def test_not_renamed_setting_returns_error(self): result = validate_settings(self.defaults,