Skip to content

Commit 0175977

Browse files
committed
Add I2P network SetReachable/IsReachable unit test assertions
and simplify similar neighboring assertions.
1 parent b87a9c4 commit 0175977

File tree

1 file changed

+24
-19
lines changed

1 file changed

+24
-19
lines changed

src/test/net_tests.cpp

Lines changed: 24 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -764,37 +764,42 @@ BOOST_AUTO_TEST_CASE(ipv4_peer_with_ipv6_addrMe_test)
764764

765765
BOOST_AUTO_TEST_CASE(LimitedAndReachable_Network)
766766
{
767-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV4), true);
768-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV6), true);
769-
BOOST_CHECK_EQUAL(IsReachable(NET_ONION), true);
767+
BOOST_CHECK(IsReachable(NET_IPV4));
768+
BOOST_CHECK(IsReachable(NET_IPV6));
769+
BOOST_CHECK(IsReachable(NET_ONION));
770+
BOOST_CHECK(IsReachable(NET_I2P));
770771

771772
SetReachable(NET_IPV4, false);
772773
SetReachable(NET_IPV6, false);
773774
SetReachable(NET_ONION, false);
775+
SetReachable(NET_I2P, false);
774776

775-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV4), false);
776-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV6), false);
777-
BOOST_CHECK_EQUAL(IsReachable(NET_ONION), false);
777+
BOOST_CHECK(!IsReachable(NET_IPV4));
778+
BOOST_CHECK(!IsReachable(NET_IPV6));
779+
BOOST_CHECK(!IsReachable(NET_ONION));
780+
BOOST_CHECK(!IsReachable(NET_I2P));
778781

779782
SetReachable(NET_IPV4, true);
780783
SetReachable(NET_IPV6, true);
781784
SetReachable(NET_ONION, true);
785+
SetReachable(NET_I2P, true);
782786

783-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV4), true);
784-
BOOST_CHECK_EQUAL(IsReachable(NET_IPV6), true);
785-
BOOST_CHECK_EQUAL(IsReachable(NET_ONION), true);
787+
BOOST_CHECK(IsReachable(NET_IPV4));
788+
BOOST_CHECK(IsReachable(NET_IPV6));
789+
BOOST_CHECK(IsReachable(NET_ONION));
790+
BOOST_CHECK(IsReachable(NET_I2P));
786791
}
787792

788793
BOOST_AUTO_TEST_CASE(LimitedAndReachable_NetworkCaseUnroutableAndInternal)
789794
{
790-
BOOST_CHECK_EQUAL(IsReachable(NET_UNROUTABLE), true);
791-
BOOST_CHECK_EQUAL(IsReachable(NET_INTERNAL), true);
795+
BOOST_CHECK(IsReachable(NET_UNROUTABLE));
796+
BOOST_CHECK(IsReachable(NET_INTERNAL));
792797

793798
SetReachable(NET_UNROUTABLE, false);
794799
SetReachable(NET_INTERNAL, false);
795800

796-
BOOST_CHECK_EQUAL(IsReachable(NET_UNROUTABLE), true); // Ignored for both networks
797-
BOOST_CHECK_EQUAL(IsReachable(NET_INTERNAL), true);
801+
BOOST_CHECK(IsReachable(NET_UNROUTABLE)); // Ignored for both networks
802+
BOOST_CHECK(IsReachable(NET_INTERNAL));
798803
}
799804

800805
CNetAddr UtilBuildAddress(unsigned char p1, unsigned char p2, unsigned char p3, unsigned char p4)
@@ -813,10 +818,10 @@ BOOST_AUTO_TEST_CASE(LimitedAndReachable_CNetAddr)
813818
CNetAddr addr = UtilBuildAddress(0x001, 0x001, 0x001, 0x001); // 1.1.1.1
814819

815820
SetReachable(NET_IPV4, true);
816-
BOOST_CHECK_EQUAL(IsReachable(addr), true);
821+
BOOST_CHECK(IsReachable(addr));
817822

818823
SetReachable(NET_IPV4, false);
819-
BOOST_CHECK_EQUAL(IsReachable(addr), false);
824+
BOOST_CHECK(!IsReachable(addr));
820825

821826
SetReachable(NET_IPV4, true); // have to reset this, because this is stateful.
822827
}
@@ -828,12 +833,12 @@ BOOST_AUTO_TEST_CASE(LocalAddress_BasicLifecycle)
828833

829834
SetReachable(NET_IPV4, true);
830835

831-
BOOST_CHECK_EQUAL(IsLocal(addr), false);
832-
BOOST_CHECK_EQUAL(AddLocal(addr, 1000), true);
833-
BOOST_CHECK_EQUAL(IsLocal(addr), true);
836+
BOOST_CHECK(!IsLocal(addr));
837+
BOOST_CHECK(AddLocal(addr, 1000));
838+
BOOST_CHECK(IsLocal(addr));
834839

835840
RemoveLocal(addr);
836-
BOOST_CHECK_EQUAL(IsLocal(addr), false);
841+
BOOST_CHECK(!IsLocal(addr));
837842
}
838843

839844
BOOST_AUTO_TEST_SUITE_END()

0 commit comments

Comments
 (0)