BIND 10 trac1314, updated. 697341a7b1e73b508d8142b8afc07767ea46f3f3 [1314] address review comments

BIND 10 source code commits bind10-changes at lists.isc.org
Thu Dec 8 16:40:41 UTC 2011


The branch, trac1314 has been updated
       via  697341a7b1e73b508d8142b8afc07767ea46f3f3 (commit)
      from  7515af0e50af796b0b936e9a966eea5bff82dfe9 (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 697341a7b1e73b508d8142b8afc07767ea46f3f3
Author: Jelte Jansen <jelte at isc.org>
Date:   Thu Dec 8 17:40:05 2011 +0100

    [1314] address review comments
    
    - changed udp tests to tcp tests where possible and enabled them
    - added some comments about why

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

Summary of changes:
 tests/lettuce/features/ixfr_out_bind10.feature |  107 +++++++++++++++---------
 1 files changed, 67 insertions(+), 40 deletions(-)

-----------------------------------------------------------------------
diff --git a/tests/lettuce/features/ixfr_out_bind10.feature b/tests/lettuce/features/ixfr_out_bind10.feature
index 07e0d1a..e84ad8c 100644
--- a/tests/lettuce/features/ixfr_out_bind10.feature
+++ b/tests/lettuce/features/ixfr_out_bind10.feature
@@ -5,9 +5,13 @@ Feature: IXFR out
 
     # A lot of these tests test specific UDP behaviour.
     #
-    # Since we do not support that at this time, these are commented out
-    # Once we implement it, we can uncomment them. The tests themselves
-    # should already work.
+    # Where possible, we use the TCP equivalent. Some of the behaviour
+    # tested is UDP-specific though. In either case, a comment above
+    # the test shows how and why it differs from the test specification,
+    # or why it is commented out for now.
+    # When we do implement UDP IXFR, we should probably keep the TCP
+    # tests, and add them to the test specification, so we still have a
+    # 1-to-1 mapping between these tests and the specification document.
     #
     # These tests use a zone with just a few records, the first serial
     # is 2, and it is incremented in steps of 2, up to serial 22.
@@ -33,6 +37,9 @@ Feature: IXFR out
         #
         # Test 1
         #
+        # We don't support UDP yet, and for TCP we currently return full zone,
+        # so this test is currently skipped
+        #
         #When I do an IXFR transfer of example.com 123 over udp
         #The transfer result should have 1 RRs
         #The full result of the last transfer should be
@@ -43,47 +50,58 @@ Feature: IXFR out
         #
         # Test 2
         #
-        #When I do an IXFR transfer of example.com 123 over udp
-        #The transfer result should have 1 RRs
-        #The full result of the last transfer should be
-        #"""
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #"""
+        # Original test specification was for UDP, using TCP for now
+        #
+        #When I do an IXFR transfer of example.com 22 over udp
+        When I do an IXFR transfer of example.com 22 over tcp
+        The transfer result should have 1 RRs
+        The full result of the last transfer should be
+        """
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        """
 
         #
         # Test 3
         #
+        # Original test specification was for UDP, using TCP for now
+        #
         #When I do an IXFR transfer of example.com 20 over udp
-        #The transfer result should have 5 RRs
-        #The full result of the last transfer should be
-        #"""
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #www.example.com. 3600 IN A 192.0.2.1
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #"""
+        When I do an IXFR transfer of example.com 20 over tcp
+        The transfer result should have 5 RRs
+        The full result of the last transfer should be
+        """
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        www.example.com. 3600 IN A 192.0.2.1
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        """
 
         #
         # Test 4
         #
+        # Original test specification was for UDP, using TCP for now
+        #
         #When I do an IXFR transfer of example.com 18 over udp
-        #The transfer result should have 8 RRs
-        #The full result of the last transfer should be
-        #"""
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 18 28800 7200 604800 18000
-        #www.example.com. 3600 IN A 192.0.2.1
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #www.example.com. 3600 IN A 192.0.2.1
-        #example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #"""
+        When I do an IXFR transfer of example.com 18 over tcp
+        The transfer result should have 8 RRs
+        The full result of the last transfer should be
+        """
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 18 28800 7200 604800 18000
+        www.example.com. 3600 IN A 192.0.2.1
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 20 28800 7200 604800 18000
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        www.example.com. 3600 IN A 192.0.2.1
+        example.com. 3600 IN SOA ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        """
 
         #
         # Test 5
         #
+        # This test does not have a TCP equivalent, so it is skipped.
+        #
         #When I do an IXFR transfer of example.com 2 over udp
         #The transfer result should have 1 RRs
         #The full result of the last transfer should be
@@ -94,6 +112,8 @@ Feature: IXFR out
         #
         # Test 6
         #
+        # This test does not have a TCP equivalent, so it is skipped.
+        #
         #When I do an IXFR transfer of example.com 5 over udp
         #The transfer result should have 1 RRs
         #The full result of the last transfer should be
@@ -132,20 +152,25 @@ Feature: IXFR out
         #
         # Test 1
         #
+        # Original test specification was for UDP, using TCP for now
+        #
         #When I do an IXFR transfer of example.com 19 over udp
-        #The transfer result should have 5 RRs
-        #The full result of the last transfer should be
-        #"""
-        #example.com.            3600    IN      SOA     ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #example.com.            3600    IN      NS      ns.example.com.
-        #ns.example.com.         3600    IN      A       192.0.2.1
-        #www.example.com.        3600    IN      A       192.0.2.1
-        #example.com.            3600    IN      SOA     ns.example.com. admin.example.com. 22 28800 7200 604800 18000
-        #"""
+        When I do an IXFR transfer of example.com 19 over tcp
+        The transfer result should have 5 RRs
+        The full result of the last transfer should be
+        """
+        example.com.            3600    IN      SOA     ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        example.com.            3600    IN      NS      ns.example.com.
+        ns.example.com.         3600    IN      A       192.0.2.1
+        www.example.com.        3600    IN      A       192.0.2.1
+        example.com.            3600    IN      SOA     ns.example.com. admin.example.com. 22 28800 7200 604800 18000
+        """
 
         #
         # Test 2
         #
+        # This test has no TCP equivalent
+        #
         #When I do an IXFR transfer of example.com 6 over udp
         #The transfer result should have 5 RRs
         #The full result of the last transfer should be
@@ -160,7 +185,9 @@ Feature: IXFR out
         #
         # Test 3
         #
-        #When I do an IXFR transfer of example.com 19 over udp
+        # This test has no TCP equivalent
+        #
+        #When I do an IXFR transfer of example.com 2 over udp
         #The transfer result should have 1 RRs
         #The full result of the last transfer should be
         #"""




More information about the bind10-changes mailing list