Cleanup: Removed the try-except-fail antipattern from tests

Found cases where testing code was doing

    try:
        whatever
    except (some excption type):
        self.fail("exception shouldn't be thrown")

replaced it with just

    whatever

as this makes the unexpected errors easier to debug, and the tests
would fail just as much and aren't rendered less readable.

Thanks Markus Holtermann for review

1 files changed, 1 insertions, 5 deletions

diff --git a/tests/files/tests.py b/tests/files/tests.py
index 13560e509c..cd2ccaf9f1 100644
--- a/tests/files/tests.py
+++ b/tests/files/tests.py
@@ -6,7 +6,6 @@ import os
 import struct
 import tempfile
 import unittest
-import zlib
 from io import BytesIO, StringIO
 
 from django.core.files import File
@@ -233,10 +232,7 @@ class InconsistentGetImageDimensionsBug(unittest.TestCase):
         get_image_dimensions fails on some pngs, while Image.size is working good on them
         """
         img_path = os.path.join(os.path.dirname(upath(__file__)), "magic.png")
-        try:
-            size = images.get_image_dimensions(img_path)
-        except zlib.error:
-            self.fail("Exception raised from get_image_dimensions().")
+        size = images.get_image_dimensions(img_path)
         with open(img_path, 'rb') as fh:
             self.assertEqual(size, Image.open(fh).size)