commit 5a45495e3eae85f4bddf5b83b29433a6aaa8cf70
author alvy <muktadir.rahman.c@gmail.com> Mon Dec 15 12:49:54 2014 -0600
committer Vince Lehman <vslehman@memphis.edu> Wed Jan 21 08:53:46 2015 -0800
tree 9601d4af860e30a7b06d9d34893caf1bb7d122b8
parent c11cc20948cb6a2d045c025d639008324e7353ea

src: ConfParameter variables should be consistent with their min and max values

refs: #2084

Change-Id: Iae71da80ae47fca44ec46f4a3f0d4d86c77e5094

tests/test-conf-file-processor.cpp

diff --git a/tests/test-conf-file-processor.cpp b/tests/test-conf-file-processor.cpp
index a86a965..817f149 100644
--- a/tests/test-conf-file-processor.cpp
+++ b/tests/test-conf-file-processor.cpp
@@ -382,7 +382,7 @@
 
   ConfParameter& conf = nlsr.getConfParameter();
 
-  BOOST_CHECK_EQUAL(conf.getLsaRefreshTime(), static_cast<int32_t>(LSA_REFRESH_TIME_DEFAULT));
+  BOOST_CHECK_EQUAL(conf.getLsaRefreshTime(), static_cast<uint32_t>(LSA_REFRESH_TIME_DEFAULT));
   BOOST_CHECK_EQUAL(conf.getLsaInterestLifetime(),
                     static_cast<ndn::time::seconds>(LSA_INTEREST_LIFETIME_DEFAULT));
   BOOST_CHECK_EQUAL(conf.getRouterDeadInterval(), (2*conf.getLsaRefreshTime()));
@@ -403,8 +403,8 @@
   ConfParameter& conf = nlsr.getConfParameter();
 
   BOOST_CHECK_EQUAL(conf.getInterestRetryNumber(), static_cast<uint32_t>(HELLO_RETRIES_DEFAULT));
-  BOOST_CHECK_EQUAL(conf.getInterestResendTime(), static_cast<int32_t>(HELLO_TIMEOUT_DEFAULT));
-  BOOST_CHECK_EQUAL(conf.getInfoInterestInterval(), static_cast<int32_t>(HELLO_INTERVAL_DEFAULT));
+  BOOST_CHECK_EQUAL(conf.getInterestResendTime(), static_cast<uint32_t>(HELLO_TIMEOUT_DEFAULT));
+  BOOST_CHECK_EQUAL(conf.getInfoInterestInterval(), static_cast<uint32_t>(HELLO_INTERVAL_DEFAULT));
   BOOST_CHECK_EQUAL(conf.getFirstHelloInterval(),
                     static_cast<uint32_t>(FIRST_HELLO_INTERVAL_DEFAULT));
   BOOST_CHECK_EQUAL(conf.getAdjLsaBuildInterval(),
@@ -454,6 +454,23 @@
   BOOST_CHECK_EQUAL(processConfigurationString(SECTION_FIB_OUT_OF_RANGE), false);
 }
 
+BOOST_AUTO_TEST_CASE(NegativeValue)
+{
+  const std::string SECTION_GENERAL_NEGATIVE_VALUE =
+  "general\n"
+  "{\n"
+  "  network /ndn/\n"
+  "  site /memphis.edu/\n"
+  "  router /cs/pollux/\n"
+  "  lsa-refresh-time -1800\n"
+  "  lsa-interest-lifetime -3\n"
+  "  router-dead-interval -86400\n"
+  "}\n\n";
+
+  // Processing should fail due to negative value
+  BOOST_CHECK_EQUAL(processConfigurationString(SECTION_GENERAL_NEGATIVE_VALUE), false);
+}
+
 BOOST_AUTO_TEST_SUITE_END()
 
 } //namespace test