diff options
| author | Nick Mathewson <nickm@torproject.org> | 2017-10-24 19:35:32 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-10-24 19:35:32 -0400 |
| commit | b95ef19a77e2e04c740bbbe16d3b0ed7a5239aee (patch) | |
| tree | 5555ffd9074588e5a9939f0d55ce5385e1ba3a00 /src/test/test_config.c | |
| parent | 594cf92498c8ea12dc0b19f743d6b88d4a98f1eb (diff) | |
| parent | f3af74ccf594c07f7edbf0bdfa576b9af21563d9 (diff) | |
| download | tor-b95ef19a77e2e04c740bbbe16d3b0ed7a5239aee.tar.gz tor-b95ef19a77e2e04c740bbbe16d3b0ed7a5239aee.zip | |
Merge branch 'feature18329_029_squashed' into maint-0.3.2
Diffstat (limited to 'src/test/test_config.c')
| -rw-r--r-- | src/test/test_config.c | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/src/test/test_config.c b/src/test/test_config.c index 68eb5419cd..456d8bcc3e 100644 --- a/src/test/test_config.c +++ b/src/test/test_config.c @@ -5449,7 +5449,8 @@ test_config_dup_and_filter(void *arg) /* If we're not configured to be a bridge, but we set * BridgeDistribution, then options_validate () should return -1. */ static void -test_config_check_bridge_distribution_setting_not_a_bridge(void *arg) { +test_config_check_bridge_distribution_setting_not_a_bridge(void *arg) +{ or_options_t* options = get_options_mutable(); or_options_t* old_options = options; or_options_t* default_options = options; @@ -5470,7 +5471,8 @@ test_config_check_bridge_distribution_setting_not_a_bridge(void *arg) { /* If the BridgeDistribution setting was valid, 0 should be returned. */ static void -test_config_check_bridge_distribution_setting_valid(void *arg) { +test_config_check_bridge_distribution_setting_valid(void *arg) +{ int ret = check_bridge_distribution_setting("https"); (void)arg; @@ -5482,11 +5484,16 @@ test_config_check_bridge_distribution_setting_valid(void *arg) { /* If the BridgeDistribution setting was invalid, -1 should be returned. */ static void -test_config_check_bridge_distribution_setting_invalid(void *arg) { - int ret = check_bridge_distribution_setting("hyphens-are-not-allowed"); +test_config_check_bridge_distribution_setting_invalid(void *arg) +{ + int ret = check_bridge_distribution_setting("hyphens-are-allowed"); (void)arg; + tt_int_op(ret, OP_EQ, 0); + + ret = check_bridge_distribution_setting("asterisks*are*forbidden"); + tt_int_op(ret, OP_EQ, -1); done: return; @@ -5495,7 +5502,8 @@ test_config_check_bridge_distribution_setting_invalid(void *arg) { /* If the BridgeDistribution setting was unrecognised, a warning should be * logged and 0 should be returned. */ static void -test_config_check_bridge_distribution_setting_unrecognised(void *arg) { +test_config_check_bridge_distribution_setting_unrecognised(void *arg) +{ int ret = check_bridge_distribution_setting("unicorn"); (void)arg; |