Merge pull request #26 from hyrodium/feature/test_error

Add test for ErrorException

Project.toml

@@ -1,7 +1,7 @@
 name = "ImageClipboard"
 uuid = "6db54171-f50f-4661-a74f-bc514ef16cee"
 authors = ["hyrodium <[email protected]> and contributors"]
-version = "0.1.0"
+version = "0.1.1"
 
 [deps]
 ColorTypes = "3da002f7-5984-5a60-b8a6-cbb66c0b333f"
@@ -17,6 +17,7 @@ ImageIO = "0.5"
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 Images = "916415d5-f1e6-5110-898d-aaa5f9f070e0"
+InteractiveUtils = "b77e0a4c-d291-57a0-90e8-8db25a27a240"
 
 [targets]
-test = ["Test", "Images"]
+test = ["Test", "Images", "InteractiveUtils"]

src/_xclip.jl

@@ -12,9 +12,13 @@ function _xclip()
 
     # Pipe clipboard image to buffer
     if success(pipeline(`xclip -selection clipboard -t image/png -o`; stdout=img_buf))
-        # Load image from buffer
-        img = load(img_buf)
-        return img
+        try
+            # Load image from buffer
+            img = load(img_buf)
+            return img
+        catch
+            error("No image in clipboard")
+        end
     else
         error("No image in clipboard")
     end

test/runtests.jl

@@ -1,6 +1,7 @@
 using ImageClipboard
 using Images
 using Test
+using InteractiveUtils
 
 @testset "ImageClipboard.jl" begin
 
@@ -9,7 +10,7 @@ using Test
         clipboard_img(img)
         img2 = clipboard_img()
 
-        # FIXME, windows..
+        # FIXME, windows exception (#10)
         if Sys.iswindows()
             @test RGBA.(img) == img2
         else
@@ -22,10 +23,15 @@ using Test
         clipboard_img(img)
         img2 = clipboard_img()
 
-        # FIXME, windows..
+        # FIXME, windows exception (#10)
         if !Sys.iswindows()
             @test img == img2
         end
     end
 
+    @testset "Error if no image" begin
+        clipboard("")
+        @test_throws ErrorException("No image in clipboard") clipboard_img()
+    end
+
 end