[5039] spelling (and a ~ file)

diff --git a/src/bin/dhcp4/simple_parser4.cc b/src/bin/dhcp4/simple_parser4.cc
index 1073cd2b9b6d29b84081d202a2261ab8befc9cbf..b1c36bfbe0e1ecdb967ecef715bbb9faab5a4810 100644 (file)
--- a/src/bin/dhcp4/simple_parser4.cc
+++ b/src/bin/dhcp4/simple_parser4.cc
@@ -21,7 +21,7 @@ namespace dhcp {
 /// is expected to be one centralized place to look at for
 /// the default values. This is expected to be looked at also by
 /// people who are not skilled in C or C++, so they may be
-/// confused with the differences between declaration and defintion.
+/// confused with the differences between declaration and definition.
 /// As such, there's one file to look at that hopefully is readable
 /// without any C or C++ skills.
 ///
@@ -83,7 +83,7 @@ size_t SimpleParser4::setAllDefaults(isc::data::ElementPtr global) {
     // Set global defaults first.
     cnt = setDefaults(global, GLOBAL4_DEFAULTS);
 
-    // Now set option defintion defaults for each specified option definition
+    // Now set option definition defaults for each specified option definition
     ConstElementPtr option_defs = global->get("option-def");
     if (option_defs) {
         BOOST_FOREACH(ElementPtr option_def, option_defs->listValue()) {

diff --git a/src/bin/dhcp4/tests/simple_parser4_unittest.cc b/src/bin/dhcp4/tests/simple_parser4_unittest.cc
index 9d02c049fddacd66611bdc8b981a2e6abce71d7b..d95e45fb3dc92e17e3d73eb71586e113d978051f 100644 (file)
--- a/src/bin/dhcp4/tests/simple_parser4_unittest.cc
+++ b/src/bin/dhcp4/tests/simple_parser4_unittest.cc
@@ -79,7 +79,7 @@ TEST_F(SimpleParser4Test, inheritGlobalToSubnet4) {
                           SimpleParser4::INHERIT_GLOBAL_TO_SUBNET4));
     EXPECT_EQ(2, num);
 
-    // Check the values. 2 of them are interited, while the third one
+    // Check the values. 2 of them are inherited, while the third one
     // was already defined in the subnet, so should not be inherited.
     checkIntegerValue(subnet, "renew-timer", 100);
     checkIntegerValue(subnet, "rebind-timer", 2);

diff --git a/src/bin/dhcp6/simple_parser6.cc b/src/bin/dhcp6/simple_parser6.cc
index d684177dd54c4b0ce1b51107635d8a582a685c8a..aa2844d93994da47a908d2f736aa14d36b38a2b3 100644 (file)
--- a/src/bin/dhcp6/simple_parser6.cc
+++ b/src/bin/dhcp6/simple_parser6.cc
@@ -21,7 +21,7 @@ namespace dhcp {
 /// is expected to be one centralized place to look at for
 /// the default values. This is expected to be looked at also by
 /// people who are not skilled in C or C++, so they may be
-/// confused with the differences between declaration and defintion.
+/// confused with the differences between declaration and definition.
 /// As such, there's one file to look at that hopefully is readable
 /// without any C or C++ skills.
 ///

diff --git a/src/bin/dhcp6/tests/simple_parser6_unittest.cc b/src/bin/dhcp6/tests/simple_parser6_unittest.cc
index e5aca51151bc1b12d8ff5f26a9f8dcc7c4aef0d1..3721a954c1cd429689c6658887f111adf80e73ec 100644 (file)
--- a/src/bin/dhcp6/tests/simple_parser6_unittest.cc
+++ b/src/bin/dhcp6/tests/simple_parser6_unittest.cc
@@ -75,7 +75,7 @@ TEST_F(SimpleParser6Test, inheritGlobalToSubnet6) {
                                                      SimpleParser6::INHERIT_GLOBAL_TO_SUBNET6));
     EXPECT_EQ(3, num);
 
-    // Check the values. 3 of them are interited, while the fourth one
+    // Check the values. 3 of them are inherited, while the fourth one
     // was already defined in the subnet, so should not be inherited.
     checkIntegerValue(subnet, "renew-timer", 100);
     checkIntegerValue(subnet, "rebind-timer", 2);

diff --git a/src/bin/dhcp6/tests/simple_parser6_unittest.cc~ b/src/bin/dhcp6/tests/simple_parser6_unittest.cc~
deleted file mode 100644 (file)
index fb7dfc2..0000000
--- a/src/bin/dhcp6/tests/simple_parser6_unittest.cc~
+++ /dev/null
@@ -1,79 +0,0 @@
-#include <config.h>
-#include <cc/data.h>
-#include <gtest/gtest.h>
-
-using namespace isc;
-using namespace isc::data;
-
-namespace {
-
-/// @brief DHCP Parser test fixture class
-class SimpleParser6Test : public ::testing::Test {
-
-    /// @brief Checks if specified map has an integer parameter with expected value
-    ///
-    /// @param map map to be checked
-    /// @param param_name name of the parameter to be checked
-    /// @param exp_value expected value of the parameter.
-    void checkIntegerValue(const ConstElementPtr& map, const std::string& param_name,
-                           int64_t exp_value) {
-
-        // First check if the passed element is a map.
-        ASSERT_EQ(Element::map, map->getType());
-
-        // Now try to get the element being checked
-        ConstElementPtr elem = map->get(param_name);
-        ASSERT_TRUE(elem);
-
-        // Now check if it's indeed integer
-        ASSERT_EQ(Element::integer, elem->getType());
-
-        // Finally, check if its value meets expectation.
-        EXPECT_EQ(exp_value, elem->intValue());
-    }
-};
-
-// This test checks if global defaults are properly set for DHCPv6.
-TEST_F(SimpleParser6Test, globalDefaults6) {
-
-    ElementPtr empty = Element::fromJSON("{ }");
-    size_t num;
-
-    EXPECT_NO_THROW(num = SimpleParser6::setDefaults(empty));
-
-    // We expect at least 4 parameters to be inserted.
-    EXPECT_TRUE(num >= 4);
-
-    checkIntegerValue(empty, "valid-lifetime", 7200);
-    checkIntegerValue(empty, "preferred-lifetime", 3600);
-    checkIntegerValue(empty, "rebind-timer", 1800);
-    checkIntegerValue(empty, "renew-timer", 900);
-}
-
-// This test checks if the parameters can be inherited from the global
-// scope to the subnet scope.
-TEST_F(SimpleParser6Test, inheritGlobalToSubnet6) {
-    ElementPtr global = Element::fromJSON("{ \"renew-timer\": 1,"
-                                          "  \"rebind-timer\": 2,"
-                                          "  \"preferred-lifetime\": 3,"
-                                          "  \"valid-lifetime\": 4"
-                                          "}");
-    ElementPtr subnet = Element::fromJSON("{ \"renew-timer\": 100 }");
-
-    // we should inherit 3 parameters. Renew-timer should remain intact,
-    // as it was already defined in the subnet scope.
-    size_t num;
-    EXPECT_NO_THROW(num = SimpleParser::deriveParams(global, subnet,
-                                                     SimpleParser6::INHERIT_GLOBAL_TO_SUBNET6));
-    EXPECT_EQ(3, num);
-
-    // Check the values. 3 of them are interited, while the fourth one
-    // was already defined in the subnet, so should not be inherited.
-    checkIntegerValue(subnet, "renew-timer", 100);
-    checkIntegerValue(subnet, "rebind-timer", 2);
-    checkIntegerValue(subnet, "preferred-lifetime", 3);
-    checkIntegerValue(subnet, "valid-lifetime", 4);
-}
-
-};
-