BIND 10 trac2268, updated. 66524e1594263bb19fd73660fef31f4e49dbe81d [2268] use textToRRset instead of building RRset manually

BIND 10 source code commits bind10-changes at lists.isc.org
Mon Oct 15 16:50:34 UTC 2012


The branch, trac2268 has been updated
       via  66524e1594263bb19fd73660fef31f4e49dbe81d (commit)
      from  0e2c14089b820cbb0bdef6e813042992896375bc (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 66524e1594263bb19fd73660fef31f4e49dbe81d
Author: JINMEI Tatuya <jinmei at isc.org>
Date:   Mon Oct 15 09:50:00 2012 -0700

    [2268] use textToRRset instead of building RRset manually
    
    this will make the test code a bit more concise.

-----------------------------------------------------------------------

Summary of changes:
 .../datasrc/tests/memory/memory_client_unittest.cc |   44 ++++++++------------
 1 file changed, 17 insertions(+), 27 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/tests/memory/memory_client_unittest.cc b/src/lib/datasrc/tests/memory/memory_client_unittest.cc
index 9546653..8785a10 100644
--- a/src/lib/datasrc/tests/memory/memory_client_unittest.cc
+++ b/src/lib/datasrc/tests/memory/memory_client_unittest.cc
@@ -582,22 +582,17 @@ TEST_F(MemoryClientTest, loadRRSIGs) {
 TEST_F(MemoryClientTest, loadRRSIGsRdataMixedCoveredTypes) {
     vector<ConstRRsetPtr> rrsets_vec;
 
-    RRsetPtr rrset(new RRset(Name("example.org"),
-                             zclass_, RRType::SOA(), RRTTL(3600)));
-    rrset->addRdata(generic::SOA(Name("ns1.example.org"),
-                                 Name("bugs.x.w.example.org"),
-                                 2010012601, 3600, 300, 3600000, 1200));
-    rrsets_vec.push_back(rrset);
-
-    rrset.reset(new RRset(Name("example.org"),
-                          zclass_, RRType::A(), RRTTL(3600)));
-    rrset->addRdata(in::A("192.0.2.1"));
-    rrset->addRdata(in::A("192.0.2.2"));
-
-    RRsetPtr rrsig(new RRset(Name("example.org"), zclass_,
-                             RRType::RRSIG(), RRTTL(300)));
-    rrsig->addRdata(generic::RRSIG("A 5 3 3600 20000101000000 20000201000000 "
-                                   "12345 example.org. FAKEFAKEFAKE"));
+    rrsets_vec.push_back(textToRRset("example.org. 3600 IN SOA "
+                                     "ns1.example.org. bugs.x.w.example.org. "
+                                     "2010012601 3600 300 3600000 1200",
+                                     zclass_, Name("example.org")));
+    RRsetPtr rrset(textToRRset("example.org. 3600 IN A 192.0.2.1\n"
+                               "example.org. 3600 IN A 192.0.2.2\n"));
+    RRsetPtr rrsig(textToRRset("example.org. 300 IN RRSIG "
+                               "A 5 3 3600 20000101000000 20000201000000 "
+                               "12345 example.org. FAKEFAKEFAKE"));
+    // textToRRset (correctly) consider this RDATA belongs to a different
+    // RRSIG, so we need to manually add it.
     rrsig->addRdata(generic::RRSIG("NS 5 3 3600 20000101000000 20000201000000 "
                                    "54321 example.org. FAKEFAKEFAKEFAKE"));
     rrset->addRRsig(rrsig);
@@ -702,17 +697,12 @@ TEST_F(MemoryClientTest, getIteratorGetSOAThrowsNotImplemented) {
 
 TEST_F(MemoryClientTest, addEmptyRRsetThrows) {
     vector<ConstRRsetPtr> rrsets_vec;
-
-    RRsetPtr rrset(new RRset(Name("example.org"),
-                             zclass_, RRType::SOA(), RRTTL(3600)));
-    rrset->addRdata(generic::SOA(Name("ns1.example.org"),
-                                 Name("bugs.x.w.example.org"),
-                                 2010012601, 3600, 300, 3600000, 1200));
-    rrsets_vec.push_back(rrset);
-
-    rrset.reset(new RRset(Name("example.org"),
-                          zclass_, RRType::A(), RRTTL(3600)));
-    rrsets_vec.push_back(rrset);
+    rrsets_vec.push_back(textToRRset("example.org. 3600 IN SOA "
+                                     "ns1.example.org. bugs.x.w.example.org. "
+                                     "2010012601 3600 300 3600000 1200",
+                                     zclass_, Name("example.org")));
+    rrsets_vec.push_back(RRsetPtr(new RRset(Name("example.org"), zclass_,
+                                            RRType::A(), RRTTL(3600))));
 
     EXPECT_THROW(
         client_->load(Name("example.org"),



More information about the bind10-changes mailing list