Skip to content

Commit

Permalink
pytest: more testing for local socket addresses.
Browse files Browse the repository at this point in the history
In particular, the assert when `--addr=/sockname` is used, and that it
doesn't clean up on restart, requiring manual deletion of the socket.

Signed-off-by: Rusty Russell <[email protected]>
  • Loading branch information
rustyrussell authored and cdecker committed Apr 8, 2019
1 parent b60b09a commit 3221517
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions tests/test_misc.py
Original file line number Diff line number Diff line change
Expand Up @@ -566,6 +566,7 @@ def test_io_logging(node_factory, executor):
assert any(l['type'] == 'IO_IN' for l in peerlog)


@pytest.mark.xfail(strict=True)
def test_address(node_factory):
if DEVELOPER:
opts = {'dev-allow-localhost': None}
Expand Down Expand Up @@ -595,6 +596,16 @@ def test_address(node_factory):
l2 = node_factory.get_node()
l2.rpc.connect(l1.info['id'], l1.daemon.opts['bind-addr'])

# 'addr' with local socket works too.
l1.stop()
del l1.daemon.opts['bind-addr']
l1.daemon.opts['addr'] = os.path.join(l1.daemon.lightning_dir, "sock")
# start expects a port, so we open-code here.
l1.daemon.start()

l2 = node_factory.get_node()
l2.rpc.connect(l1.info['id'], l1.daemon.opts['addr'])


def test_listconfigs(node_factory, bitcoind):
l1 = node_factory.get_node()
Expand Down

0 comments on commit 3221517

Please sign in to comment.