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,