Skip to content

Commit

Permalink
Fix tests for python 2.x
Browse files Browse the repository at this point in the history
  • Loading branch information
Evgeni Kunev committed Oct 7, 2015
1 parent ea1bf32 commit 09e4984
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions tests/test_config.py
Expand Up @@ -15,7 +15,8 @@ class TestInitializeMutornadomon(unittest.TestCase):

@mock.patch('mutornadomon.config.MuTornadoMon')
@mock.patch('mutornadomon.config.WebCollector')
def test_initialize_mutornadmon(self, web_collector_mock, mutornadomon_mock):
@mock.patch('mutornadomon.config.UtilizationCollector')
def test_initialize_mutornadmon(self, utilization_collector_mock, web_collector_mock, mutornadomon_mock):
"""Test initialize_mutornadomon() sets up HTTP endpoints interface"""
app = sentinel.application,
result = initialize_mutornadomon(app, host_limit='test')
Expand All @@ -26,6 +27,7 @@ def test_initialize_mutornadmon(self, web_collector_mock, mutornadomon_mock):

mutornadomon_mock.assert_called_once()
web_collector_mock.assert_called_once_with(monitor_inst, app)
utilization_collector_mock.assert_called_once_with(monitor_inst)

# MuTornadoMon was created with monitor config values
arg_list = mutornadomon_mock.call_args_list
Expand All @@ -39,7 +41,13 @@ def test_initialize_mutornadmon(self, web_collector_mock, mutornadomon_mock):

@mock.patch('mutornadomon.config.MuTornadoMon')
@mock.patch('mutornadomon.config.WebCollector')
def test_initialize_mutornadmon_passes_publisher(self, web_collector_mock, mutornadomon_mock):
@mock.patch('mutornadomon.config.UtilizationCollector')
def test_initialize_mutornadmon_passes_publisher(
self,
utilization_collector_mock,
web_collector_mock,
mutornadomon_mock
):
"""Test initialize_mutornadomon() sets up publishing interface"""

def publisher(monitor):
Expand All @@ -55,6 +63,7 @@ def publisher(monitor):
self.assertEqual(result, monitor_inst)

web_collector_mock.assert_called_once_with(monitor_inst, app)
utilization_collector_mock.assert_called_once_with(monitor_inst)

mutornadomon_mock.assert_called_once()
arg_list = mutornadomon_mock.call_args_list
Expand All @@ -72,8 +81,9 @@ def test_initialize_mutornadmon_works_with_publisher_and_no_app(self, mutornadom
def publisher(monitor):
pass

result = initialize_mutornadomon(publisher=publisher)
monitor_inst = mutornadomon_mock.return_value
monitor_inst.io_loop.add_callback.__name__ = 'add_callback'
result = initialize_mutornadomon(publisher=publisher)

# initialize_mutornadomon() should return the monitor instance
self.assertEqual(result, monitor_inst)
Expand Down

0 comments on commit 09e4984

Please sign in to comment.