Skip to content

Commit d887fbb

Browse files
committed
mock client.request to fix tests
1 parent 224523c commit d887fbb

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

tests/unit_tests/test_platform_deployer.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,8 @@ def test_validate_platform_api_connection_fails(monkeypatch, mocker):
9696
monkeypatch.setenv("API_TOKEN", "test_token")
9797

9898
deployer = PlatformDeployer()
99-
mock_webapp_exists = mocker.patch.object(deployer.client, "webapp_exists")
100-
mock_webapp_exists.side_effect = Exception("Connection failed")
99+
mock_request = mocker.patch.object(deployer.client, "request")
100+
mock_request.side_effect = Exception("Connection failed")
101101

102102
with pytest.raises(DSDCommandError, match="Failed to connect to PythonAnywhere API"):
103103
deployer._validate_platform()
@@ -109,8 +109,10 @@ def test_validate_platform_success(monkeypatch, mocker):
109109
monkeypatch.setenv("API_TOKEN", "test_token")
110110

111111
deployer = PlatformDeployer()
112-
mock_webapp_exists = mocker.patch.object(deployer.client, "webapp_exists")
113-
mock_webapp_exists.return_value = True
112+
mock_request = mocker.patch.object(deployer.client, "request")
113+
mock_response = mocker.Mock()
114+
mock_response.ok = True
115+
mock_request.return_value = mock_response
114116

115117
# Should not raise any exception
116118
deployer._validate_platform()

0 commit comments

Comments
 (0)