clean

Author: lucasmcdonald3

src/aws_encryption_sdk/streaming_client.py

@@ -77,7 +77,6 @@
     from aws_encryption_sdk.materials_managers.mpl.cmm import CryptoMaterialsManagerFromMPL
 
 except ImportError as e:
-    print("streamingerror " + str(e))
     _HAS_MPL = False
 
 _LOGGER = logging.getLogger(__name__)

test_vector_handlers/src/awses_test_vectors/manifests/full_message/decrypt.py

@@ -130,12 +130,14 @@ def match(self, name, decrypt_fn):
             # fails in some way, and hence the overly-broad implicit try/catch here.
 
             with pytest.raises(Exception):
-                # When the exception is raised,
-                # it will write stderrs to console.
-                # However, this branch is expected to raise an exception,
-                # and will write that exception to console.
-                # Swallow the exception so CI build logs are cleaner,
+                # Here, an exception is expected.
+                # However, when the expected exception is raised,
+                # the Python environment will write stderrs to console.
+                # Redirect stderr to null-like sink
+                # so local and CI build logs are cleaner,
                 # and any actual issues are easier to see.
+                # If an exception is not raised as expected,
+                # `pytest.raises` will fail.
                 f = io.StringIO()
                 with contextlib.redirect_stderr(f):
                     decrypt_fn()