permpress 0.0.1

data/spec/fixtures/good.json

@@ -0,0 +1,4 @@
+{
+  "good": 1,
+  "not": "bad"
+}

data/spec/fixtures/good.rb

@@ -0,0 +1,5 @@
+# Bob is useless
+class Bob
+  def x_man
+  end
+end

data/spec/fixtures/good.scss

@@ -0,0 +1,3 @@
+p {
+  border: 0;
+}

data/spec/fixtures/lint.txt

@@ -0,0 +1 @@
+lint

data/spec/permpress/checkstyle_spec.rb

@@ -0,0 +1,66 @@
+require 'spec_helper'
+
+describe Permpress::Checkstyle do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    context 'when config flag is present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'java',
+          %w(Good.java bad.java),
+          %W(
+            -jar #{permpress_path}/lib/permpress/checkstyle/checkstyle_logger-all.jar
+            -c checks.xml
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::Checkstyle.start(['lint', '--config', 'checks.xml', 'Good.java', 'bad.java'])
+      end
+    end
+
+    context 'when config flag is not present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'java',
+          %w(Good.java bad.java),
+          %W(
+            -jar #{permpress_path}/lib/permpress/checkstyle/checkstyle_logger-all.jar
+            -c #{permpress_path}/lib/permpress/checkstyle/sun_checks.xml
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::Checkstyle.start(['lint', 'Good.java', 'bad.java'])
+      end
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.java', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['checkstyle', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:1:0::com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTypeCheck:error:"\
+          "Missing a Javadoc comment.\n"\
+          "#{file}:1:14::com.puppycrawl.tools.checkstyle.checks.naming.TypeNameCheck:error:"\
+          "Name 'bad' must match pattern '^[A-Z][a-zA-Z0-9]*$'.\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/Good.java', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['checkstyle', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end

data/spec/permpress/coffeelint_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Permpress::CoffeeLint do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    context 'when config flag is present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'coffeelint',
+          %w(good.coffee bad.coffee),
+          %W(
+            --reporter=#{permpress_path}/lib/permpress/coffeelint/lintci.coffee
+            --nocolor
+            --file=coffeelint.json
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::CoffeeLint.start(['lint', '--config', 'coffeelint.json', 'good.coffee', 'bad.coffee'])
+      end
+    end
+
+    context 'when config flag is not present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'coffeelint',
+          %w(good.coffee bad.coffee),
+          %W(
+            --reporter=#{permpress_path}/lib/permpress/coffeelint/lintci.coffee
+            --nocolor
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::CoffeeLint.start(['lint', 'good.coffee', 'bad.coffee'])
+      end
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.coffee', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['coffeelint', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:1:::camel_case_classes:error:Class names should be camel cased\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.coffee', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['coffeelint', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end

data/spec/permpress/command_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+require 'stringio'
+
+describe Permpress::Command do
+  let(:file){'fake file.txt'}
+  let(:command){Permpress::Command.new('cat', [file], %w(-b))}
+
+  describe '#run' do
+    let(:output){StringIO.new}
+
+    context 'when command is successfully run' do
+      let(:file){File.expand_path('../../fixtures/lint.txt', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{command.run(output)}.to exit_successfully
+
+        expect(output.string).to eq("     1\tlint\n")
+      end
+    end
+
+    context 'when command is unsuccessfully run' do
+      let(:command){Permpress::Command.new('this-command-does-not-exist', [file], %w(-b))}
+
+      it 'generates the expected output' do
+        expect{command.run(output)}.to_not exit_successfully
+
+        expect(output.string).to eq("sh: this-command-does-not-exist: command not found\n")
+      end
+    end
+  end
+
+  describe '#command/#to_s' do
+    it 'generates an executable command' do
+      expect(command.command).to eq('cat -b fake\ file.txt 2>&1')
+    end
+  end
+end

data/spec/permpress/csslint_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe Permpress::CSSLint do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    context 'when config flag is present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'csslint',
+          %w(good.css bad.css),
+          %W(
+            --errors=#{Permpress::CSSLint::RULES}
+            --format=compact
+            --config=.csslintrc
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::CSSLint.start(['lint', '--config', '.csslintrc', 'good.css', 'bad.css'])
+      end
+    end
+
+    context 'when config flag is not present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'csslint',
+          %w(good.css bad.css),
+          %W(
+            --errors=#{Permpress::CSSLint::RULES}
+            --format=compact
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::CSSLint.start(['lint', 'good.css', 'bad.css'])
+      end
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.css', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['csslint', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}: line 2, col 5, Error - Using width with border can sometimes"\
+          " make elements larger than you expect.\n\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.css', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['csslint', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq("#{file}: Lint Free!\n")
+      end
+    end
+  end
+end

data/spec/permpress/golint_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+
+describe Permpress::GoLint do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    it 'executes command with expected arguments' do
+      expect(Permpress::Command).to receive(:new).with(
+        'fgt golint',
+        %w(good.go bad.go),
+        []
+      ).and_return(instance_double(Permpress::Command, run: nil))
+
+      Permpress::GoLint.start(['lint', 'good.go', 'bad.go'])
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.go', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['golint', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:5:1: exported function Main should have comment or be unexported\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.go', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['golint', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end

data/spec/permpress/jshint_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe Permpress::JSHint do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    context 'when config flag is present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'jshint',
+          %w(good.js bad.js),
+          %W(
+            --reporter #{permpress_path}/lib/permpress/jshint/formatter.js
+            --config .jshintrc
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::JSHint.start(['lint', '--config', '.jshintrc', 'good.js', 'bad.js'])
+      end
+    end
+
+    context 'when config flag is not present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'jshint',
+          %w(good.js bad.js),
+          %W(
+            --reporter #{permpress_path}/lib/permpress/jshint/formatter.js
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::JSHint.start(['lint', 'good.js', 'bad.js'])
+      end
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.js', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['jshint', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:2:13::E031:error:Bad assignment.\n"\
+          "#{file}:2:13::W030:warning:Expected an assignment or function call and instead saw an expression.\n"\
+          "#{file}:2:14::W033:warning:Missing semicolon.\n"\
+          "#{file}:2:15::W030:warning:Expected an assignment or function call and instead saw an expression.\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.js', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['jshint', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end

data/spec/permpress/jsonlint_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe Permpress::JSONLint do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    it 'executes command with expected arguments' do
+      expect(Permpress::Command).to receive(:new).with(
+        'durable-json-lint',
+        %w(good.json bad.json),
+        ['--format', '{{file}}:{{line}}:{{column}}:::error:{{{description}}}']
+      ).and_return(instance_double(Permpress::Command, run: nil))
+
+      Permpress::JSONLint.start(['lint', 'good.json', 'bad.json'])
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.json', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['jsonlint', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:2:2:::error:Json strings must use double quotes\n"\
+          "#{file}:3:2:::error:Keys must be double quoted in Json. Did you mean \"not\"?\n"\
+          "#{file}:3:7:::error:Invalid Json number\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.json', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['jsonlint', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end

data/spec/permpress/rubocop_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+describe Permpress::RuboCop do
+  describe '#lint', :unit do
+    let(:permpress_path){File.expand_path('../../..', __FILE__)}
+
+    context 'when config flag is present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'rubocop',
+          %w(good.rb bad.rb),
+          %W(
+            --require #{permpress_path}/lib/permpress/rubocop/formatter.rb
+            --format Permpress::Rubocop::Formatter
+            --no-color
+            --config .rubocop.yml
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::RuboCop.start(['lint', '--config', '.rubocop.yml', 'good.rb', 'bad.rb'])
+      end
+    end
+
+    context 'when config flag is not present' do
+      it 'executes command with expected arguments' do
+        expect(Permpress::Command).to receive(:new).with(
+          'rubocop',
+          %w(good.rb bad.rb),
+          %W(
+            --require #{permpress_path}/lib/permpress/rubocop/formatter.rb
+            --format Permpress::Rubocop::Formatter
+            --no-color
+          )
+        ).and_return(instance_double(Permpress::Command, run: nil))
+
+        Permpress::RuboCop.start(['lint', 'good.rb', 'bad.rb'])
+      end
+    end
+  end
+
+  describe '#lint', :integration do
+    include_context 'observed output'
+
+    context 'when the file is not lint free' do
+      let(:file){File.expand_path('../../fixtures/bad.rb', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['rubocop', 'lint', file])}.to_not exit_successfully
+
+        expect($stdout.string).to eq(
+          "#{file}:1:1:5:Style/Documentation:convention:Missing top-level class documentation comment.\n"\
+          "#{file}:2:7:4:Style/MethodName:convention:Use snake_case for methods.\n"
+        )
+      end
+    end
+
+    context 'when the file is lint free' do
+      let(:file){File.expand_path('../../fixtures/good.rb', __FILE__)}
+
+      it 'generates the expected output' do
+        expect{Permpress::CLI.start(['rubocop', 'lint', file])}.to exit_successfully
+
+        expect($stdout.string).to eq('')
+      end
+    end
+  end
+end