November 2016 - tor-commits - lists.torproject.org

[tor/release-0.2.9] Add all four error messages to test_address_get_if_addrs6_list_internal()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 01492d3869a49ff005dd8590e11466dd3cc0045a Author: Neel Chauhan <neel(a)neelc.org> Date: Tue Nov 15 20:35:48 2016 -0500 Add all four error messages to test_address_get_if_addrs6_list_internal() --- src/test/test_address.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/test/test_address.c b/src/test/test_address.c index e52779c..e0a5ea4 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -808,8 +808,10 @@ test_address_get_if_addrs6_list_internal(void *arg) results = get_interface_address6_list(LOG_ERR, AF_INET6, 1); tt_int_op(smartlist_len(mock_saved_logs()), OP_LE, 1); if (smartlist_len(mock_saved_logs()) == 1) { - expect_log_msg_containing_either("connect() failed", - "unable to create socket"); + expect_log_msg_containing_either4("connect() failed", + "unable to create socket", + "Address that we determined via UDP socket magic is unsuitable for public comms.", + "getsockname() to determine interface failed"); } teardown_capture_of_logs();

1 0

[tor/master] Merge remote-tracking branch 'public/bug20630'
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit a742637f5ae033a1f05490cbf1d4444cd0529d32 Merge: 427dcb2 307d244 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Nov 17 20:09:44 2016 -0500 Merge remote-tracking branch 'public/bug20630' src/or/connection_edge.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-)

1 0

[tor/release-0.2.9] Add all four error messages to test_address_get_if_addrs6_list_no_internal()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 0825fc6af90b2015cb6a4abe6b8682f6f44d64e2 Author: Neel Chauhan <neel(a)neelc.org> Date: Tue Nov 15 20:36:53 2016 -0500 Add all four error messages to test_address_get_if_addrs6_list_no_internal() --- src/test/test_address.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/test/test_address.c b/src/test/test_address.c index e0a5ea4..fa32cf9 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -848,8 +848,11 @@ test_address_get_if_addrs6_list_no_internal(void *arg) results = get_interface_address6_list(LOG_ERR, AF_INET6, 0); tt_int_op(smartlist_len(mock_saved_logs()), OP_LE, 1); if (smartlist_len(mock_saved_logs()) == 1) { - expect_log_msg_containing_either("connect() failed", - "unable to create socket"); + expect_log_msg_containing_either4("connect() failed", + "unable to create socket", + "Address that we determined via UDP socket magic is unsuitable for public comms.", + "getsockname() to determine interface failed"); + } teardown_capture_of_logs();

1 0

[tor/release-0.2.9] fixup! Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 150a2b39b0efc17205bef07845e7a6d68b8c5148 Author: teor <teor2345(a)gmail.com> Date: Wed Nov 16 13:03:20 2016 +1100 fixup! Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4() Fix typos: * extra '(' * use assert_log_predicate (without 3 or 4 at the end) Tidy whitespace. Wrap long lines. --- src/test/log_test_helpers.h | 28 ++++++++++++++-------------- src/test/test_address.c | 15 ++++++++++----- 2 files changed, 24 insertions(+), 19 deletions(-) diff --git a/src/test/log_test_helpers.h b/src/test/log_test_helpers.h index a8ad7e8..922c68b 100644 --- a/src/test/log_test_helpers.h +++ b/src/test/log_test_helpers.h @@ -49,22 +49,22 @@ void mock_dump_saved_logs(void); #define expect_log_msg_containing_either(str1, str2) \ assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ mock_saved_log_has_message_containing(str2), \ - "expected log to contain " # str1 " or " # str2); - -#define expect_log_msg_containing_either3(str1, str2, str3) \ - assert_log_predicate3(mock_saved_log_has_message_containing(str1) || \ - (mock_saved_log_has_message_containing(str2) || \ - mock_saved_log_has_message_containing(str3), \ - "expected log to contain " # str1 " or " # str2 " or " # str3); - -#define expect_log_msg_containing_either4(str1, str2, str3, str4) \ - assert_log_predicate4(mock_saved_log_has_message_containing(str1) || \ - (mock_saved_log_has_message_containing(str2) || \ - (mock_saved_log_has_message_containing(str3) || \ - mock_saved_log_has_message_containing(str4), \ - "expected log to contain " # str1 " or " # str2 " or " # str3 " or " # str4); + "expected log to contain " # str1 " or " # str2); +#define expect_log_msg_containing_either3(str1, str2, str3) \ + assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ + mock_saved_log_has_message_containing(str2) || \ + mock_saved_log_has_message_containing(str3), \ + "expected log to contain " # str1 " or " # str2 \ + " or " # str3); +#define expect_log_msg_containing_either4(str1, str2, str3, str4) \ + assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ + mock_saved_log_has_message_containing(str2) || \ + mock_saved_log_has_message_containing(str3) || \ + mock_saved_log_has_message_containing(str4), \ + "expected log to contain " # str1 " or " # str2 \ + " or " # str3 " or " # str4); #define expect_single_log_msg(str) \ do { \ diff --git a/src/test/test_address.c b/src/test/test_address.c index fa32cf9..0d142ad 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -810,8 +810,11 @@ test_address_get_if_addrs6_list_internal(void *arg) if (smartlist_len(mock_saved_logs()) == 1) { expect_log_msg_containing_either4("connect() failed", "unable to create socket", - "Address that we determined via UDP socket magic is unsuitable for public comms.", - "getsockname() to determine interface failed"); + "Address that we determined via UDP " + "socket magic is unsuitable for public " + "comms.", + "getsockname() to determine interface " + "failed"); } teardown_capture_of_logs(); @@ -850,9 +853,11 @@ test_address_get_if_addrs6_list_no_internal(void *arg) if (smartlist_len(mock_saved_logs()) == 1) { expect_log_msg_containing_either4("connect() failed", "unable to create socket", - "Address that we determined via UDP socket magic is unsuitable for public comms.", - "getsockname() to determine interface failed"); - + "Address that we determined via UDP " + "socket magic is unsuitable for public " + "comms.", + "getsockname() to determine interface " + "failed"); } teardown_capture_of_logs();

