From d1ab5e38725b9580acdd81fcbdd886898d9a7974 Mon Sep 17 00:00:00 2001
From: Rusty Russell <rusty@rustcorp.com.au>
Date: Tue, 15 Jun 2021 06:37:38 +0930
Subject: [PATCH] pyln-client: hack in test that getroute returns the same
 old/new.

It sometimes fails because the two race, and sometimes because there's
randomness, but it generally works (and doesn't fail systemically).

We remove this before the final merge.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
---
 contrib/pyln-client/pyln/client/lightning.py | 26 +++++++++++++++++++-
 1 file changed, 25 insertions(+), 1 deletion(-)

diff --git a/contrib/pyln-client/pyln/client/lightning.py b/contrib/pyln-client/pyln/client/lightning.py
index f4e53f319..7c088c249 100644
--- a/contrib/pyln-client/pyln/client/lightning.py
+++ b/contrib/pyln-client/pyln/client/lightning.py
@@ -816,7 +816,31 @@ class LightningRpc(UnixDomainSocketRpc):
             "exclude": exclude,
             "maxhops": maxhops
         }
-        return self.call("getroute", payload)
+
+        # This is a hack to make sure old and new routines return the same result.
+        e1 = None
+        ret1 = None
+        err = None
+        try:
+            ret1 = self.call("getroute", payload)
+        except RpcError as e:
+            err = e
+            e1 = e.error
+        e2 = None
+        ret2 = None
+        try:
+            ret2 = self.call("getrouteold", payload)
+        except RpcError as e:
+            e2 = e.error
+
+        print("new getroute: {} exception {}".format(ret1, e1))
+        print("old getroute: {} exception {}".format(ret2, e2))
+        assert ret1 == ret2
+        assert e1 == e2
+
+        if err is not None:
+            raise err
+        return ret1
 
     def help(self, command=None):
         """