Merge pull request #261 from benfrankel/fix-dry-run

Avoid logging or returning from cache when `dry_run=True`

Author: khaeru

doc/whatsnew.rst

@@ -16,6 +16,7 @@ Next release
 - Fix two regressions in :func:`.to_pandas` introduced in v2.23.0 (:issue:`251`, :pull:`252`).
 - Fix false cache hits and misses (thanks :gh-user:`benfrankel` for :issue:`256`, :pull:`257`).
 - Fix a bug where supplying `references=...` to ESTAT or EMPL would raise :class:`ValueError` (thanks :gh-user:`benfrankel` for :issue:`259`, :pull:`260`).
+- Adjust :meth:`.Client.get` to avoid logging or returning a message from cache when `dry_run=True` (thanks :gh-user:`benfrankel` for :pull:`261`).
 
 v2.23.1 (2025-10-01)
 ====================

sdmx/client.py

@@ -456,6 +456,8 @@ def get(
             req = self._request_from_args(kwargs)
 
         req_prepared = self.session.prepare_request(req)
+        if dry_run:
+            return req_prepared  # type: ignore [return-value]
 
         # Now get the SDMX message via HTTP
         log.info(f"Request {req_prepared.url}")
@@ -469,9 +471,6 @@ def get(
                 log.info("Not found in cache")
                 pass
 
-        if dry_run:
-            return req_prepared  # type: ignore [return-value]
-
         try:
             # Send the request
             response = self.session.send(req_prepared, **self._send_kwargs)