Commit fac44a85339ac104b21afe11ac5883339d0bb9cf
Committed by
Sergio Oliveira
1 parent
5971df7b
Exists in
master
and in
30 other branches
Fix colab.signals tests
Signed-off-by: Lucas Moura <lucas.moura128@gmail.com>
Showing
1 changed file
with
3 additions
and
3 deletions
Show diff stats
colab/signals/tests/test_signals.py
| @@ -49,16 +49,16 @@ class SignalsTest(TestCase): | @@ -49,16 +49,16 @@ class SignalsTest(TestCase): | ||
| 49 | @patch('colab.signals.signals.Signal.connect') | 49 | @patch('colab.signals.signals.Signal.connect') |
| 50 | def test_connect_already_registered_signal(self, mock): | 50 | def test_connect_already_registered_signal(self, mock): |
| 51 | sender = 'Test' | 51 | sender = 'Test' |
| 52 | - handling_method = MagicMock | 52 | + handling_method = MagicMock() |
| 53 | type(handling_method).delay = PropertyMock(return_value='Test') | 53 | type(handling_method).delay = PropertyMock(return_value='Test') |
| 54 | signal_name = 'a' | 54 | signal_name = 'a' |
| 55 | 55 | ||
| 56 | register_signal(self.plugin_name, self.list_signal) | 56 | register_signal(self.plugin_name, self.list_signal) |
| 57 | 57 | ||
| 58 | - connect_signal(signal_name, sender, handling_method.delay) | 58 | + connect_signal(signal_name, sender, handling_method) |
| 59 | args, kwargs = mock.call_args | 59 | args, kwargs = mock.call_args |
| 60 | 60 | ||
| 61 | - self.assertEqual(args[0], handling_method) | 61 | + self.assertEqual(args[0], handling_method.delay) |
| 62 | self.assertEqual(kwargs['sender'], sender) | 62 | self.assertEqual(kwargs['sender'], sender) |
| 63 | self.assertTrue(mock.is_called) | 63 | self.assertTrue(mock.is_called) |
| 64 | 64 |