Parcourir la source

[1791] missing regression fix: needed to adjust python datasrc tests.

this is a result of the "unrelated" bug fix at commit b446618.
I also clarified the magic number of expected RRsets in comments.
JINMEI Tatuya il y a 13 ans
Parent
commit
9858d6dd9a
1 fichiers modifiés avec 3 ajouts et 3 suppressions
  1. 3 3
      src/lib/python/isc/datasrc/tests/datasrc_test.py

+ 3 - 3
src/lib/python/isc/datasrc/tests/datasrc_test.py

@@ -262,16 +262,16 @@ class DataSrcClient(unittest.TestCase):
         rrets = dsc.get_iterator(isc.dns.Name("example.com"))
         # there are more than 80 RRs in this zone... let's just count them
         # (already did a full check of the smaller zone above)
-        self.assertEqual(55, len(list(rrets)))
+        # There are 40 non-RRSIG RRsets and 32 dinstinct RRSIGs.
+        self.assertEqual(72, len(list(rrets)))
 
         # same test, but now with explicit False argument for separate_rrs
         dsc = isc.datasrc.DataSourceClient("sqlite3", READ_ZONE_DB_CONFIG)
         rrets = dsc.get_iterator(isc.dns.Name("example.com"), False)
         # there are more than 80 RRs in this zone... let's just count them
         # (already did a full check of the smaller zone above)
-        self.assertEqual(55, len(list(rrets)))
+        self.assertEqual(72, len(list(rrets)))
 
-        # Count should be 71 if we request individual rrsets for differing ttls
         dsc = isc.datasrc.DataSourceClient("sqlite3", READ_ZONE_DB_CONFIG)
         rrets = dsc.get_iterator(isc.dns.Name("example.com"), True)
         # there are more than 80 RRs in this zone... let's just count them