anncsu-sdk 0.0.9.dev16__tar.gz → 0.0.9.dev17__tar.gz

{anncsu_sdk-0.0.9.dev16 → anncsu_sdk-0.0.9.dev17}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: anncsu-sdk
-Version: 0.0.9.dev16
+Version: 0.0.9.dev17
 Summary: ANNCSU Software Development Kit for API consumption
 Author-email: Francesco Bartoli <francesco.bartoli@geobeyond.it>
 License-File: LICENSE

{anncsu_sdk-0.0.9.dev16 → anncsu_sdk-0.0.9.dev17}/src/anncsu/accessi/errors/accesso_validation.py

@@ -133,6 +133,27 @@ class FieldNotAllowedForOperationError(AccessoValidationError):
         )
 
 
+@dataclass
+class SezioneCensimentoRequiredError(AccessoValidationError):
+    """Raised when sezione_censimento is missing for I (insert) or R (replace).
+
+    Per OAS spec: ``sezione_censimento`` has no ``nullable: true`` and is
+    described as "da valorizzare solo per operazione_civico='I','R'".
+    Missing it triggers a server-side rejection like
+    ``Sezione di censimento:   non presente nel Comune <X>``; catching it
+    locally gives a clearer message before the API call.
+    """
+
+    operazione: str
+
+    def __str__(self) -> str:
+        return (
+            f"Il campo 'sezione_censimento' e' obbligatorio per "
+            f"operazione_civico='{self.operazione}'. "
+            f"Per 'S' (soppressione) non deve essere valorizzato."
+        )
+
+
 @dataclass
 class AccessoMaxLengthError(AccessoValidationError):
     """Raised when a string field exceeds its OAS maxLength constraint.
@@ -163,5 +184,6 @@ __all__ = [
     "NumeroMetricoMutexError",
     "FieldNotAllowedForDeleteError",
     "FieldNotAllowedForOperationError",
+    "SezioneCensimentoRequiredError",
     "AccessoMaxLengthError",
 ]

{anncsu_sdk-0.0.9.dev16 → anncsu_sdk-0.0.9.dev17}/src/anncsu/accessi/models/validated.py

@@ -22,6 +22,8 @@ Rules (derived from the OAS spec):
 6. ``coordinate``, when present, is re-validated through the existing
    ``ValidatedCoordinate`` model (Italy bounds, X/Y dependency, metodo
    1-4, maxLength X/Y/Z).
+7. ``sezione_censimento`` is mandatory for ``I`` and ``R`` (OAS field has
+   no ``nullable: true``); rule 4 already forbids it for ``S``.
 """
 
 from __future__ import annotations
@@ -36,6 +38,7 @@ from anncsu.accessi.errors.accesso_validation import (
     NumeroMetricoMutexError,
     OperazioneCivicoError,
     ProgrCivicoRequiredError,
+    SezioneCensimentoRequiredError,
 )
 from anncsu.accessi.models.richiestaoperazione import Accesso
 from anncsu.coordinate.models.validated import ValidatedCoordinate
@@ -120,6 +123,12 @@ class ValidatedAccesso(Accesso):
             if not has_numero and not has_metrico:
                 raise NumeroMetricoMutexError(has_numero=False, has_metrico=False)
 
+        # Rule 7 (issue #30): sezione_censimento required for I/R.
+        # OAS declares the field without ``nullable: true`` — the API rejects
+        # the request server-side if absent. Mirror that contract locally.
+        if op in {"I", "R"} and self._get_value(self.sezione_censimento) is None:
+            raise SezioneCensimentoRequiredError(operazione=op)
+
         # Rule 6: re-validate embedded coordinates via ValidatedCoordinate
         if self.coordinate is not None:
             payload = self.coordinate.model_dump(exclude_unset=True)

{anncsu_sdk-0.0.9.dev16 → anncsu_sdk-0.0.9.dev17}/tests/accessi/test_accesso_validation.py