1 0

[tor/master] Changes file for bug 20634
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 44f3563d872b56593c11a6e557b130714fa8916b Author: teor <teor2345(a)gmail.com> Date: Wed Nov 16 13:08:39 2016 +1100 Changes file for bug 20634 --- changes/bug20634 | 3 +++ 1 file changed, 3 insertions(+) diff --git a/changes/bug20634 b/changes/bug20634 new file mode 100644 index 0000000..62fc9f4 --- /dev/null +++ b/changes/bug20634 @@ -0,0 +1,3 @@ + o Minor bugfixes (unit tests): + - Stop spurious failures in the local interface address discovery unit + tests. Fixes bug 20634; bugfix on 0.2.8.1-alpha; patch by Neel Chauhan.

1 0

[tor/release-0.2.9] Merge branch 'maint-0.2.9' into release-0.2.9
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 33c54701b397e8983e913f5500b68b4dd5c5ef02 Merge: c290053 8a767ba Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Nov 17 20:08:24 2016 -0500 Merge branch 'maint-0.2.9' into release-0.2.9 changes/bug20634 | 3 +++ src/test/log_test_helpers.h | 17 ++++++++++++++++- src/test/test_address.c | 18 ++++++++++++++---- 3 files changed, 33 insertions(+), 5 deletions(-)

1 0

[tor/maint-0.2.9] Add all four error messages to test_address_get_if_addrs6_list_internal()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 01492d3869a49ff005dd8590e11466dd3cc0045a Author: Neel Chauhan <neel(a)neelc.org> Date: Tue Nov 15 20:35:48 2016 -0500 Add all four error messages to test_address_get_if_addrs6_list_internal() --- src/test/test_address.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/test/test_address.c b/src/test/test_address.c index e52779c..e0a5ea4 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -808,8 +808,10 @@ test_address_get_if_addrs6_list_internal(void *arg) results = get_interface_address6_list(LOG_ERR, AF_INET6, 1); tt_int_op(smartlist_len(mock_saved_logs()), OP_LE, 1); if (smartlist_len(mock_saved_logs()) == 1) { - expect_log_msg_containing_either("connect() failed", - "unable to create socket"); + expect_log_msg_containing_either4("connect() failed", + "unable to create socket", + "Address that we determined via UDP socket magic is unsuitable for public comms.", + "getsockname() to determine interface failed"); } teardown_capture_of_logs();

1 0

[tor/maint-0.2.9] Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit cccc08ad565f340bd4e6d19d0137bc45ced93107 Author: Neel Chauhan <neel(a)neelc.org> Date: Tue Nov 15 20:33:31 2016 -0500 Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4() --- src/test/log_test_helpers.h | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/test/log_test_helpers.h b/src/test/log_test_helpers.h index 4c020c7..a8ad7e8 100644 --- a/src/test/log_test_helpers.h +++ b/src/test/log_test_helpers.h @@ -51,6 +51,21 @@ void mock_dump_saved_logs(void); mock_saved_log_has_message_containing(str2), \ "expected log to contain " # str1 " or " # str2); +#define expect_log_msg_containing_either3(str1, str2, str3) \ + assert_log_predicate3(mock_saved_log_has_message_containing(str1) || \ + (mock_saved_log_has_message_containing(str2) || \ + mock_saved_log_has_message_containing(str3), \ + "expected log to contain " # str1 " or " # str2 " or " # str3); + +#define expect_log_msg_containing_either4(str1, str2, str3, str4) \ + assert_log_predicate4(mock_saved_log_has_message_containing(str1) || \ + (mock_saved_log_has_message_containing(str2) || \ + (mock_saved_log_has_message_containing(str3) || \ + mock_saved_log_has_message_containing(str4), \ + "expected log to contain " # str1 " or " # str2 " or " # str3 " or " # str4); + + + #define expect_single_log_msg(str) \ do { \ \

1 0

