diff options
| author | David Goulet <dgoulet@torproject.org> | 2021-02-11 16:32:17 -0500 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2021-02-12 13:13:43 -0500 |
| commit | d47e937a5074c8c9799e1d8bbd6645eea74b86ab (patch) | |
| tree | 5637782c37736c9579e8a8e809d2a117423eba86 | |
| parent | dfcb050bbf424b6e72acb1bccd2dd99e8c96cb8c (diff) | |
| download | tor-d47e937a5074c8c9799e1d8bbd6645eea74b86ab.tar.gz tor-d47e937a5074c8c9799e1d8bbd6645eea74b86ab.zip | |
test: Fix duplicate ORPort test
The comment of that specific unit test wanted 4 ORPorts but for some reasons
we tested for 3 which before the previous commit related to #40289, test would
pass but it was in fact wrong.
Now the code is correct and 4 was in fact correct expected number of ports.
Related to #40289
Signed-off-by: David Goulet <dgoulet@torproject.org>
| -rw-r--r-- | src/test/test_config.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/test/test_config.c b/src/test/test_config.c index 4eb4ac9cf5..cd7a54b97a 100644 --- a/src/test/test_config.c +++ b/src/test/test_config.c @@ -6907,12 +6907,14 @@ test_config_duplicate_orports(void *arg) /* We have four address here, 1 IPv4 on 9050, IPv6 on 9050, IPv6 on 9051 and * a different IPv6 on 9051. */ - tt_int_op(smartlist_len(ports), OP_EQ, 3); + tt_int_op(smartlist_len(ports), OP_EQ, 4); tt_str_op(describe_relay_port(smartlist_get(ports, 0)), OP_EQ, "ORPort 9050"); tt_str_op(describe_relay_port(smartlist_get(ports, 1)), OP_EQ, "ORPort [4242::1]:9051"); tt_str_op(describe_relay_port(smartlist_get(ports, 2)), OP_EQ, + "ORPort [4242::2]:9051"); + tt_str_op(describe_relay_port(smartlist_get(ports, 3)), OP_EQ, "ORPort 9050"); /* Reset. Test different ORPort value. */ |