paytest: Fix issue with payments destined for the current node

See previous commit for details.
lightningd · Mar 25, 2021 · efd1b7e · efd1b7e
1 parent fb77c49
commit efd1b7e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/paytest/paytest.py b/paytest/paytest.py
@@ -246,7 +246,7 @@ def on_htlc_accepted(onion, htlc, request, plugin, *args, **kwargs):
         )
     )
     # If this is not a test payment, pass it on
-    if onion["short_channel_id"] != "1x1x1":
+    if 'short_channel_id' not in onion or onion["short_channel_id"] != "1x1x1":
         return request.set_result({"result": "continue"})
 
     # Decode the onion so we get the details the virtual recipient

diff --git a/paytest/test_paytest.py b/paytest/test_paytest.py
@@ -57,7 +57,6 @@ def test_mpp_pay(node_factory):
     assert len(is16399) >= 1
 
 
-@pytest.mark.xfail(strict=True)
 def test_incoming_payment(node_factory):
     """Ensure that we don't fail if the payment is not a paytest.
     """