[tor/maint-0.2.9] Add all four error messages to test_address_get_if_addrs6_list_no_internal()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 0825fc6af90b2015cb6a4abe6b8682f6f44d64e2 Author: Neel Chauhan <neel(a)neelc.org> Date: Tue Nov 15 20:36:53 2016 -0500 Add all four error messages to test_address_get_if_addrs6_list_no_internal() --- src/test/test_address.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/test/test_address.c b/src/test/test_address.c index e0a5ea4..fa32cf9 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -848,8 +848,11 @@ test_address_get_if_addrs6_list_no_internal(void *arg) results = get_interface_address6_list(LOG_ERR, AF_INET6, 0); tt_int_op(smartlist_len(mock_saved_logs()), OP_LE, 1); if (smartlist_len(mock_saved_logs()) == 1) { - expect_log_msg_containing_either("connect() failed", - "unable to create socket"); + expect_log_msg_containing_either4("connect() failed", + "unable to create socket", + "Address that we determined via UDP socket magic is unsuitable for public comms.", + "getsockname() to determine interface failed"); + } teardown_capture_of_logs();

1 0

[tor/maint-0.2.9] fixup! Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4()
by nickm＠torproject.org 17 Nov '16

17 Nov '16

commit 150a2b39b0efc17205bef07845e7a6d68b8c5148 Author: teor <teor2345(a)gmail.com> Date: Wed Nov 16 13:03:20 2016 +1100 fixup! Add expect_log_msg_containing_either3() and expect_log_msg_containing_either4() Fix typos: * extra '(' * use assert_log_predicate (without 3 or 4 at the end) Tidy whitespace. Wrap long lines. --- src/test/log_test_helpers.h | 28 ++++++++++++++-------------- src/test/test_address.c | 15 ++++++++++----- 2 files changed, 24 insertions(+), 19 deletions(-) diff --git a/src/test/log_test_helpers.h b/src/test/log_test_helpers.h index a8ad7e8..922c68b 100644 --- a/src/test/log_test_helpers.h +++ b/src/test/log_test_helpers.h @@ -49,22 +49,22 @@ void mock_dump_saved_logs(void); #define expect_log_msg_containing_either(str1, str2) \ assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ mock_saved_log_has_message_containing(str2), \ - "expected log to contain " # str1 " or " # str2); - -#define expect_log_msg_containing_either3(str1, str2, str3) \ - assert_log_predicate3(mock_saved_log_has_message_containing(str1) || \ - (mock_saved_log_has_message_containing(str2) || \ - mock_saved_log_has_message_containing(str3), \ - "expected log to contain " # str1 " or " # str2 " or " # str3); - -#define expect_log_msg_containing_either4(str1, str2, str3, str4) \ - assert_log_predicate4(mock_saved_log_has_message_containing(str1) || \ - (mock_saved_log_has_message_containing(str2) || \ - (mock_saved_log_has_message_containing(str3) || \ - mock_saved_log_has_message_containing(str4), \ - "expected log to contain " # str1 " or " # str2 " or " # str3 " or " # str4); + "expected log to contain " # str1 " or " # str2); +#define expect_log_msg_containing_either3(str1, str2, str3) \ + assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ + mock_saved_log_has_message_containing(str2) || \ + mock_saved_log_has_message_containing(str3), \ + "expected log to contain " # str1 " or " # str2 \ + " or " # str3); +#define expect_log_msg_containing_either4(str1, str2, str3, str4) \ + assert_log_predicate(mock_saved_log_has_message_containing(str1) || \ + mock_saved_log_has_message_containing(str2) || \ + mock_saved_log_has_message_containing(str3) || \ + mock_saved_log_has_message_containing(str4), \ + "expected log to contain " # str1 " or " # str2 \ + " or " # str3 " or " # str4); #define expect_single_log_msg(str) \ do { \ diff --git a/src/test/test_address.c b/src/test/test_address.c index fa32cf9..0d142ad 100644 --- a/src/test/test_address.c +++ b/src/test/test_address.c @@ -810,8 +810,11 @@ test_address_get_if_addrs6_list_internal(void *arg) if (smartlist_len(mock_saved_logs()) == 1) { expect_log_msg_containing_either4("connect() failed", "unable to create socket", - "Address that we determined via UDP socket magic is unsuitable for public comms.", - "getsockname() to determine interface failed"); + "Address that we determined via UDP " + "socket magic is unsuitable for public " + "comms.", + "getsockname() to determine interface " + "failed"); } teardown_capture_of_logs(); @@ -850,9 +853,11 @@ test_address_get_if_addrs6_list_no_internal(void *arg) if (smartlist_len(mock_saved_logs()) == 1) { expect_log_msg_containing_either4("connect() failed", "unable to create socket", - "Address that we determined via UDP socket magic is unsuitable for public comms.", - "getsockname() to determine interface failed"); - + "Address that we determined via UDP " + "socket magic is unsuitable for public " + "comms.", + "getsockname() to determine interface " + "failed"); } teardown_capture_of_logs();

1 0