general: better cached calls

* Adds parameters and comments to `cached` wrapper.

Co-Authored-by: Peter Weber <[email protected]>

rero_ils/modules/cli/utils.py

@@ -273,7 +273,7 @@ def add_org_lib_doc(item, doc_pid="dummy"):
     return item
 
 
-@cached()
+@cached(timeout=2 * 60 * 60, key_refix="doc_item_lofi_schemas", query_string=True)  # 2 hour timeout
 def get_doc_item_lofi_schemas():
     """Get document, item, local field schemas."""
     # document schema

rero_ils/modules/documents/api_views.py

@@ -35,7 +35,7 @@
 
 
 @api_blueprint.route("/cover/<isbn>")
-@cached(timeout=300, query_string=True)
+@cached(timeout=5 * 60, query_string=True)  # 5 minutes timeout
 def cover(isbn):
     """Document cover service."""
     return jsonify(get_remote_cover(isbn))
@@ -51,7 +51,7 @@ def document_availability(pid):
 
 
 @api_blueprint.route("/advanced-search-config")
-@cached(timeout=300, query_string=True)
+@cached(timeout=5 * 60, query_string=True)  # 5 minutes timeout
 @check_logged_as_librarian
 def advanced_search_config():
     """Advanced search config."""

rero_ils/modules/views.py

@@ -51,7 +51,7 @@
 
 @api_blueprint.route("/permissions/<route_name>", methods=["GET"])
 @api_blueprint.route("/permissions/<route_name>/<record_pid>", methods=["GET"])
-@cached(timeout=10, query_string=True)
+@cached(timeout=10, query_string=True)  # 10 seconds timeout
 @check_authentication
 def permissions(route_name, record_pid=None):
     """HTTP GET request for record permissions.

rero_ils/theme/views.py

@@ -85,8 +85,8 @@ def error():
 
 
 @blueprint.route("/robots.txt")
-@cached()
-def robots(timeout=60 * 60):  # 1 hour timeout
+@cached(timeout=60 * 60, query_string=True)  # 1 hour timeout
+def robots():
     """Robots.txt generate response."""
     response = current_app.config["RERO_ILS_ROBOTS"]
     response = Response(response=response, status=200, mimetype="text/plain")

tests/unit/test_cli_utils.py

@@ -74,7 +74,6 @@ def test_cli_validate_documents_items_lofi(app, loc_public_martigny):
     file_name = join(dirname(__file__), "../data/documents_items_lofi.json")
 
     res = runner.invoke(validate_documents_with_items_lofis_cli, [file_name, "-v"])
-    print(">>>>", res)
     assert res.output.strip().split("\n")[1:] == [
         "1          document: dummy_1 errors: 1",
         "    documents: 'type' is a required property",