From ddae12928ae7dd919086ffcc4c0ec940e9024360 Mon Sep 17 00:00:00 2001
From: homm <homm86@gmail.com>
Date: Wed, 8 Oct 2014 06:49:56 +0400
Subject: [PATCH] add detailed test for blur and simple test for sharp

---
 Tests/images/color_snakes.png | Bin 0 -> 1311 bytes
 Tests/test_imageops_usm.py    |  34 +++++++++++++++++++++++++++++++---
 2 files changed, 31 insertions(+), 3 deletions(-)
 create mode 100644 Tests/images/color_snakes.png

diff --git a/Tests/images/color_snakes.png b/Tests/images/color_snakes.png
new file mode 100644
index 0000000000000000000000000000000000000000..bf3a351964f4d9ce61ef88222d133f66b45e4b2b
GIT binary patch
literal 1311
zcmZ`(Uue@n9KONSnKDEL_YYZ&;#+e`*WsF_f!eCwSm*2}cA~N@z0SC_iAhRRL73vp
z1QC^qj0rwDL>LV8!HGV|UVKu)M<)m}WeDP%h@y9C*Tv8UlH7g2@4Mgc`z3eNLxabf
zsdfqg(A<|yr16`Ksiv*?c_Z}Z1%5S7#8NQ;7H<ZA%UkfAR+8xy0GAE|Fm(-pUl^MD
z3P6zqU~LQl;T`}j#@x~g5eK##$x#b{z;1FkfZ4fLyxswIBx7e%1A?OGSvjkZBi6|q
zm<@pF2zZ!BwoE&DtzZdGlyMmX9upWcv`eukqf90>M8|a#(cNsA<rs;gX<9V1IU$`m
zT9xBjlsRwPh5%uyRANhAtZq&~J`#yQE(AlNAZ7%ua>16JV8Pm7S!8{k1hN!UHEdNc
z&}3bCTrb*Dh9QY+SEWu{&23~VSXEotAS4mUvmC4^Lyo$Mj6^n-WtEyMqiAZbr>p{z
zhVrOjTUf`}O^1716aH7EBO8@uZWAlsL{hnsMwYG>NefOER9oUj*N`9H=txBs*Yk#n
zEDOt|?yAI#`T3cRG}56^Rp15vg5#!&oyw%HT1{Te&mXOqzzcc=r1ip(>L4wXP;o<D
z7d$7zx&gm1u4}qEY{&{Cg}D+B<Y&8SMmJ0233!BAMX-wF7m#9mz83yw?!P~bE<z71
z!n$k`{|Q1$&#Jj{Lbj0<;`p$@a{|{xbKPXn(<4G3$5R1`#7!jIx;dijn&i63lPTTB
zhFI*|&!~z;NcNFuUsX1(i{f1?QCPWt0!Y;Ss#e5@0_sb|Mq1l0-?~D9eNFL|l|O%5
zwoT^F#cn@%^<;im@+B9(^X_MP;oGC-{*Febzd!bH@8{l~r(d7l1KXZ2&PJXp%O3;x
zzpVb~Y|dDnGt8OkyQi){Yabq1`_TL8F+UUh-ubox?dWI;yf25zG1PZtFtKoW?CL*2
Cf{m{L

literal 0
HcmV?d00001

diff --git a/Tests/test_imageops_usm.py b/Tests/test_imageops_usm.py
index ba65f211b..d1bdacd52 100644
--- a/Tests/test_imageops_usm.py
+++ b/Tests/test_imageops_usm.py
@@ -5,6 +5,7 @@ from PIL import ImageOps
 from PIL import ImageFilter
 
 im = Image.open("Tests/images/hopper.ppm")
+snakes = Image.open("Tests/images/color_snakes.png")
 
 
 class TestImageOpsUsm(PillowTestCase):
@@ -16,7 +17,7 @@ class TestImageOpsUsm(PillowTestCase):
         self.assertEqual(i.size, (128, 128))
         # i.save("blur.bmp")
 
-        i = ImageOps.usm(im, 2.0, 125, 8)
+        i = ImageOps.unsharp_mask(im, 2.0, 125, 8)
         self.assertEqual(i.mode, "RGB")
         self.assertEqual(i.size, (128, 128))
         # i.save("usm.bmp")
@@ -33,7 +34,7 @@ class TestImageOpsUsm(PillowTestCase):
         self.assertEqual(i.mode, "RGB")
         self.assertEqual(i.size, (128, 128))
 
-    def test_usm(self):
+    def test_usm_formats(self):
 
         usm = ImageOps.unsharp_mask
         self.assertRaises(ValueError, lambda: usm(im.convert("1")))
@@ -45,7 +46,7 @@ class TestImageOpsUsm(PillowTestCase):
         usm(im.convert("CMYK"))
         self.assertRaises(ValueError, lambda: usm(im.convert("YCbCr")))
 
-    def test_blur(self):
+    def test_blur_formats(self):
 
         blur = ImageOps.gaussian_blur
         self.assertRaises(ValueError, lambda: blur(im.convert("1")))
@@ -57,6 +58,33 @@ class TestImageOpsUsm(PillowTestCase):
         blur(im.convert("CMYK"))
         self.assertRaises(ValueError, lambda: blur(im.convert("YCbCr")))
 
+    def test_usm_accuracy(self):
+
+        i = snakes._new(ImageOps.unsharp_mask(snakes, 5, 1024, 0))
+        # Image should not be changed because it have only 0 and 255 levels.
+        self.assertEqual(i.tobytes(), snakes.tobytes())
+
+    def test_blur_accuracy(self):
+
+        i = snakes._new(ImageOps.gaussian_blur(snakes, .9))
+        # Alpha channel must match whole.
+        self.assertEqual(i.split()[3], snakes.split()[3])
+        # These pixels surrounded with pixels with 255 intensity.
+        # They must be 255.
+        for x, y, c in [(1, 0, 1), (2, 0, 1), (7, 8, 1), (8, 8, 1), (2, 9, 1),
+                        (7, 3, 0), (8, 3, 0), (5, 8, 0), (5, 9, 0), (1, 3, 0),
+                        (4, 3, 2), (4, 2, 2)]:
+            self.assertEqual(i.im.getpixel((x, y))[c], 255)
+        # Fuzzy match.
+        gp = lambda x, y: i.im.getpixel((x, y))
+        self.assertTrue(212 <= gp(7, 4)[0] <= 214)
+        self.assertTrue(212 <= gp(7, 5)[2] <= 214)
+        self.assertTrue(212 <= gp(7, 6)[2] <= 214)
+        self.assertTrue(212 <= gp(7, 7)[1] <= 214)
+        self.assertTrue(212 <= gp(8, 4)[0] <= 214)
+        self.assertTrue(212 <= gp(8, 5)[2] <= 214)
+        self.assertTrue(212 <= gp(8, 6)[2] <= 214)
+        self.assertTrue(212 <= gp(8, 7)[1] <= 214)
 
 if __name__ == '__main__':
     unittest.main()