jslint-v8 1.1.2 → 1.1.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    OTlmNDUzNWNjZmExMTQ0OTJmYTdkYWEzOGY1ODE0NTM5OTZiYTFhMg==
+    OWIyMDg4ODBiODUwNTYyNmE3Nzg3YmRlYTI5MjE3OTdiZTg4Y2QzOQ==
   data.tar.gz: !binary |-
-    NWJmYWJmMjVhOGQ4Y2NhNTkyNmIzNzM1ZWQ0MzFkZWQ5ZDhhMGRkOA==
+    YjYzNGQ1OThlOGQ1ODZjZTQ2ZWZiYTgyYzlmMmFiNWE2NjkxODJhOA==
 SHA512:
   metadata.gz: !binary |-
-    MzkyODk3OTZmY2JiMDVmYTZlZDIyMjYzYWZlNTk1ZmM2N2U2MzRlYzBiY2Y5
-    NzViNTczMDRhNzM2MzgxNzQ2OTVhYmU0OTUzYjBjYmNlMzA5YTExNWY4MmRi
-    OTE0Y2JkYzc5Zjc3YWYwYzZiNmFlODM2ZWYyYTkyYmIwNGEwNGE=
+    NTJmMDcwNDdhZmJhMzY5MGFhNDRiMmJhNmYxYzQwMzZhZjQ0ODMxOThkM2Q3
+    YmIyZTllMTE5ZTljMWI2OWVlODQ4OTI0Yjc5YWVlOWQ4NDgwYjFlZTc1ZTNj
+    YmRjYjA5MmU2MjcyNTdhZmJiMTA5MmVmZGE0NTdmY2RlMTg0NjI=
   data.tar.gz: !binary |-
-    YThiNzExZmIwZjc5MjBiODU1NTZmODZmMDQyMGE3NmJjZjBlZTg4YTRlMzY5
-    MDc3ZGEzMWM0ODBmZTkzMzgxODQwNmMwMWRhMjU3YTNkOTg4YmM0Y2M1YmQ1
-    Y2YzNDM2YTZhNDgyOTFkNzM4ZGY0ODE0MTJjNzgyOGM0NzE5NmY=
+    YWQ1N2ViNzE0MTIxODUyMjNjNzc0Y2RkNjk0OTY5MDE4YzlmN2IwMWUzZDky
+    OTUzYjFmYTE2YzJiMDNhMmRmMTNkYWI4OTMyMGE1YjdlN2ZjMWM2N2RiNTA5
+    OWNhZDk2MTY2NmVhMzkwZTJiMGY5OTYyMzJkNjhlYTBjODYwMmU=

data/bin/jslint-v8

@@ -1,6 +1,7 @@
 #!/usr/bin/env ruby
-require 'rubygems'
-require File.expand_path("../lib/jslint-v8", File.dirname(__FILE__))
+$LOAD_PATH.unshift File.expand_path("../lib", File.dirname(__FILE__))
+
+require "jslint-v8"
 
 def print_usage
    STDERR.puts JSLintV8::OptionParser.new.help
@@ -27,7 +28,11 @@ end
 print_usage if ARGV.empty?
 
 # set up a formatter for standard output
-formatter = JSLintV8::Formatter.new(STDOUT)
+if (!parser.options[:json])
+  formatter = JSLintV8::Formatter.new(STDOUT)
+else
+  formatter = JSLintV8::Json.new(STDOUT)
+end
 
 # set up a runner
 runner = JSLintV8::Runner.new(ARGV)

data/lib/jslint-v8.rb

@@ -1,8 +1,9 @@
 module JSLintV8 end
 
-require File.expand_path("jslint-v8/runner", File.dirname(__FILE__))
-require File.expand_path("jslint-v8/rake_task", File.dirname(__FILE__))
-require File.expand_path("jslint-v8/lint_error", File.dirname(__FILE__))
-require File.expand_path("jslint-v8/formatter", File.dirname(__FILE__))
-require File.expand_path("jslint-v8/option_parser", File.dirname(__FILE__))
-require File.expand_path("jslint-v8/version", File.dirname(__FILE__))
+require 'jslint-v8/runner'
+require 'jslint-v8/rake_task'
+require 'jslint-v8/lint_error'
+require 'jslint-v8/formatter'
+require 'jslint-v8/json'
+require 'jslint-v8/option_parser'
+require 'jslint-v8/version'

data/lib/jslint-v8/json.rb

@@ -0,0 +1,39 @@
+require 'json'
+
+module JSLintV8
+  class Json
+    attr_reader :output_stream
+
+    def initialize(stream)
+      @output_stream = stream
+    end
+
+    def tick(errors)
+    end
+
+    def summary(tested_files, lint_result)
+      if lint_result.keys.any?
+        output = lint_result.map do |file_errors|
+          { file_errors[0] =>
+              file_errors[1].map do |errors|
+                { "line" => errors.line_number,
+                  "column" => errors.character,
+                  "reason" => errors.reason,
+                  "evidence" => errors.evidence
+                }
+              end
+          }
+        end
+        print_error_summary(output)
+      end
+    end
+
+  private
+
+    def print_error_summary(result)
+      out = output_stream
+      out.print JSON.dump(result)
+    end
+
+  end
+end

data/lib/jslint-v8/option_parser.rb

@@ -25,6 +25,10 @@ module JSLintV8
             end
          end
 
