pytest: use pyln.client for functional tests

tests/test_closing.py

@@ -1,7 +1,10 @@
 from fixtures import *  # noqa: F401,F403
 from flaky import flaky
-from lightning import RpcError
+from pyln.client import RpcError
-from utils import only_one, sync_blockheight, wait_for, DEVELOPER, TIMEOUT, VALGRIND, SLOW_MACHINE, COMPAT
+from utils import (
+    only_one, sync_blockheight, wait_for, DEVELOPER, TIMEOUT, VALGRIND,
+    SLOW_MACHINE, COMPAT
+)
 
 import os
 import queue

tests/test_connection.py

@@ -3,8 +3,11 @@ from decimal import Decimal
 from fixtures import *  # noqa: F401,F403
 from fixtures import TEST_NETWORK
 from flaky import flaky  # noqa: F401
-from lightning import RpcError
+from pyln.client import RpcError
-from utils import DEVELOPER, only_one, wait_for, sync_blockheight, VALGRIND, TIMEOUT, SLOW_MACHINE, COMPAT, expected_features
+from utils import (
+    DEVELOPER, only_one, wait_for, sync_blockheight, VALGRIND, TIMEOUT,
+    SLOW_MACHINE, COMPAT, expected_features
+)
 from bitcoin.core import CMutableTransaction, CMutableTxOut
 
 import binascii

tests/test_gossip.py

@@ -2,8 +2,10 @@ from collections import Counter
 from ephemeral_port_reserve import reserve
 from fixtures import *  # noqa: F401,F403
 from fixtures import TEST_NETWORK
-from lightning import RpcError
+from pyln.client import RpcError
-from utils import wait_for, TIMEOUT, only_one, sync_blockheight, expected_features
+from utils import (
+    wait_for, TIMEOUT, only_one, sync_blockheight, expected_features
+)
 
 import json
 import logging

tests/test_invoices.py

@@ -1,6 +1,6 @@
 from fixtures import *  # noqa: F401,F403
 from fixtures import TEST_NETWORK
-from lightning import RpcError
+from pyln.client import RpcError
 from utils import only_one, DEVELOPER, wait_for, wait_channel_quiescent
 
 

tests/test_misc.py

@@ -3,9 +3,12 @@ from decimal import Decimal
 from fixtures import *  # noqa: F401,F403
 from fixtures import LightningNode, TEST_NETWORK
 from flaky import flaky  # noqa: F401
-from lightning import RpcError
+from pyln.client import RpcError
 from threading import Event
-from pyln.testing.utils import DEVELOPER, TIMEOUT, VALGRIND, sync_blockheight, only_one, wait_for, TailableProc, env
+from pyln.testing.utils import (
+    DEVELOPER, TIMEOUT, VALGRIND, sync_blockheight, only_one, wait_for,
+    TailableProc, env
+)
 from ephemeral_port_reserve import reserve
 
 import json

tests/test_pay.py

@@ -2,8 +2,11 @@ from binascii import hexlify
 from fixtures import *  # noqa: F401,F403
 from fixtures import TEST_NETWORK
 from flaky import flaky  # noqa: F401
-from lightning import RpcError, Millisatoshi
+from pyln.client import RpcError, Millisatoshi
-from utils import DEVELOPER, wait_for, only_one, sync_blockheight, SLOW_MACHINE, TIMEOUT, VALGRIND
+from utils import (
+    DEVELOPER, wait_for, only_one, sync_blockheight, SLOW_MACHINE, TIMEOUT,
+    VALGRIND
+)
 
 import concurrent.futures
 import copy

tests/test_plugin.py

@@ -1,8 +1,10 @@
 from collections import OrderedDict
 from fixtures import *  # noqa: F401,F403
 from flaky import flaky  # noqa: F401
-from lightning import RpcError, Millisatoshi
+from pyln.client import RpcError, Millisatoshi
-from utils import DEVELOPER, only_one, sync_blockheight, TIMEOUT, wait_for, TEST_NETWORK
+from utils import (
+    DEVELOPER, only_one, sync_blockheight, TIMEOUT, wait_for, TEST_NETWORK
+)
 
 import json
 import os

tests/test_wallet.py

@@ -2,8 +2,11 @@ from decimal import Decimal
 from fixtures import *  # noqa: F401,F403
 from fixtures import TEST_NETWORK
 from flaky import flaky  # noqa: F401
-from lightning import RpcError, Millisatoshi
+from pyln.client import RpcError, Millisatoshi
-from utils import only_one, wait_for, sync_blockheight, EXPERIMENTAL_FEATURES, COMPAT, VALGRIND
+from utils import (
+    only_one, wait_for, sync_blockheight, EXPERIMENTAL_FEATURES, COMPAT,
+    VALGRIND
+)
 
 import os
 import pytest