From: JINMEI Tatuya <jinmei@isc.org>
Date: Wed, 25 Jan 2012 08:39:58 +0000 (-0800)
Subject: [1571] added a couple of new tests to confirm additional cases regarding DS.
X-Git-Tag: trac2351_base~278^2~10^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=c6c8a84bbbd9faf4275e4889f2035d4a2861ddc7;p=thirdparty%2Fkea.git

[1571] added a couple of new tests to confirm additional cases regarding DS.
(they should have succeeded in the original version of implementation)
---

diff --git a/src/lib/datasrc/tests/memory_datasrc_unittest.cc b/src/lib/datasrc/tests/memory_datasrc_unittest.cc
index 54670595bc..2fdc4521e4 100644
--- a/src/lib/datasrc/tests/memory_datasrc_unittest.cc
+++ b/src/lib/datasrc/tests/memory_datasrc_unittest.cc
@@ -666,8 +666,17 @@ TEST_F(InMemoryZoneFinderTest, delegationWithDS) {
     EXPECT_EQ(SUCCESS, zone_finder_.add(rr_child_ns_));
     EXPECT_EQ(SUCCESS, zone_finder_.add(rr_child_ds_));
 
+    // Normal types of query should result in delegation, but DS query
+    // should be considered in-zone.
+    findTest(Name("child.example.org"), RRType::A(), ZoneFinder::DELEGATION,
+             true, rr_child_ns_);
     findTest(Name("child.example.org"), RRType::DS(), ZoneFinder::SUCCESS,
              true, rr_child_ds_);
+
+    // There's nothing special for DS query at the zone apex.  It would
+    // normally result in NXRRSET.
+    findTest(Name("example.org"), RRType::DS(), ZoneFinder::NXRRSET,
+             true, ConstRRsetPtr());
 }
 
 TEST_F(InMemoryZoneFinderTest, findAny) {