]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[4286] Updated CfgOptionDefTest.overrideStdOptionDef after option 65 added.
authorTomek Mrugalski <tomasz@isc.org>
Wed, 24 Feb 2016 14:34:40 +0000 (15:34 +0100)
committerTomek Mrugalski <tomasz@isc.org>
Wed, 24 Feb 2016 14:34:40 +0000 (15:34 +0100)
src/lib/dhcpsrv/tests/cfg_option_def_unittest.cc

index 608d7fada03c22459b55aaaab046eaab442a5307..773a8f0e9426dbd7ce2cb00bfc7f0c186423c36a 100644 (file)
@@ -210,12 +210,10 @@ TEST(CfgOptionDefTest, overrideStdOptionDef) {
     def.reset(new OptionDefinition("routers", DHO_ROUTERS, "uint32"));
     EXPECT_THROW(cfg.add(def, DHCP4_OPTION_SPACE), isc::BadValue);
 
-    /// @todo There is no definition for the NIS Server Addr option in
-    /// libdhcp++. Once it is implemented it should be not allowed to
-    /// add a custom definition for it. At the moment, it should be ok
-    /// to add a definition for this option (using configuration mechanism)
-    /// because we haven't implemented the one in libdhcp++.
-    def.reset(new OptionDefinition("nis-server-addr", 65, "uint16"));
+    /// There is no definition for the Access Network Domain Name Option
+    /// (RFC5986, option code 213) in libdhcp++. Once it is implemented it
+    /// should be not allowed to add a custom definition for it.
+    def.reset(new OptionDefinition("access-netwokr-domain-name", 213, "string"));
     EXPECT_NO_THROW(cfg.add(def, DHCP4_OPTION_SPACE));
 
     // It is not allowed to override the definition of the option which