[tor-commits] [tor/master] Make get_ifaddrs tests more tolerant of unusual network configs

nickm at torproject.org nickm at torproject.org
Wed Oct 7 19:21:27 UTC 2015


commit c464a367728dcd123b9ad9bd0cf6a4a3e4397310
Author: teor (Tim Wilson-Brown) <teor2345 at gmail.com>
Date:   Tue Oct 6 15:32:44 2015 +1100

    Make get_ifaddrs tests more tolerant of unusual network configs
    
    * Don't assume that every test box has an IPv4 address
    * Don't assume that every test box has a non-local address
    
    Resolves issue #17255 released in unit tests in 0.2.7.3-rc.
---
 changes/ifaddrs-tests-network-configs |    7 +++++
 src/test/test_address.c               |   46 +++++++++++++++++++++------------
 2 files changed, 37 insertions(+), 16 deletions(-)

diff --git a/changes/ifaddrs-tests-network-configs b/changes/ifaddrs-tests-network-configs
new file mode 100644
index 0000000..f7f3d3e
--- /dev/null
+++ b/changes/ifaddrs-tests-network-configs
@@ -0,0 +1,7 @@
+  o Minor fixes (unit tests, networking):
+    - Make get_ifaddrs_* unit tests more tolerant of different network
+      configurations:
+        - Don't assume every test box has an IPv4 address.
+        - Don't assume every test box has a non-localhost address.
+    - Closes ticket #17255. Patch by "teor".
+      Fix for unit tests released in 0.2.7.3.
diff --git a/src/test/test_address.c b/src/test/test_address.c
index 938f776..3e73c3e 100644
--- a/src/test/test_address.c
+++ b/src/test/test_address.c
@@ -688,16 +688,20 @@ test_address_get_if_addrs_list_internal(void *arg)
   results = get_interface_address_list(LOG_ERR, 1);
 
   tt_assert(results != NULL);
-  /* Assume every system has at least 1 non-local non-multicast IPv4
-   * interface, even if it is an internal one */
-  tt_int_op(smartlist_len(results),>=,1);
+  /* When the network is down, a system might not have any non-local
+   * non-multicast addresseses, not even internal ones.
+   * Unit tests shouldn't fail because of this. */
+  tt_int_op(smartlist_len(results),>=,0);
 
   tt_assert(!smartlist_contains_localhost_tor_addr(results));
   tt_assert(!smartlist_contains_multicast_tor_addr(results));
   /* The list may or may not contain internal addresses */
 
-  tt_assert(smartlist_contains_ipv4_tor_addr(results));
-  tt_assert(!smartlist_contains_ipv6_tor_addr(results));
+  /* Allow unit tests to pass on IPv6-only machines */
+  if (smartlist_len(results) > 0) {
+    tt_assert(smartlist_contains_ipv4_tor_addr(results)
+              || smartlist_contains_ipv6_tor_addr(results));
+  }
 
  done:
   free_interface_address_list(results);
@@ -721,7 +725,10 @@ test_address_get_if_addrs_list_no_internal(void *arg)
   tt_assert(!smartlist_contains_multicast_tor_addr(results));
   tt_assert(!smartlist_contains_internal_tor_addr(results));
 
-    /* The list may or may not contain IPv4 addresses */
+  /* if there are any addresses, they must be IPv4 */
+  if (smartlist_len(results) > 0) {
+    tt_assert(smartlist_contains_ipv4_tor_addr(results));
+  }
   tt_assert(!smartlist_contains_ipv6_tor_addr(results));
 
  done:
@@ -746,8 +753,11 @@ test_address_get_if_addrs6_list_internal(void *arg)
   tt_assert(!smartlist_contains_multicast_tor_addr(results));
   /* The list may or may not contain internal addresses */
 
+  /* if there are any addresses, they must be IPv6 */
   tt_assert(!smartlist_contains_ipv4_tor_addr(results));
-  /* The list may or may not contain IPv6 addresses */
+  if (smartlist_len(results) > 0) {
+    tt_assert(smartlist_contains_ipv6_tor_addr(results));
+  }
 
  done:
   free_interface_address6_list(results);
@@ -772,7 +782,9 @@ test_address_get_if_addrs6_list_no_internal(void *arg)
   tt_assert(!smartlist_contains_internal_tor_addr(results));
 
   tt_assert(!smartlist_contains_ipv4_tor_addr(results));
-  /* The list may or may not contain IPv6 addresses */
+  if (smartlist_len(results) > 0) {
+    tt_assert(smartlist_contains_ipv6_tor_addr(results));
+  }
 
  done:
   free_interface_address6_list(results);
@@ -883,16 +895,18 @@ test_address_get_if_addrs(void *arg)
 
   rv = get_interface_address(LOG_ERR, &addr_h);
 
-  /* Assume every system has at least 1 non-local non-multicast IPv4
-   * interface, even if it is an internal one */
-  tt_assert(rv == 0);
-  tor_addr_from_ipv4h(&tor_addr, addr_h);
+  /* When the network is down, a system might not have any non-local
+   * non-multicast IPv4 addresses, not even internal ones.
+   * Unit tests shouldn't fail because of this. */
+  if (rv == 0) {
+    tor_addr_from_ipv4h(&tor_addr, addr_h);
 
-  tt_assert(!tor_addr_is_loopback(&tor_addr));
-  tt_assert(!tor_addr_is_multicast(&tor_addr));
-  /* The address may or may not be an internal address */
+    tt_assert(!tor_addr_is_loopback(&tor_addr));
+    tt_assert(!tor_addr_is_multicast(&tor_addr));
+    /* The address may or may not be an internal address */
 
-  tt_assert(tor_addr_is_v4(&tor_addr));
+    tt_assert(tor_addr_is_v4(&tor_addr));
+  }
 
  done:
   return;





More information about the tor-commits mailing list