Add todos for tests

Author: eedgar

st2actions/tests/unit/test_paramiko_ssh.py

@@ -803,10 +803,8 @@ def test_use_ssh_config_port_value_provided_in_the_config(self, mock_sshclient):
         call_kwargs = mock_client.connect.call_args[1]
         self.assertEqual(call_kwargs['port'], 9999)
 
-    @patch('paramiko.SSHClient', Mock)
     @patch.object(ParamikoSSHClient, '_is_key_file_needs_passphrase',
                   MagicMock(return_value=False))
-
     def test_socket_closed(self):
         conn_params = {'hostname': 'dummy.host.org',
                        'username': 'ubuntu',
@@ -823,14 +821,14 @@ def test_socket_closed(self):
         ssh_client.socket = Mock()
 
         # Make sure we havent called any close methods at this point
+        # TODO: Replace these with .assert_not_called() once it's Python 3.6+ only
         self.assertEqual(ssh_client.socket.process.kill.call_count, 0)
         self.assertEqual(ssh_client.socket.process.poll.call_count, 0)
 
         # Call the function that has changed
         ssh_client.close()
 
         # Make sure we have called kill and poll
+        # TODO: Replace these with .assert_called_once() once it's Python 3.6+ only
         self.assertEqual(ssh_client.socket.process.kill.call_count, 1)
         self.assertEqual(ssh_client.socket.process.poll.call_count, 1)
-
-