diff --git a/src/lib/dhcpsrv/tests/cb_ctl_dhcp_unittest.cc b/src/lib/dhcpsrv/tests/cb_ctl_dhcp_unittest.cc index ff0f8ef362178755b2494b72faafb2b2fa704569..118fe1bd4d934e530f38c680a15d6ac0520379bc 100644 --- a/src/lib/dhcpsrv/tests/cb_ctl_dhcp_unittest.cc +++ b/src/lib/dhcpsrv/tests/cb_ctl_dhcp_unittest.cc @@ -454,7 +454,7 @@ public: auto found_network = networks->getByName("one"); if (fetchConfigElement("dhcp4_shared_network") && (getTimestamp("dhcp4_shared_network") > lb_modification_time)) { - EXPECT_TRUE(found_network); + ASSERT_TRUE(found_network); EXPECT_TRUE(found_network->hasFetchGlobalsFn()); } else { diff --git a/src/lib/dhcpsrv/tests/cfg_hosts_unittest.cc b/src/lib/dhcpsrv/tests/cfg_hosts_unittest.cc index 97ce820176cf47bcc554a705e079ef1dd7f556d2..a27e8c711f83fcfbee413db2de75dec5ec635aad 100644 --- a/src/lib/dhcpsrv/tests/cfg_hosts_unittest.cc +++ b/src/lib/dhcpsrv/tests/cfg_hosts_unittest.cc @@ -41,6 +41,11 @@ public: /// 0 and 0x32. CfgHostsTest(); + /// @brief Destructor. + /// + /// This destructor resets global state after tests are run. + ~CfgHostsTest(); + /// @brief Increases last byte of an address. /// /// @param address Address to be increased. @@ -88,6 +93,10 @@ CfgHostsTest::CfgHostsTest() { } } +CfgHostsTest::~CfgHostsTest() { + CfgMgr::instance().setFamily(AF_INET); +} + IOAddress CfgHostsTest::increase(const IOAddress& address, const uint8_t num) const { std::vector vec = address.toBytes(); diff --git a/src/lib/dhcpsrv/tests/cfgmgr_unittest.cc b/src/lib/dhcpsrv/tests/cfgmgr_unittest.cc index 408168e071b351ed6a45ade463269c564210cea7..1069de2365dce034cec61cfc8d99debe5068f566 100644 --- a/src/lib/dhcpsrv/tests/cfgmgr_unittest.cc +++ b/src/lib/dhcpsrv/tests/cfgmgr_unittest.cc @@ -424,6 +424,9 @@ TEST_F(CfgMgrTest, d2ClientConfig) { // and not the original configuration. EXPECT_EQ(*new_cfg, *updated_config); EXPECT_NE(*original_config, *updated_config); + + // Revert to default configuration. + ASSERT_NO_THROW(CfgMgr::instance().setD2ClientConfig(original_config)); } // This test verifies that the configuration staging, commit and rollback works