Fix backwards-compatibility with deprecated remove_expired_responses

2 files changed, 5 insertions, 5 deletions

diff --git a/requests_cache/backends/base.py b/requests_cache/backends/base.py
index 3afe4e7..67f7832 100644
--- a/requests_cache/backends/base.py
+++ b/requests_cache/backends/base.py
@@ -185,7 +185,7 @@ class BaseCache:
         invalid: bool = False,
         older_than: ExpirationTime = None,
     ) -> Iterator[CachedResponse]:
-        """Get responses from the cache, with optional filters
+        """Get responses from the cache, with optional filters for which responses to include:
 
         Args:
             valid: Include valid and unexpired responses; set to ``False`` to get **only**
@@ -298,9 +298,9 @@ class BaseCache:
             'please use .delete(expired=True) instead',
             DeprecationWarning,
         )
-        self.delete(expired=True, invalid=True)
         if expire_after:
             self.reset_expiration(expire_after)
+        self.delete(expired=True, invalid=True)
 
     def values(self, check_expiry: bool = False) -> Iterator[CachedResponse]:
         warn('BaseCache.values() is deprecated; please use .filter() instead', DeprecationWarning)
diff --git a/requests_cache/session.py b/requests_cache/session.py
index 3ed7a6d..d94adf8 100644
--- a/requests_cache/session.py
+++ b/requests_cache/session.py
@@ -307,9 +307,9 @@ class CacheMixin(MIXIN_BASE):
         super().close()
         self.cache.close()
 
-    def remove_expired_responses(self):
-        """**Deprecated:** Use ``session.cache.delete(expired=True)`` instead"""
-        self.cache.delete(expired=True, invalid=True)
+    def remove_expired_responses(self, expire_after: ExpirationTime = None):
+        # Deprecated; will be replaced by CachedSession.cache.delete(expired=True)
+        self.cache.remove_expired_responses(expire_after)
 
     def __getstate__(self):
         # Unlike requests.Session, CachedSession may contain backend connection objects that can't