From cf73187c7c13a7d913a8d069849f37b91157f497 Mon Sep 17 00:00:00 2001
From: Bob Halley <halley@dnspython.org>
Date: Sun, 23 May 2021 10:06:07 -0700
Subject: [PATCH] remove or comment out some prints in tests

---
 tests/test_processing_order.py | 2 --
 tests/test_rdataset.py         | 1 -
 tests/test_transaction.py      | 5 ++---
 tests/test_xfr.py              | 1 -
 tests/util.py                  | 4 ++--
 5 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/tests/test_processing_order.py b/tests/test_processing_order.py
index 45a90cff..4be695a0 100644
--- a/tests/test_processing_order.py
+++ b/tests/test_processing_order.py
@@ -80,14 +80,12 @@ def test_processing_order_empty():
 def test_processing_singleton_priority():
     rds = dns.rdataset.from_text('in', 'mx', 300, '10 a')
     po = rds.processing_order()
-    print(po)
     assert po == [rds[0]]
 
 
 def test_processing_singleton_weighted():
     rds = dns.rdataset.from_text('in', 'srv', 300, '1 10 1234 a')
     po = rds.processing_order()
-    print(po)
     assert po == [rds[0]]
 
 
diff --git a/tests/test_rdataset.py b/tests/test_rdataset.py
index 4710e2a9..69ec6ded 100644
--- a/tests/test_rdataset.py
+++ b/tests/test_rdataset.py
@@ -117,7 +117,6 @@ class RdatasetTestCase(unittest.TestCase):
                                      '2143 foo Ym9ndXM=')
         # Using startswith as I don't care about the repr of the rdata,
         # just the covers
-        print(repr(rds))
         self.assertTrue(repr(rds).startswith(
             '<DNS IN RRSIG(NSEC) rdataset:'))
 
diff --git a/tests/test_transaction.py b/tests/test_transaction.py
index bb69b714..9ac9f560 100644
--- a/tests/test_transaction.py
+++ b/tests/test_transaction.py
@@ -322,7 +322,7 @@ def test_explicit_rollback_and_commit(zone):
         with zone.writer() as txn:
             txn.rollback()
             for rdataset in txn:
-                print(rdataset)
+                pass
 
 def test_zone_changed(zone):
     # Read-only is not changed!
@@ -417,7 +417,6 @@ def test_write_after_rollback(zone):
 
 def test_zone_get_deleted(zone):
     with zone.writer() as txn:
-        print(zone.to_text())
         ns1 = dns.name.from_text('ns1', None)
         assert txn.get(ns1, dns.rdatatype.A) is not None
         txn.delete(ns1)
@@ -471,9 +470,9 @@ class ExpectedException(Exception):
     pass
 
 def test_zone_rollback(zone):
+    a99 = dns.name.from_text('a99.example.')
     try:
         with zone.writer() as txn:
-            a99 = dns.name.from_text('a99.example.')
             rds = dns.rdataset.from_text('in', 'a', 300, '10.0.0.99')
             txn.add(a99, rds)
             assert txn.name_exists(a99)
diff --git a/tests/test_xfr.py b/tests/test_xfr.py
index 7429fa35..3fa1eba0 100644
--- a/tests/test_xfr.py
+++ b/tests/test_xfr.py
@@ -600,7 +600,6 @@ def test_extract_serial_from_query():
     assert s is None
     assert s == xs
     (q, s) = dns.xfr.make_query(z, serial=10)
-    print(q)
     xs = dns.xfr.extract_serial_from_query(q)
     assert s == 10
     assert s == xs
diff --git a/tests/util.py b/tests/util.py
index 644a88a1..e8c13630 100644
--- a/tests/util.py
+++ b/tests/util.py
@@ -33,8 +33,8 @@ def check_enum_exports(module, eq_callback, only=None):
     """Make sure module exports all mnemonics from enums"""
     for attr in enumerate_module(module, enum.Enum):
         if only is not None and attr not in only:
-            print('SKIP', attr)
+            #print('SKIP', attr)
             continue
         for flag, value in attr.__members__.items():
-            print(module, flag, value)
+            #print(module, flag, value)
             eq_callback(getattr(module, flag), value)
-- 
2.47.3