From ce9a0dd1a222a2b1197d6bbcae0895ee3b8f8370 Mon Sep 17 00:00:00 2001 From: Henning Jacobs Date: Tue, 24 Nov 2015 10:50:04 +0100 Subject: [PATCH] #152 fix output message of senza traffic --- senza/traffic.py | 2 +- tests/test_cli.py | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/senza/traffic.py b/senza/traffic.py index 545d939a..46bf7772 100644 --- a/senza/traffic.py +++ b/senza/traffic.py @@ -107,7 +107,7 @@ def compensate(calculation_error, compensations, identifier, new_record_weights, def set_new_weights(dns_names: list, identifier, lb_dns_name: str, new_record_weights, percentage): - action('Setting weights for {dns_names}..', **vars()) + action('Setting weights for {dns_names}..', dns_names=', '.join(dns_names)) dns_changes = {} for idx, dns_name in enumerate(dns_names): domain = dns_name.split('.', 1)[1] diff --git a/tests/test_cli.py b/tests/test_cli.py index 2f0b6fd5..bb0e48d4 100644 --- a/tests/test_cli.py +++ b/tests/test_cli.py @@ -1161,6 +1161,7 @@ def change_rr_set(HostedZoneId, ChangeBatch): def run(opts): result = runner.invoke(cli, common_opts + opts, catch_exceptions=False) + assert 'Setting weights for myapp.example.org..' in result.output return result def weights():