Browse Source

[2436] More renaming from validation to check

For consistency.
Michal 'vorner' Vaner 12 years ago
parent
commit
30bb4f426c

+ 2 - 2
src/lib/datasrc/tests/zone_loader_unittest.cc

@@ -422,7 +422,7 @@ TEST_F(ZoneLoaderTest, loadSyntaxError) {
 }
 
 // Test there's validation of the data in the zone loader.
-TEST_F(ZoneLoaderTest, loadValidation) {
+TEST_F(ZoneLoaderTest, loadCheck) {
     ZoneLoader loader(destination_client_, Name::ROOT_NAME(),
                       TEST_DATA_DIR "/novalidate.zone");
     EXPECT_THROW(loader.loadIncremental(10), ZoneContentError);
@@ -431,7 +431,7 @@ TEST_F(ZoneLoaderTest, loadValidation) {
 }
 
 // The same test, but for copying from other data source
-TEST_F(ZoneLoaderTest, copyValidation) {
+TEST_F(ZoneLoaderTest, copyCheck) {
     prepareSource(Name::ROOT_NAME(), "novalidate.zone");
     ZoneLoader loader(destination_client_, Name::ROOT_NAME(), source_client_);
 

+ 1 - 1
src/lib/datasrc/zone_loader.cc

@@ -154,7 +154,7 @@ ZoneLoader::loadIncremental(size_t limit) {
             callbacks(boost::bind(&logError, &zone_name, &zone_class, _1),
                       boost::bind(&logWarning, &zone_name, &zone_class, _1));
         if (!dns::checkZone(zone_name, zone_class, collection, callbacks)) {
-            // Validation failed.
+            // The post-load check failed.
             loaded_ok_ = false;
             isc_throw(ZoneContentError, "Errors found when validating zone " <<
                       zone_name << "/" << zone_class);

+ 3 - 2
src/lib/datasrc/zone_loader.h

@@ -48,7 +48,7 @@ public:
     {}
 };
 
-/// \brief Exception thrown when the zone doesn't validate.
+/// \brief Exception thrown when the zone doesn't pass post-load check.
 ///
 /// This is thrown by the ZoneLoader when the zone is loaded, but it
 /// doesn't pass basic sanity checks.
@@ -138,7 +138,8 @@ public:
     /// After the last RR is loaded, a sanity check of the zone is performed by
     /// isc::dns::validateZone. It reports errors and warnings by logging them
     /// directly. If there are any errors, a ZoneContentError exception is
-    /// thrown.
+    /// thrown and the load is aborted (preserving the old version of zone, if
+    /// any).
     ///
     /// \param limit The maximum allowed number of RRs to be loaded during this
     ///     call.