Allow for no exceptions to be raised and cleanup

Allow for no exceptions to be raised

Add some helpful comments

Remove doctest skips

Remove TODO note

Cleanup and skip failing doctest

skimage/_shared/_warnings.py

@@ -85,12 +85,15 @@ def expected_warnings(matching):
     Uses `all_warnings` to ensure all warnings are raised.
     Upon exiting, it checks the recorded warnings for the desired matching
     string.  Raises a warning if the match was not found or an Unexpected
-    warning is found.
+    warning is found.  You can pass an empty regex as on of the matches
+    to allow for no matches: "\A\Z".
 
     """
     with all_warnings() as w:
+        # enter context
         yield w
-        remaining = [m for m in matching]
+        # exited user context, check the recorded warnings
+        remaining = [m for m in matching if not '\A\Z' in m.split('|')]
         for warn in w:
             found = False
             for match in matching:

skimage/_shared/testing.py

@@ -117,7 +117,7 @@ def color_check(plugin, fmt='png'):
     testing.assert_allclose(img2.astype(np.uint8), r2)
 
     img3 = img_as_float(img)
-    with expected_warnings(['precision loss']):
+    with expected_warnings(['precision loss|unclosed file']):
         r3 = roundtrip(img3, plugin, fmt)
     testing.assert_allclose(r3, img)
 
@@ -129,12 +129,12 @@ def color_check(plugin, fmt='png'):
             r4 = roundtrip(img4, plugin, fmt)
         testing.assert_allclose(r4, img4)
     else:
-        with expected_warnings(['sign loss']):
+        with expected_warnings(['sign loss|precision loss|unclosed file']):
             r4 = roundtrip(img4, plugin, fmt)
             testing.assert_allclose(r4, img_as_ubyte(img4))
 
     img5 = img_as_uint(img)
-    with expected_warnings(['precision loss']):
+    with expected_warnings(['precision loss|unclosed file']):
         r5 = roundtrip(img5, plugin, fmt)
     testing.assert_allclose(r5, img)
 
@@ -154,7 +154,7 @@ def mono_check(plugin, fmt='png'):
     testing.assert_allclose(img2.astype(np.uint8), r2)
 
     img3 = img_as_float(img)
-    with expected_warnings(['precision loss']):
+    with expected_warnings(['precision|unclosed file|\A\Z']):
         r3 = roundtrip(img3, plugin, fmt)
     if r3.dtype.kind == 'f':
         testing.assert_allclose(img3, r3)
@@ -165,11 +165,11 @@ def mono_check(plugin, fmt='png'):
         img4 = img_as_int(img)
     if fmt.lower() in (('tif', 'tiff')):
         img4 -= 100
-        with expected_warnings(['sign loss']):
+        with expected_warnings(['sign loss|\A\Z']):
             r4 = roundtrip(img4, plugin, fmt)
         testing.assert_allclose(r4, img4)
     else:
-        with expected_warnings(['sign loss']):
+        with expected_warnings(['precision loss|sign loss|unclosed file']):
             r4 = roundtrip(img4, plugin, fmt)
             testing.assert_allclose(r4, img_as_uint(img4))
 

skimage/_shared/utils.py

@@ -163,4 +163,3 @@ def assert_nD(array, ndim, arg_name='image'):
         ndim = [ndim]
     if not array.ndim in ndim:
         raise ValueError(msg % (arg_name, '-or-'.join([str(n) for n in ndim])))
-