Make sure we reply if all of our fallbacks fail.

plugins/Weather.py

@@ -134,6 +134,8 @@ class Weather(callbacks.Privmsg):
                         break
                     except NoLocation:
                         self.log.info('%s lookup failed as backup.', command)
+            irc.error('Could not retrieve weather for %s.' %
+                      utils.quoted(location))
     weather = wrap(weather, [additional('text')])
 
 

test/test_Weather.py

@@ -71,6 +71,9 @@ if network:
         def testNoEscapingWebError(self):
             self.assertNotRegexp('ham "buenos aires"', 'WebError')
 
+        def testWeatherRepliesWithBogusLocation(self):
+            self.assertRegexp('weather some place that doesn\'t exist', r'.')
+
         def testConvertConfig(self):
             try:
                 convert = conf.supybot.plugins.Weather.convert()