fail if did not got version, resolves part of #39

toy · Oct 2, 2014 · 97cc12c · 97cc12c
1 parent c6d9054
commit 97cc12c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -12,7 +12,7 @@ Metrics/CyclomaticComplexity:
   Max: 10
 
 Metrics/MethodLength:
-  Max: 20
+  Max: 25
 
 Metrics/PerceivedComplexity:
   Max: 8

diff --git a/CHANGELOG.markdown b/CHANGELOG.markdown
@@ -2,6 +2,7 @@
 
 ## unreleased
 
+* Fail if version of bin can't be detected [#39](https://github.com/toy/image_optim/issues/39) [@toy](https://github.com/toy)
 * Check path in `XXX_BIN` to exist, be a file and be executable [@toy](https://github.com/toy)
 
 ## v0.16.0 (2014-09-12)

diff --git a/lib/image_optim/bin_resolver.rb b/lib/image_optim/bin_resolver.rb
@@ -29,6 +29,8 @@ def to_s
       end
 
       def check!
+        fail BadVersion, "didn't get version of `#{self}`" unless version
+
         is = ComparableCondition.is
         case name
         when :pngcrush
-Original file line number
+Diff line change
@@ Expand Up / @@ -12,7 +12,7 @@ Metrics/CyclomaticComplexity: @@
       Max: 10
     Metrics/MethodLength:
-      Max: 20
+      Max: 25
     Metrics/PerceivedComplexity:
       Max: 8
@@ Expand Down @@