requests: Expand tests and fix MicroPython compatibility.

Order-independent mock asserts, live TCP tests, unix/ESP32 runners,
prefer /lib over frozen requests on ESP32, and HEAD/auth/encoding fixes.

Signed-off-by: Pablo Ventura <pablogventura@gmail.com>

Author: pablogventura

python-ecosys/requests/requests/__init__.py

@@ -63,7 +63,7 @@ def request(
         import binascii
 
         username, password = auth
-        formatted = (username + ":" + password).encode()
+        formatted = _to_bytes(username) + b":" + _to_bytes(password)
         formatted = str(binascii.b2a_base64(formatted)[:-1], "ascii")
         headers["Authorization"] = "Basic {}".format(formatted)
 
@@ -193,7 +193,7 @@ def request(
             max_body,
         )
 
-    if method == "HEAD":
+    if method == b"HEAD":
         body = b""
     else:
         body = read_body(s, resp_headers, max_body)

python-ecosys/requests/requests/_http.py

@@ -108,9 +108,11 @@ def read_body(stream, headers, max_body=None):
     """Read the response body according to Content-Length or chunked encoding."""
     max_remaining = max_body
     encoding = headers.get("transfer-encoding", "")
-    if encoding and "chunked" in encoding.lower():
-        body, _ = _read_chunked(stream, max_remaining)
-        return body
+    if encoding:
+        enc = encoding if isinstance(encoding, str) else str(encoding, "utf-8")
+        if "chunked" in enc.lower():
+            body, _ = _read_chunked(stream, max_remaining)
+            return body
     content_length = headers.get("content-length")
     if content_length is not None:
         length = int(content_length)

python-ecosys/requests/run-esp32-tests.sh

@@ -0,0 +1,26 @@
+#!/usr/bin/env bash
+# Deploy requests package to ESP32 and run mock + live tests via mpremote.
+set -euo pipefail
+
+PORT="${PORT:-/dev/ttyACM0}"
+MPREMOTE="${MPREMOTE:-pipx run mpremote}"
+ROOT="$(cd "$(dirname "$0")" && pwd)"
+
+run_mp() {
+  $MPREMOTE connect "$PORT" "$@"
+}
+
+cd "$ROOT"
+echo "== Deploy requests to :lib/requests =="
+run_mp fs mkdir :lib 2>/dev/null || true
+run_mp fs mkdir :lib/requests 2>/dev/null || true
+run_mp fs cp requests/__init__.py :lib/requests/__init__.py
+run_mp fs cp requests/_http.py :lib/requests/_http.py
+
+echo "== Mock tests =="
+run_mp run test_requests.py
+
+echo "== Live tests =="
+run_mp run test_requests_live.py
+
+echo "== ESP32 tests passed =="

python-ecosys/requests/run-unix-tests.sh

@@ -0,0 +1,21 @@
+#!/usr/bin/env bash
+# Run requests package tests on MicroPython unix port (mock + live).
+set -euo pipefail
+
+MICROPYTHON="${MICROPYTHON:-../../../micropython/ports/unix/build-standard/micropython}"
+ROOT="$(cd "$(dirname "$0")" && pwd)"
+
+if [[ ! -x "$MICROPYTHON" ]]; then
+  echo "Build unix port first:" >&2
+  echo "  cd micropython/ports/unix && make submodules && make" >&2
+  exit 1
+fi
+
+cd "$ROOT"
+echo "== Mock tests (test_requests.py) =="
+"$MICROPYTHON" test_requests.py
+
+echo "== Live tests (test_requests_live.py) =="
+"$MICROPYTHON" test_requests_live.py
+
+echo "== All unix tests passed =="