aws-python-helper 1.15.0__tar.gz → 1.15.2__tar.gz

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aws-python-helper
-Version: 1.15.0
+Version: 1.15.2
 Summary: AWS Python Helper Framework
 Author-email: Fabian Claros <neufabiae@gmail.com>
 License-Expression: MIT

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/aws_python_helper/model_query/base.py

@@ -92,6 +92,11 @@ class _ModelQuerySchema(BaseModel):
     # y `extra="ignore"` la descartaba en silencio: la consulta terminaba en la
     # base del estado de la sesión y devolvía vacío sin error.
     database: Optional[str] = None
+    # Quién pregunta. Lo usa el servicio que expone esta lambda para recortar lo
+    # que devuelve a lo que le corresponde a ese servicio. Declarado por la
+    # misma razón que `database`: sin esto `extra="ignore"` lo descarta y el
+    # recorte se hace contra un servicio vacío, que es "nada".
+    service: Optional[str] = None
     filter: Optional[dict] = None
     pipeline: Optional[list] = None
     fields: Optional[dict] = None

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/aws_python_helper/settings.py

@@ -31,7 +31,11 @@ from typing import Any, Dict, List, Optional
 
 logger = logging.getLogger(__name__)
 
-PATH = "/services/settings"
+# Un solo segmento, y no `/services/settings`, porque para GET el fetcher
+# decide por posición: con dos segmentos el segundo es siempre un id, así que
+# buscaba `api/services/get.py` y devolvía 404. Un sub-recurso GET no existe en
+# esta convención.
+PATH = "/service-settings"
 TIMEOUT_SECONDS = 5
 # Si core no contestó, no se vuelve a preguntar en cada request.
 RETRY_FLOOR_SECONDS = 60

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/aws_python_helper.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aws-python-helper
-Version: 1.15.0
+Version: 1.15.2
 Summary: AWS Python Helper Framework
 Author-email: Fabian Claros <neufabiae@gmail.com>
 License-Expression: MIT

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/aws_python_helper.egg-info/SOURCES.txt

@@ -60,5 +60,6 @@ aws_python_helper/utils/response.py
 aws_python_helper/utils/serializer.py
 tests/test_audit_stamp.py
 tests/test_jwt_keys.py
+tests/test_model_query_schema.py
 tests/test_settings.py
 tests/test_sns_publisher.py

{aws_python_helper-1.15.0 → aws_python_helper-1.15.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "aws-python-helper"
-version = "1.15.0"
+version = "1.15.2"
 description = "AWS Python Helper Framework"
 readme = "README.md"
 requires-python = ">=3.9"

aws_python_helper-1.15.2/tests/test_model_query_schema.py

@@ -0,0 +1,57 @@
+"""Lo que el payload manda llega al código, o se pierde en silencio.
+
+El schema usa `extra="ignore"`, así que un parámetro que no esté declarado
+**desaparece sin error**. Ya pasó dos veces: `database` hacía que la consulta
+fuera a la base equivocada y devolviera vacío, y `service` hacía que el recorte
+por servicio se midiera contra uno vacío y no saliera nada.
+
+Las dos veces el síntoma fue el mismo: todo "funciona", la respuesta es 200, y
+lo que falta es justo lo que se pidió.
+
+Correr con un intérprete que tenga las dependencias del paquete:
+    ../constitution-core/venv/bin/python tests/test_model_query_schema.py
+"""
+
+import os
+import sys
+
+sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
+
+from aws_python_helper.model_query.base import _ModelQuerySchema
+
+failures = []
+
+
+def check(label, condition):
+    print(f"  {'ok  ' if condition else 'FALLA'} {label}")
+    if not condition:
+        failures.append(label)
+
+
+print("== cada parámetro documentado sobrevive al schema ==")
+parsed = _ModelQuerySchema(
+    collection="regions",
+    database="core",
+    service="dockets",
+    pipeline=[{"$match": {"slug": "x"}}],
+    skip=10,
+)
+for campo, esperado in (
+    ("collection", "regions"),
+    ("database", "core"),
+    ("service", "dockets"),
+    ("skip", 10),
+):
+    check(f"{campo} = {esperado!r}", getattr(parsed, campo) == esperado)
+
+print("\n== y lo que no está declarado sí se descarta ==")
+# Es el comportamiento buscado: lo que sobra se ignora. El problema nunca fue
+# que ignore, sino declarar de menos.
+otro = _ModelQuerySchema(collection="regions", inventado="x")
+check("un parámetro inventado no explota", not hasattr(otro, "inventado"))
+
+print()
+if failures:
+    print(f"{len(failures)} fallas")
+    sys.exit(1)
+print("all good")