From a525427272b9463b32a18c2591c67f1b76183df2 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Sun, 18 Aug 2019 19:04:53 +0930 Subject: [PATCH] pytest: don't run test_forward_local_failed_stats under Travis w/ VALGRIND It's timing out: I suspect it's simply too much memory. Signed-off-by: Rusty Russell --- tests/test_pay.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/test_pay.py b/tests/test_pay.py index 80180984a..0376855bc 100644 --- a/tests/test_pay.py +++ b/tests/test_pay.py @@ -1,7 +1,7 @@ from fixtures import * # noqa: F401,F403 from flaky import flaky # noqa: F401 from lightning import RpcError, Millisatoshi -from utils import DEVELOPER, wait_for, only_one, sync_blockheight, SLOW_MACHINE, TIMEOUT +from utils import DEVELOPER, wait_for, only_one, sync_blockheight, SLOW_MACHINE, TIMEOUT, VALGRIND import copy @@ -1120,7 +1120,7 @@ def test_forward_stats(node_factory, bitcoind): assert 'received_time' in stats['forwards'][2] and 'resolved_time' not in stats['forwards'][2] -@unittest.skipIf(not DEVELOPER, "needs DEVELOPER=1") +@unittest.skipIf(not DEVELOPER or (VALGRIND and SLOW_MACHINE), "Gossip too slow without DEVELOPER, and too stressful if VALGRIND on slow machines") def test_forward_local_failed_stats(node_factory, bitcoind, executor): """Check that we track forwarded payments correctly.