Quiet tests

This disables the console logger on the  test backend so that tests
which pass do not spew their output. As nose captures logging already,
you'll only see the relevant logging for tests which fail, making it a
lot easier to find what is wrong.

Author: Nick Groenen

errbot/backends/test.py

@@ -122,7 +122,6 @@ def setUpClass(cls, extra_test_file=None):
         file = logging.FileHandler(config_module.BOT_LOG_FILE, encoding='utf-8')
         logger = logging.getLogger('')
         logger.setLevel(logging.DEBUG)
-        logger.addHandler(console)
         logger.addHandler(file)
         if extra_test_file:
             import config
@@ -144,4 +143,4 @@ def assertCommand(self, command, response, timeout=5):
 
     def assertCommandFound(self, command, timeout=5):
         pushMessage(command)
-        self.assertNotIn('not found', popMessage(), timeout)
+        self.assertNotIn('not found', popMessage(), timeout)

run_tests.py

@@ -17,7 +17,7 @@
 if not PY3:  # hack the path system to take the python 2 converted sources
     print('Changing root to ' + py2_root)
     print('Sys path ' + ', '.join(sys.path))
-    if nose.run('tests', argv=['-v', '-w', py2_root]):
+    if nose.run('tests', argv=['-w', py2_root]):
         exit(0)  # no error
 else:
     if nose.run('tests'):

tests/webhooks_tests.py

@@ -1,12 +1,16 @@
 from errbot.backends.test import FullStackTest, pushMessage, popMessage
 import requests
 from errbot import PY2
+from time import sleep
 
 
 class TestWebhooks(FullStackTest):
     @classmethod
     def setUpClass(cls, extra_test_file=None):
         super(TestWebhooks, cls).setUpClass()
+        # Give the rocket webserver just a fraction of extra time to initialize
+        # Was getting 50/50 chance of test failures (Connection refused) without this delay
+        sleep(0.1) 
         pushMessage("!config Webserver {'HOST': 'localhost', 'PORT': 3141, 'SSL':  None}")
         popMessage()