Makes auto ssl check for errors.

Author: filiplajszczak

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pythonanywhere-core"
-version = "0.3.0"
+version = "0.3.1"
 description = "API wrapper for programmatic management of PythonAnywhere services."
 authors = ["PythonAnywhere <developers@pythonanywhere.com>"]
 license = "MIT"

pythonanywhere_core/__init__.py

@@ -1 +1 @@
-__version__ = "0.3.0"
+__version__ = "0.3.1"

pythonanywhere_core/website.py

@@ -90,6 +90,10 @@ def auto_ssl(self, domain_name: str) -> dict:
             "post",
             json={"cert_type": "letsencrypt-auto-renew"}
         )
+        if not response.ok:
+            raise PythonAnywhereApiException(
+                f"POST to set Let's Encrypt SSL certificate via API failed, got {response}:{response.text}"
+            )
         return response.json()
 
     def get_ssl_info(self, domain_name) -> dict:

tests/test_website.py

@@ -170,6 +170,23 @@ def test_sets_lets_encrypt_cert(api_responses, domain_name, domains_base_url):
     }
 
 
+def test_raises_if_setting_lets_encrypt_cert_does_not_return_200(
+    api_responses, domain_name, domains_base_url
+):
+    api_responses.add(
+        responses.POST,
+        url=f"{domains_base_url}{domain_name}/ssl/",
+        status=500,
+        body='{"status": "error", "error_message": "Certificate issuance failed"}',
+    )
+
+    with pytest.raises(PythonAnywhereApiException) as e:
+        Website().auto_ssl(domain_name=domain_name)
+
+    assert "POST to set Let's Encrypt SSL certificate via API failed, got" in str(e.value)
+    assert "Certificate issuance failed" in str(e.value)
+
+
 def test_returns_ssl_info(api_responses, domain_name, domains_base_url):
     api_responses.add(
         responses.GET,