+         on("--json", "Return output in JSON format") do |j|
+            options[:json] = j
+         end
+
          on("-h", "--help", "Show this message") do
             STDERR.puts self.help
             exit(-1)

data/lib/jslint-v8/version.rb

@@ -3,7 +3,7 @@ module JSLintV8
   module Version
      MAJOR = 1
      MINOR = 1
-     PATCH = 2
+     PATCH = 3
      BUILD = nil
     
      STRING = [MAJOR, MINOR, PATCH, BUILD].compact.join('.')

data/test/helper.rb

@@ -9,6 +9,10 @@ def erb_fixture(basename)
   ::ERB.new(File.read(erb_filename(basename))).result
 end
 
+def json_fixture(basename)
+  File.read(json_filename(basename))
+end
+
 def erb_filename(basename)
   File.expand_path("fixtures/#{basename}.txt.erb", File.dirname(__FILE__))
 end
@@ -16,3 +20,7 @@ end
 def js_filename(basename)
   File.expand_path("fixtures/#{basename}.js", File.dirname(__FILE__))
 end
+
+def json_filename(basename)
+  File.expand_path("fixtures/#{basename}.json", File.dirname(__FILE__))
+end

data/test/test_json.rb

@@ -0,0 +1,34 @@
+require File.expand_path('helper', File.dirname(__FILE__))
+require 'erb'
+require 'stringio'
+require 'json'
+
+class TestJsonFormatter < Test::Unit::TestCase
+
+  def setup
+    @output = String.new
+    @output_stream = StringIO.new(@output, "w+")
+
+    @formatter = JSLintV8::Json.new(@output_stream)
+  end
+
+  def test_summary_with_errors
+    error_1 = JSLintV8::LintError.new("line" => 42, "character" => 10, "reason" => "because i can")
+    error_2 = JSLintV8::LintError.new("line" => 43, "character" => 3, "reason" => "a proof is not a proof")
+    error_3 = JSLintV8::LintError.new("line" => 139, "character" => 12, "reason" => "undefined global 'foo'")
+    result = {
+      "foo" => [error_1, error_2],
+      "bar" => [error_3]
+    }
+
+    @formatter.summary(%w(foo bar baz one two three), result)
+
+    assert_equal json_fixture("json-formatter-summary-errors").strip, @output
+  end
+
+  def test_summary_with_no_errors
+    @formatter.summary(%w(foo bar), {})
+
+    assert_equal json_fixture("json-formatter-summary-noerrors"), @output
+  end
+end

data/test/test_task.rb

@@ -27,7 +27,7 @@ class TestTask < Test::Unit::TestCase
   end
 
   def test_default_description
-    JSLintV8::RakeTask.new 
+    JSLintV8::RakeTask.new
 
     task = Rake.application.lookup("lint")
 
@@ -35,7 +35,7 @@ class TestTask < Test::Unit::TestCase
   end
 
   def test_default_include_pattern
-    task = JSLintV8::RakeTask.new 
+    task = JSLintV8::RakeTask.new
 
     assert_equal "app/javascripts/**/*.js", task.include_pattern
   end
@@ -63,9 +63,9 @@ class TestTask < Test::Unit::TestCase
 
     task = JSLintV8::RakeTask.new do |task|
       task.name = "foo"
-      
+
       task.description = "Points out the bad codezzzz"
-      
+
       task.include_pattern = "js/**/*.js"
       task.exclude_pattern = "js/**/*.txt"
 
@@ -93,7 +93,7 @@ class TestTask < Test::Unit::TestCase
       task.exclude_pattern = File.expand_path("fixtures/*.erb", File.dirname(__FILE__))
     end
 
-    expected_files = Dir.glob(File.expand_path("fixtures/*.js", File.dirname(__FILE__))).sort
+    expected_files = Dir.glob(File.expand_path("fixtures/*.{js,json}", File.dirname(__FILE__))).sort
 
     assert_equal expected_files, task.files_to_run
   end
@@ -151,4 +151,4 @@ class TestTask < Test::Unit::TestCase
 
     assert_equal erb_fixture("cli-suite-expected-output"), result
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jslint-v8
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.3
 platform: ruby
 authors:
 - William Howard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-08 00:00:00.000000000 Z
+date: 2015-09-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: therubyracer
@@ -68,6 +68,7 @@ files:
 - lib/jslint-v8.rb
 - lib/jslint-v8/formatter.rb
 - lib/jslint-v8/js/jslint.js
+- lib/jslint-v8/json.rb
 - lib/jslint-v8/lint_error.rb
 - lib/jslint-v8/option_parser.rb
 - lib/jslint-v8/rake_task.rb
@@ -77,6 +78,7 @@ files:
 - test/suite.rb
 - test/test_cli.rb
 - test/test_formatter.rb
+- test/test_json.rb
 - test/test_lint_error.rb
 - test/test_runner.rb
 - test/test_task.rb
@@ -99,15 +101,16 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.3
 signing_key: 
 specification_version: 4
 summary: JSLint CLI and rake tasks via therubyracer (JavaScript V8 gem)
 test_files:
-- test/test_formatter.rb
-- test/helper.rb
-- test/test_cli.rb
-- test/test_lint_error.rb
+- test/suite.rb
 - test/test_runner.rb
+- test/helper.rb
 - test/test_task.rb
-- test/suite.rb
+- test/test_lint_error.rb
+- test/test_formatter.rb
+- test/test_cli.rb
+- test/test_json.rb