Browse Source

[4286] Updated CfgOptionDefTest.overrideStdOptionDef after option 65 added.

Tomek Mrugalski 9 years ago
parent
commit
b4dffd05e1
1 changed files with 4 additions and 6 deletions
  1. 4 6
      src/lib/dhcpsrv/tests/cfg_option_def_unittest.cc

+ 4 - 6
src/lib/dhcpsrv/tests/cfg_option_def_unittest.cc

@@ -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