Browse Source

[3427] Changes after review:

 - variables, test renamed in daemon unit-tests
 - verbose parameter documented in Daemon class
Tomek Mrugalski 10 years ago
parent
commit
3aac5ac21c
2 changed files with 6 additions and 4 deletions
  1. 2 0
      src/lib/dhcpsrv/daemon.h
  2. 4 4
      src/lib/dhcpsrv/tests/daemon_unittest.cc

+ 2 - 0
src/lib/dhcpsrv/daemon.h

@@ -138,6 +138,8 @@ public:
     /// Verbose mode (-v in command-line) triggers loggers to log everythin
     /// (sets severity to DEBUG and debuglevel to 99). Values specified in the
     /// config file are ignored.
+    ///
+    /// @param verbose specifies if verbose should be set or not
     void setVerbose(bool verbose) {
         verbose_ = verbose;
     }

+ 4 - 4
src/lib/dhcpsrv/tests/daemon_unittest.cc

@@ -27,12 +27,12 @@ namespace {
 
 // Very simple test. Checks whether Daemon can be instantiated and its
 // default parameters are sane
-TEST(DaemonTest, noop) {
-    EXPECT_NO_THROW(Daemon x);
+TEST(DaemonTest, constructor) {
+    EXPECT_NO_THROW(Daemon instance1);
 
     // Check that the verbose mode is not set by default.
-    Daemon y;
-    EXPECT_FALSE(y.getVerbose());
+    Daemon instance2;
+    EXPECT_FALSE(instance2.getVerbose());
 }
 
 // Checks that configureLogger method is behaving properly.