config.json

@@ -5,7 +5,7 @@
   "short_description": "A NWC service provider for LNbits",
   "description": "",
   "tile": "/nwcprovider/static/image/nwcprovider.png",
-  "version": "1.1.3-aio.1",
+  "version": "1.1.3",
   "min_lnbits_version": "1.4.0",
   "contributors": [
     {

tasks.py

@@ -264,11 +264,9 @@ async def _on_make_invoice(
     # Ensures amount is provided
     if not amount_msats:
         raise Exception("Missing amount")
-    # Optional params may arrive as explicit JSON null, which dict.get does
+    description = params.get("description", "")
-    # not default, so coerce here rather than trusting the fallback.
+    description_hash = params.get("description_hash", None)
-    description = params.get("description") or ""
+    expiry = params.get("expiry", None)
-    description_hash = params.get("description_hash") or None
-    expiry = params.get("expiry") or None
 
     # hardening #
     assert_valid_msats(amount_msats)
@@ -286,7 +284,7 @@ async def _on_make_invoice(
         currency="sat",
         memo=description,
         description_hash=bytes.fromhex(description_hash) if description_hash else None,
-        unhashed_description=description.encode("utf-8") if description else None,
+        unhashed_description=description.encode("utf-8"),
         expiry=expiry,
     )
     payment_hash = payment.payment_hash
@@ -396,12 +394,12 @@ async def _on_list_transactions(
     if not nwc:
         raise Exception("Pubkey has no associated wallet")
     params = payload.get("params", 0)
-    tfrom = params.get("from") or 0
+    tfrom = params.get("from", 0)
-    tuntil = params.get("until") or int(time.time())
+    tuntil = params.get("until", int(time.time()))
-    limit = params.get("limit") or 10
+    limit = params.get("limit", 10)
-    offset = params.get("offset") or 0
+    offset = params.get("offset", 0)
-    unpaid = params.get("unpaid") or False
+    unpaid = params.get("unpaid", False)
-    tx_type = params.get("type") or ""
+    tx_type = params.get("type", "")
 
     # hardening #
     assert_valid_positive_int(tfrom)

tests/unit/test_tasks.py

@@ -78,77 +78,3 @@ async def test_process_invoice_backs_off_pending_payment_polling_to_configured_m
     assert result["preimage"] == "b" * 64
     assert result["fee_msats"] == 10
     assert result["paid"] is True
-
-
-def _stub_make_invoice(monkeypatch, captured: dict):
-    async def fake_get_nwc(*args, **kwargs):
-        return SimpleNamespace(wallet="wallet123")
-
-    async def fake_check(nwc, method):
-        return None
-
-    async def fake_create_invoice(**kwargs):
-        captured.update(kwargs)
-        return SimpleNamespace(payment_hash="b" * 64, bolt11="lnbc1example")
-
-    async def fake_check_transaction_status(wallet_id: str, payment_hash: str):
-        return SimpleNamespace(preimage=None)
-
-    monkeypatch.setattr(tasks, "get_nwc", fake_get_nwc)
-    monkeypatch.setattr(tasks, "_check", fake_check)
-    monkeypatch.setattr(tasks, "create_invoice", fake_create_invoice)
-    monkeypatch.setattr(
-        tasks, "check_transaction_status", fake_check_transaction_status
-    )
-
-
-@pytest.mark.asyncio
-@pytest.mark.parametrize(
-    "params",
-    [
-        # rust-nostr, Alby JS SDK: unused optional fields are omitted
-        {"amount": 21000},
-        # Amethyst: unused optional fields are sent as explicit null
-        {
-            "amount": 21000,
-            "description": None,
-            "description_hash": None,
-            "expiry": None,
-        },
-    ],
-)
-async def test_make_invoice_accepts_absent_or_null_optional_params(monkeypatch, params):
-    captured: dict = {}
-    _stub_make_invoice(monkeypatch, captured)
-
-    [(result, error, _)] = await tasks._on_make_invoice(
-        SimpleNamespace(), "a" * 64, {"params": params}
-    )
-
-    assert error is None
-    assert result["invoice"] == "lnbc1example"
-    assert result["description"] == ""
-    assert "expires_at" not in result
-    assert captured["memo"] == ""
-    assert captured["description_hash"] is None
-    assert captured["unhashed_description"] is None
-    assert captured["expiry"] is None
-
-
-@pytest.mark.asyncio
-async def test_make_invoice_passes_description_and_expiry_through(monkeypatch):
-    captured: dict = {}
-    _stub_make_invoice(monkeypatch, captured)
-
-    [(result, error, _)] = await tasks._on_make_invoice(
-        SimpleNamespace(),
-        "a" * 64,
-        {"params": {"amount": 21000, "description": "coffee", "expiry": 600}},
-    )
-
-    assert error is None
-    assert captured["memo"] == "coffee"
-    assert captured["unhashed_description"] == b"coffee"
-    assert captured["expiry"] == 600
-    assert result["description"] == "coffee"
-    assert "expires_at" in result