From ffaab09043120c2dbf52655757c694c8f0b8fde5 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Fri, 1 Sep 2017 13:48:38 +0930 Subject: [PATCH] gossipd/test: fix flaky test. The test is could actually go each way, since for 1000000 the fee is the same either way. Increase to 300000, and add an extra test when the alternate path is disabled. Signed-off-by: Rusty Russell --- gossipd/test/run-find_route.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/gossipd/test/run-find_route.c b/gossipd/test/run-find_route.c index 5c13e7a25..9087c18e7 100644 --- a/gossipd/test/run-find_route.c +++ b/gossipd/test/run-find_route.c @@ -108,11 +108,19 @@ int main(void) assert(fee == 0); /* Will go via B for large amounts. */ - nc = find_route(ctx, rstate, &a, &c, 1000000, 1.0, &fee, &route); + nc = find_route(ctx, rstate, &a, &c, 3000000, 1.0, &fee, &route); assert(nc); assert(tal_count(route) == 1); assert(pubkey_eq(&route[0]->src->id, &b)); - assert(fee == 2); + assert(fee == 1 + 3); + + /* Make B->C inactive, force it back via D */ + get_connection(rstate, &b, &c)->active = false; + nc = find_route(ctx, rstate, &a, &c, 3000000, 1.0, &fee, &route); + assert(nc); + assert(tal_count(route) == 1); + assert(pubkey_eq(&route[0]->src->id, &d)); + assert(fee == 0 + 6); tal_free(ctx); return 0;