anncsu-sdk 0.0.9.dev28__tar.gz → 0.0.9.dev29__tar.gz

{anncsu_sdk-0.0.9.dev28 → anncsu_sdk-0.0.9.dev29}/PKG-INFO

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

{anncsu_sdk-0.0.9.dev28 → anncsu_sdk-0.0.9.dev29}/docs/CLI.md

@@ -2236,6 +2236,8 @@ Options:
 
 > **Note**: `delete` does NOT accept `--dug`, `--denom-*`, `--codice-comunale`, `--provv-*`, or `--prefettura-*` — Typer rejects them at parse time as unknown options because they have no meaning for a soppressione.
 
+> **Note on the S response shape**: the server response for a soppressione **does not** carry `esito` (it is `null`). The CLI infers success from the presence of `data_FINE` and `data_fine_valid_amm` in the `dati` payload — those mark the closure timestamp of the odonimo. When the CLI shows `Operation 'S' successful!`, this is the path that was taken. The behaviour is identical between the real `delete` and `delete --dry-run` since both call the same `_build_result` helper.
+
 #### `anncsu odonimo status`
 
 Check the status of the Odonimi API service:

{anncsu_sdk-0.0.9.dev28 → anncsu_sdk-0.0.9.dev29}/docs/sdk-odonimi.md

@@ -199,6 +199,64 @@ L'operazione effettiva è determinata dal campo `Richiesta.tipo_operazione`:
 |--------|-------------|
 | `show_status` | Verifica stato del servizio |
 
+## Response shape (asimmetria fra I/R e S)
+
+Le tre operazioni **non** rispondono con la stessa forma. Verificato empiricamente su UAT 2026-05-25.
+
+### I e R — `esito` valorizzato
+
+```json
+{
+  "idRichiesta": "316768",
+  "esito": "0",
+  "messaggio": "OK",
+  "dati": [{"codcom": "H501", "progr_nazionale": "1342677", ...}]
+}
+```
+
+Detection di successo: `esito == "0"`.
+
+### S — `esito` omesso, marker `data_FINE`
+
+```json
+{
+  "idRichiesta": "316775",
+  "dati": [{
+    "codcom": "H501",
+    "progr_nazionale": "1342672",
+    "dug": "VIA",
+    "denom_delibera": "TEST SDK INSERT 20260525",
+    "data_inizio": "25/05/2026",
+    "data_FINE": "25/05/2026",
+    "data_ini_valid_amm": "25/05/2026",
+    "data_fine_valid_amm": "25/05/2026"
+  }]
+}
+```
+
+`esito` e `messaggio` sono assenti / `null`. Il marker di successo per S è la presenza di `data_FINE` (alias Speakeasy: `data_fine`) o di `data_fine_valid_amm` nel primo elemento di `dati`.
+
+### Detection lato client (programmatic)
+
+Se usi l'SDK direttamente, replica questo pattern (è esattamente quello che usa la CLI `_build_result`):
+
+```python
+def is_success(response, tipo_operazione: str) -> bool:
+    esito = getattr(response, "esito", None)
+    dati = getattr(response, "dati", []) or []
+    if esito is not None:
+        return esito == "0"
+    if tipo_operazione == "S" and dati:
+        first = dati[0]
+        return bool(
+            getattr(first, "data_fine", None)              # alias per data_FINE
+            or getattr(first, "data_fine_valid_amm", None)
+        )
+    return False
+```
+
+Una check ingenua `esito == "0"` produce un **false-negative** sulle S che vanno a buon fine — è una causa nota di errori in codice cliente.
+
 ## Validazione Odonimo
 
 L'SDK fornisce un modello `ValidatedOdonimo` con validazione delle business rules ANNCSU. Va sempre usato prima della chiamata API per intercettare errori a livello locale.

{anncsu_sdk-0.0.9.dev28 → anncsu_sdk-0.0.9.dev29}/src/anncsu/cli/commands/odonimo.py

@@ -972,17 +972,8 @@ def _execute_operation(
     if raw_output:
         _print_raw(response)
 
-    esito = getattr(response, "esito", None)
-    success = esito == "0"
-
-    result = OdonimoOperationResult(
-        success=success,
-        tipo_operazione=tipo_operazione,
-        id_richiesta=getattr(response, "id_richiesta", None),
-        esito=esito,
-        messaggio=getattr(response, "messaggio", None),
-        dati_count=len(getattr(response, "dati", []) or []),
-    )
+    result = _build_result(response, tipo_operazione)
+    success = result.success
 
     if json_output:
         print(result.model_dump_json(indent=2))

{anncsu_sdk-0.0.9.dev28 → anncsu_sdk-0.0.9.dev29}/tests/cli/test_odonimo.py

@@ -294,6 +294,63 @@ class TestOdonimoDelete:
 
         assert result.exit_code == 0, result.output
 
+    def test_delete_success_with_s_marker_response(
+        self, cli_runner: CliRunner, tmp_path: Path, mock_private_key: Path
+    ) -> None:
+        """Real (non-dry-run) ``delete`` must treat an S response that omits
+        ``esito`` but carries ``data_FINE`` / ``data_fine_valid_amm`` markers
+        as success — same logic as ``_build_result`` already applies in the
+        dry-run helpers.
+
+        Regression: ``_execute_operation`` used to evaluate ``success =
+        esito == "0"`` directly, producing a false-negative ``Operation 'S'
+        failed`` on UAT even when the server had committed the cancellation.
+        """
+        from anncsu.cli import app
+
+        item = MagicMock()
+        item.data_fine = "25/05/2026"
+        item.data_fine_valid_amm = "25/05/2026"
+        response = MagicMock()
+        response.id_richiesta = "REQ-S-MARKER"
+        response.esito = None
+        response.messaggio = None
+        response.dati = [item]
+
+        with cli_runner.isolated_filesystem(temp_dir=tmp_path):
+            Path("private_key.pem").write_text(mock_private_key.read_text())
+            with patch(
+                "anncsu.cli.commands.odonimo.ClientAssertionSettings"
+            ) as mock_settings:
+                settings = MagicMock()
+                settings.has_modi_audit_context.return_value = False
+                mock_settings.return_value = settings
+                with patch(
+                    "anncsu.cli.commands.odonimo.PDNDAuthManager"
+                ) as mock_manager:
+                    manager = MagicMock()
+                    manager.get_access_token.return_value = "mock-token"
+                    mock_manager.return_value = manager
+                    with patch("anncsu.cli.commands.odonimo.AnncsuOdonimi") as mock_sdk:
+                        sdk = MagicMock()
+                        sdk.anncsu.gestione_anncsu_odonimi_pdnd.return_value = response
+                        mock_sdk.return_value = sdk
+
+                        result = cli_runner.invoke(
+                            app,
+                            [
+                                "odonimo",
+                                "delete",
+                                "--codcom",
+                                "H501",
+                                "--prognaz",
+                                "1342672",
+                            ],
+                        )
+
+        assert result.exit_code == 0, result.output
+        assert "successful" in result.output.lower()
+
 
 # ---------------------------------------------------------------------------
 # JSON output