test: cover shared timeout defaults

Co-authored-by: erseco <1876752+erseco@users.noreply.github.com>

Author: Copilot

src/py_moodle/config.py

@@ -1,4 +1,12 @@
-"""Shared timeout defaults for HTTP operations."""
+"""Shared timeout defaults for HTTP operations.
+
+``DEFAULT_REQUEST_TIMEOUT`` is the standard timeout for routine Moodle GET/POST
+requests. ``DEFAULT_SCRAPE_TIMEOUT`` is shorter for quick HTML fetches used as
+fallback probes. Upload helpers use ``(connect_timeout, read_timeout)`` tuples:
+``DEFAULT_UPLOAD_TIMEOUT`` for typical uploads and
+``DEFAULT_LARGE_UPLOAD_TIMEOUT`` for larger package transfers such as SCORM and
+draft-file uploads.
+"""
 
 DEFAULT_REQUEST_TIMEOUT = 30
 DEFAULT_SCRAPE_TIMEOUT = 15

tests/unit/test_timeouts.py

@@ -15,9 +15,9 @@
 class FakeResponse:
     """Minimal response object for timeout tests."""
 
-    def __init__(self, text="", json_data=None, status_code=200):
+    def __init__(self, text="", json_data_value=None, status_code=200):
         self.text = text
-        self._json_data = json_data
+        self._json_data = json_data_value
         self.status_code = status_code
 
     def raise_for_status(self):
@@ -35,7 +35,7 @@ class RecordingSession:
 
     def __init__(self, get_response=None, post_response=None):
         self.get_response = get_response or FakeResponse()
-        self.post_response = post_response or FakeResponse(json_data={})
+        self.post_response = post_response or FakeResponse(json_data_value={})
         self.get_calls = []
         self.post_calls = []
 
@@ -61,7 +61,7 @@ def test_session_call_uses_shared_default_request_timeout():
         cas_url=None,
         webservice_token=None,
     )
-    session = RecordingSession(post_response=FakeResponse(json_data={"ok": True}))
+    session = RecordingSession(post_response=FakeResponse(json_data_value={"ok": True}))
     moodle_session = MoodleSession(settings)
     moodle_session._session = session
     moodle_session._token = "token"
@@ -78,7 +78,7 @@ def test_detect_moodle_compatibility_uses_shared_default_request_timeout():
         get_response=FakeResponse(
             text='<meta name="generator" content="Moodle 4.5.2+ (Build: 20241001)">'
         ),
-        post_response=FakeResponse(json_data={"exception": "disabled"}),
+        post_response=FakeResponse(json_data_value={"exception": "disabled"}),
     )
 
     compatibility = detect_moodle_compatibility(
@@ -97,11 +97,11 @@ def test_upload_helpers_use_shared_default_timeouts(tmp_path, monkeypatch):
 
     upload_calls = []
 
-    def fake_requests_post(url, **kwargs):
+    def mock_requests_post(url, **kwargs):
         upload_calls.append({"url": url, "kwargs": kwargs})
-        return FakeResponse(json_data=[{"itemid": 7}])
+        return FakeResponse(json_data_value=[{"itemid": 7}])
 
-    monkeypatch.setattr("py_moodle.upload.requests.post", fake_requests_post)
+    monkeypatch.setattr("py_moodle.upload.requests.post", mock_requests_post)
 
     itemid = upload_file_webservice(
         "https://moodle.example.test", "token", str(file_path)
@@ -112,7 +112,7 @@ def fake_requests_post(url, **kwargs):
 
     monkeypatch.setattr("py_moodle.draftfile.detect_upload_repo", lambda *args: 9)
     draft_session = RecordingSession(
-        post_response=FakeResponse(json_data={"id": 99, "filename": "demo.txt"})
+        post_response=FakeResponse(json_data_value={"id": 99, "filename": "demo.txt"})
     )
 
     draft_itemid, filename = upload_file_to_draft_area(