@@ -39,6 +39,7 @@ from anncsu.accessi.errors.accesso_validation import (
     NumeroMetricoMutexError,
     OperazioneCivicoError,
     ProgrCivicoRequiredError,
+    SezioneCensimentoRequiredError,
 )
 from anncsu.accessi.models.validated import ValidatedAccesso
 from anncsu.coordinate.errors.coordinate_validation import (
@@ -64,13 +65,18 @@ class TestAccessoOperazioneCivicoValidation:
 
     def test_operazione_civico_I_valid(self) -> None:
         """Test that operazione_civico='I' (insert) is accepted."""
-        accesso = ValidatedAccesso(operazione_civico="I", numero="12")
+        accesso = ValidatedAccesso(
+            operazione_civico="I", numero="12", sezione_censimento="9"
+        )
         assert accesso.operazione_civico == "I"
 
     def test_operazione_civico_R_valid(self) -> None:
         """Test that operazione_civico='R' (replace) is accepted."""
         accesso = ValidatedAccesso(
-            operazione_civico="R", progr_civico="1370588", numero="12"
+            operazione_civico="R",
+            progr_civico="1370588",
+            numero="12",
+            sezione_censimento="9",
         )
         assert accesso.operazione_civico == "R"
 
@@ -115,7 +121,9 @@ class TestAccessoProgrCivicoDependency:
 
     def test_insert_without_progr_civico_valid(self) -> None:
         """Test that I (insert) does not require progr_civico (assigned by API)."""
-        accesso = ValidatedAccesso(operazione_civico="I", numero="12")
+        accesso = ValidatedAccesso(
+            operazione_civico="I", numero="12", sezione_censimento="9"
+        )
         assert accesso.progr_civico is None
 
     def test_replace_requires_progr_civico(self) -> None:
@@ -137,7 +145,10 @@ class TestAccessoProgrCivicoDependency:
     def test_replace_with_progr_civico_valid(self) -> None:
         """Test that R with progr_civico is valid."""
         accesso = ValidatedAccesso(
-            operazione_civico="R", progr_civico="1370588", numero="12"
+            operazione_civico="R",
+            progr_civico="1370588",
+            numero="12",
+            sezione_censimento="9",
         )
         assert accesso.progr_civico == "1370588"
 
@@ -157,12 +168,16 @@ class TestAccessoNumeroMetricoMutex:
 
     def test_insert_with_numero_only_valid(self) -> None:
         """Test that I with only numero is valid (civico)."""
-        accesso = ValidatedAccesso(operazione_civico="I", numero="12")
+        accesso = ValidatedAccesso(
+            operazione_civico="I", numero="12", sezione_censimento="9"
+        )
         assert accesso.numero == "12"
 
     def test_insert_with_metrico_only_valid(self) -> None:
         """Test that I with only metrico is valid (metric-identified)."""
-        accesso = ValidatedAccesso(operazione_civico="I", metrico="300")
+        accesso = ValidatedAccesso(
+            operazione_civico="I", metrico="300", sezione_censimento="9"
+        )
         assert accesso.metrico == "300"
 
     def test_insert_with_both_numero_and_metrico_invalid(self) -> None:
@@ -182,14 +197,20 @@ class TestAccessoNumeroMetricoMutex:
     def test_replace_with_numero_only_valid(self) -> None:
         """Test that R with only numero is valid."""
         accesso = ValidatedAccesso(
-            operazione_civico="R", progr_civico="1370588", numero="12"
+            operazione_civico="R",
+            progr_civico="1370588",
+            numero="12",
+            sezione_censimento="9",
         )
         assert accesso.numero == "12"
 
     def test_replace_with_metrico_only_valid(self) -> None:
         """Test that R with only metrico is valid."""
         accesso = ValidatedAccesso(
-            operazione_civico="R", progr_civico="1370588", metrico="300"
+            operazione_civico="R",
+            progr_civico="1370588",
+            metrico="300",
+            sezione_censimento="9",
         )
         assert accesso.metrico == "300"
 
@@ -275,6 +296,68 @@ class TestAccessoFieldsNotAllowedForDelete:
         assert accesso.progr_civico == "1370588"
 
 
+# ---------------------------------------------------------------------------
+# 4b. sezione_censimento required for I/R (Issue #30)
+# ---------------------------------------------------------------------------
+
+
+class TestAccessoSezioneCensimentoRequired:
+    """Tests that ``sezione_censimento`` is mandatory for I and R.
+
+    Per OAS spec the field has no ``nullable: true`` and is described as
+    "da valorizzare solo per operazione_civico='I','R'". Missing it for
+    I or R must fail locally before the API call.
+    """
+
+    def test_insert_without_sezione_censimento_raises(self) -> None:
+        with pytest.raises(ValidationError) as exc_info:
+            ValidatedAccesso(operazione_civico="I", numero="12")
+        cause = get_validation_error_cause(exc_info)
+        assert isinstance(cause, SezioneCensimentoRequiredError)
+        assert "I" in str(cause)
+
+    def test_replace_without_sezione_censimento_raises(self) -> None:
+        with pytest.raises(ValidationError) as exc_info:
+            ValidatedAccesso(operazione_civico="R", progr_civico="1370588", numero="12")
+        cause = get_validation_error_cause(exc_info)
+        assert isinstance(cause, SezioneCensimentoRequiredError)
+        assert "R" in str(cause)
+
+    def test_insert_with_empty_sezione_censimento_raises(self) -> None:
+        """Empty string must be treated as missing (matches _get_value semantics)."""
+        with pytest.raises(ValidationError) as exc_info:
+            ValidatedAccesso(operazione_civico="I", numero="12", sezione_censimento="")
+        cause = get_validation_error_cause(exc_info)
+        assert isinstance(cause, SezioneCensimentoRequiredError)
+
+    def test_insert_with_sezione_censimento_valid(self) -> None:
+        accesso = ValidatedAccesso(
+            operazione_civico="I", numero="12", sezione_censimento="9"
+        )
+        assert accesso.sezione_censimento == "9"
+
+    def test_replace_with_sezione_censimento_valid(self) -> None:
+        accesso = ValidatedAccesso(
+            operazione_civico="R",
+            progr_civico="1370588",
+            numero="12",
+            sezione_censimento="9",
+        )
+        assert accesso.sezione_censimento == "9"
+
+    def test_delete_without_sezione_censimento_still_valid(self) -> None:
+        """Regression: S must NOT require sezione_censimento (Rule 4 still forbids it)."""
+        accesso = ValidatedAccesso(operazione_civico="S", progr_civico="1370588")
+        assert accesso.operazione_civico == "S"
+        assert accesso.sezione_censimento is None
+
+    def test_sezione_censimento_error_is_accesso_validation_error(self) -> None:
+        """The new error must sit under the AccessoValidationError hierarchy
+        so existing ``except AccessoValidationError`` blocks catch it."""
+        err = SezioneCensimentoRequiredError(operazione="I")
+        assert isinstance(err, AccessoValidationError)
+
+
 # ---------------------------------------------------------------------------
 # 5. maxLength constraints from OAS
 # ---------------------------------------------------------------------------
@@ -380,6 +463,7 @@ class TestAccessoCoordinateIntegration:
         accesso = ValidatedAccesso(
             operazione_civico="I",
             numero="12",
+            sezione_censimento="9",
             coordinate={
                 "x": "13.1022000",
                 "y": "41.8847600",
@@ -395,6 +479,7 @@ class TestAccessoCoordinateIntegration:
             ValidatedAccesso(
                 operazione_civico="I",
                 numero="12",
+                sezione_censimento="9",
                 coordinate={
                     "x": "99.0",  # outside Italy bounds (6-18)
                     "y": "41.8847600",
@@ -424,7 +509,9 @@ class TestAccessoCoordinateIntegration:
 
     def test_accesso_without_coordinates_valid(self) -> None:
         """Test that accesso without coordinates is valid (coordinates optional)."""
-        accesso = ValidatedAccesso(operazione_civico="I", numero="12")
+        accesso = ValidatedAccesso(
+            operazione_civico="I", numero="12", sezione_censimento="9"
+        )
         assert accesso.coordinate is None
 
 
@@ -444,6 +531,7 @@ class TestAccessoValidationErrorHierarchy:
             FieldNotAllowedForDeleteError,
             NumeroMetricoMutexError,
             FieldNotAllowedForOperationError,
+            SezioneCensimentoRequiredError,
             AccessoMaxLengthError,
         ):
             assert issubclass(err_class, AccessoValidationError)
@@ -492,6 +580,7 @@ class TestAccessoIntegration:
             operazione_civico="R",
             progr_civico="1370588",
             numero="12",
+            sezione_censimento="9",
             coordinate={
                 "x": "13.1022001",
                 "y": "41.8847601",
@@ -517,6 +606,7 @@ class TestAccessoIntegration:
         accesso = ValidatedAccesso(
             operazione_civico="I",
             metrico="300",
+            sezione_censimento="9",
             coordinate={
                 "x": "13.1022000",
                 "y": "41.8847600",

{anncsu_sdk-0.0.9.dev16 → anncsu_sdk-0.0.9.dev17}/tests/cli/test_accesso.py

@@ -148,6 +148,8 @@ class TestAccessoInsert:
                                 "2000449",
                                 "--numero",
                                 "12",
+                                "--sezione-censimento",
+                                "9",
                             ],
                         )
 
@@ -269,6 +271,8 @@ class TestAccessoUpdate:
                                 "1370588",
                                 "--numero",
                                 "12",
+                                "--sezione-censimento",
+                                "9",
                             ],
                         )
 
@@ -469,6 +473,8 @@ class TestAccessoJsonOutput:
                                 "2000449",
                                 "--numero",
                                 "12",
+                                "--sezione-censimento",
+                                "9",
                                 "--json",
                             ],
                         )
@@ -545,6 +551,8 @@ class TestAccessoInsertDryRun:
                         "2000449",
                         "--numero",
                         "12",
+                        "--sezione-censimento",
+                        "9",
                         "--dry-run",
                     ],
                 )
@@ -606,6 +614,8 @@ class TestAccessoInsertDryRun:
                         "2000449",
                         "--numero",
                         "12",
+                        "--sezione-censimento",
+                        "9",
                         "--dry-run",
                     ],
                 )
@@ -656,6 +666,8 @@ class TestAccessoInsertDryRun:
                         "2000449",
                         "--numero",
                         "12",
+                        "--sezione-censimento",
+                        "9",
                         "--dry-run",
                         "--json",
                     ],