rake_check 0.1.7 → 0.1.8

data/lib/rake_check/brakeman_checker.rb

@@ -1,5 +1,6 @@
 require 'colored'
 require 'json'
+require 'brakeman'
 ##
 # BrakemanChecker checks the output for undocumented classes and methods
 #
@@ -11,9 +12,9 @@ class BrakemanChecker
   # @return [Hash] Checkresult
   # @author dmasur
   def result
-    @shell_output = begin
-      `brakeman -f json 2>/dev/null`
-    rescue Errno::ENOENT
+    begin
+      @tracker = Brakeman.run `pwd`.strip
+    rescue
       "Brakeman not found"
     end
     {:type => :brakeman, :check_output => output, :status => status}
@@ -38,43 +39,26 @@ class BrakemanChecker
     # @return [String] Checkstatus
     # @author dmasur
     def status
-      if @shell_output == ''
+      if @tracker.nil?
         return 'N/A'
       else
-        begin
-          warnings_string = "#{color_count data["warnings"].count} Warnings"
-          errors_string = "#{color_count data["errors"].count} Errors"
-          return "#{warnings_string}, #{errors_string}"
-        rescue JSON::ParserError
-          return 'Parse Error'
-        end
+        "#{color_count @tracker.checks.warnings.count} Warnings"./app/views/admin/pages/index.html.haml
       end
     end
 
-    ##
-    # Parses the JSON Output
-    #
-    # @author dmasur
-    def data
-      raise JSON::ParserError if @shell_output.empty?
-      json_string = @shell_output.split("Generating report...").last
-      json = JSON.parse(json_string)
-    end
-
     ##
     # Gives the check output
     #
     # @return [String] Output
     # @author dmasur
     def output
-      if @shell_output == ''
-        return ''
+      if @tracker.nil?
+        ''
       else
-        begin
-          (data["warnings"].map { |warning| warning["message"] } +
-          data["errors"].map { |error| error["error"] }).join(", ")
-        rescue JSON::ParserError
-          return @shell_output
+        if @tracker.checks.warnings.empty?
+          return ''
+        else
+          return @tracker.report
         end
       end
     end

data/lib/rake_check/version.rb

@@ -3,5 +3,5 @@ module RakeCheck
   # The Version Constant
   #
   # @author dmasur
-  VERSION = "0.1.7"
+  VERSION = "0.1.8"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rake_check
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-19 00:00:00.000000000 Z
+date: 2012-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70129581054120 !ruby/object:Gem::Requirement
+  requirement: &70366337294260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581054120
+  version_requirements: *70366337294260
 - !ruby/object:Gem::Dependency
   name: colored
-  requirement: &70129581052460 !ruby/object:Gem::Requirement
+  requirement: &70366337338960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581052460
+  version_requirements: *70366337338960
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70129581050700 !ruby/object:Gem::Requirement
+  requirement: &70366337337680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581050700
+  version_requirements: *70366337337680
 - !ruby/object:Gem::Dependency
   name: reek
-  requirement: &70129581049100 !ruby/object:Gem::Requirement
+  requirement: &70366337336820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581049100
+  version_requirements: *70366337336820
 - !ruby/object:Gem::Dependency
   name: cane
-  requirement: &70129581048080 !ruby/object:Gem::Requirement
+  requirement: &70366337336080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581048080
+  version_requirements: *70366337336080
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &70129581084760 !ruby/object:Gem::Requirement
+  requirement: &70366337335360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581084760
+  version_requirements: *70366337335360
 - !ruby/object:Gem::Dependency
   name: brakeman
-  requirement: &70129581081740 !ruby/object:Gem::Requirement
+  requirement: &70366337334660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,7 +87,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129581081740
+  version_requirements: *70366337334660
 description: Checking the Project for Code Smells and bad documentation
 email:
 - dominik.masur@googlemail.com
@@ -142,7 +142,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2895400743441379252
+      hash: 2623682170575336086
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -151,7 +151,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2895400743441379252
+      hash: 2623682170575336086
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.10