Browse Source

[1371] update comments (now as docstring) for rrsets_equal() to make them
more sense.

JINMEI Tatuya 13 years ago
parent
commit
15e23bca2c
1 changed files with 7 additions and 5 deletions
  1. 7 5
      src/lib/python/isc/testutils/rrset_utils.py

+ 7 - 5
src/lib/python/isc/testutils/rrset_utils.py

@@ -18,12 +18,14 @@
 from isc.dns import *
 
 def rrsets_equal(a, b):
-    '''Compare two RRsets, return True if equal, otherwise False'''
+    '''Compare two RRsets, return True if equal, otherwise False
 
-    # no accessor for sigs either (so this only checks name, class, type, ttl,
-    # and rdata)
-    # also, because of the fake data in rrsigs, if the type is rrsig, the
-    # rdata is not checked
+    We provide this function as part of test utils we have no direct rrset
+    comparison atm.  There's no accessor for sigs either (so this only checks
+    name, class, type, ttl, and rdata).
+    Also, since we often use fake data in RRSIGs, RRSIG RDATA are not checked.
+
+    '''
     return a.get_name() == b.get_name() and \
            a.get_class() == b.get_class() and \
            a.get_type() == b.get_type() and \