[change] Use PropertyMock for stdout.channel.exit_status on exec_command

Author: Alexandre Vincent

openwisp_controller/connection/tests/test_models.py

@@ -1,5 +1,6 @@
 import socket
 from unittest import mock
+from unittest.mock import PropertyMock
 
 import paramiko
 from django.contrib.auth.models import ContentType
@@ -46,7 +47,7 @@ def _exec_command_return_value(
         stderr_ = mock.Mock()
         stdin_.read().decode.return_value = stdin
         stdout_.read().decode.return_value = stdout
-        stdout_.channel.exit_status = exit_code
+        type(stdout_.channel).exit_status = PropertyMock(return_value=exit_code)
         stderr_.read().decode.return_value = stderr
         return (stdin_, stdout_, stderr_)
 
@@ -1009,7 +1010,7 @@ def _assert_applying_conf_test_command(mocked_exec):
                 # 1. Checking openwisp_config returns with 0
                 # 2. Testing presence of /tmp/openwisp/applying_conf returns with 1
                 # 3. Restarting openwisp_config returns with 0 exit code
-                stdout.channel.recv_exit_status.side_effect = [0, 1, 1]
+                type(stdout.channel).exit_status = PropertyMock(side_effect=[0, 1, 1])
                 mocked_exec_command.return_value = (stdin, stdout, stderr)
                 conf.save()
                 self.assertEqual(mocked_exec_command.call_count, 3)