fasterer 0.4.2 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1febf30dbcdf9c5b961a85811548a95f9fb1e4bb
-  data.tar.gz: 7338f2684178241b12cb2d95cd06c4df07b263dc
+  metadata.gz: dad76ac74cc33d9e1f82e23005cef6f1bd60532e
+  data.tar.gz: ff12c05326d55e9e10b4c07f09bc9206e4994a11
 SHA512:
-  metadata.gz: ee35ae5bd949ccc8bd03143cd012aa7238ed3721d4cd2d4a74e15f7f266af7e1f537c360d153f352627d1fa661af63fdcb92507fdf15bafe048a419203e895a7
-  data.tar.gz: ba4cec762866c5f61b0a21118b82ff7aa3102acb6d84be2fdebfaf6342f669e5383328d29d15bdd555827075d082574f642fc3626d5a71395c5e3aa1f926d671
+  metadata.gz: c54c879d1264f00f6a783ddf42f0bd973c2fac6bfd7cea52857659215ea0fc7cd5f7937e683a6e8ef8f9bb3e99ae81b75d081d3c2a3c4fed588ce5a6f997e0ae
+  data.tar.gz: 5e0c03c6cbb14319cd8c3e1f1971cbe94962c3e97e5d2b522f639c875d424f9f602586d05bfa89537bcc02c359fd62690277b51021dffede35841244fa9dc1b5

data/.fasterer.yml

@@ -15,7 +15,6 @@ speedups:
   block_vs_symbol_to_proc: true
   proc_call_vs_yield: true
   gsub_vs_tr: true
-  select_last_vs_reverse_detect: true
   getter_vs_attr_reader: true
   setter_vs_attr_writer: true
 

data/.travis.yml

@@ -1,7 +1,5 @@
 language: ruby
 before_install:
-  # Travis bundler versions are quite out of date and can cause install errors
-  # see: https://github.com/rubygems/rubygems/issues/1419
   - gem update bundler
 rvm:
 - 1.9.3
@@ -15,3 +13,12 @@ rvm:
 env:
   global:
     secure: MBTxmpWCjrsNKPlOoFwJUMHze3GPMz8YCXQFQG3zJBh3WGNQnz4z91ra/1P/DClyVCxHGSFCOswxCNe4kJ2zAnPyQLqGSinXy9uDpqZQUEdaRoQbPnh4/bguZNSJ429gtTpMdDSNOgQ+Hra2EFnWwHA+rLF6ImksMsu3XGKGxGE=
+    - CC_TEST_REPORTER_ID=86c5edffe4c38e16cb1467e3220a4f3dfa0a0ac8fa6ee8fddd62e200245f4b77
+before_script:
+  - curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+  - chmod +x ./cc-test-reporter
+  - ./cc-test-reporter before-build
+script:
+  - bundle exec rspec
+after_script:
+  - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.5.0
+
+- New style of outputting offenses: `spec/support/output/sample_code.rb:1 For loop is slower than using each.`
+
 ## 0.4.1
 - Upgrade ruby parser version to 3.11.0 (to stop warnings)
 

data/README.md

@@ -2,7 +2,6 @@
 [![Code Climate](https://codeclimate.com/github/DamirSvrtan/fasterer/badges/gpa.svg)](https://codeclimate.com/github/DamirSvrtan/fasterer)
 [![Gem Version](https://badge.fury.io/rb/fasterer.svg)](http://badge.fury.io/rb/fasterer)
 [![Test Coverage](https://codeclimate.com/github/DamirSvrtan/fasterer/badges/coverage.svg)](https://codeclimate.com/github/DamirSvrtan/fasterer/coverage)
-[![Dependency Status](https://gemnasium.com/DamirSvrtan/fasterer.svg)](https://gemnasium.com/DamirSvrtan/fasterer)
 
 # Fasterer
 

data/lib/fasterer/file_traverser.rb

@@ -80,14 +80,14 @@ module Fasterer
     end
 
     def output(analyzer)
-      puts analyzer.file_path.colorize(:red)
-
       offenses_grouped_by_type(analyzer).each do |error_group_name, error_occurences|
-        puts "#{Fasterer::Offense::EXPLANATIONS[error_group_name]}."\
-             " Occurred at lines: #{error_occurences.map(&:line_number).join(', ')}."
+        error_occurences.map(&:line_number).each do |line|
+          file_and_line = "#{analyzer.file_path}:#{line}"
+          print "#{file_and_line.colorize(:red)} #{Fasterer::Offense::EXPLANATIONS[error_group_name]}.\n"
+        end
       end
 
-      puts
+      print "\n"
     end
 
     def offenses_grouped_by_type(analyzer)

data/lib/fasterer/version.rb

@@ -1,3 +1,3 @@
 module Fasterer
-  VERSION = '0.4.2'
+  VERSION = '0.5.0'
 end

data/spec/lib/fasterer/file_traverser_spec.rb

@@ -334,4 +334,24 @@ describe Fasterer::FileTraverser do
         .to match_array(['user.rb - RubyParser::SyntaxError - unterminated string meets end of file. near line 1: ""'])
     end
   end
+
+  describe 'output' do
+    let(:test_file_path) { RSpec.root.join('support', 'output', 'sample_code.rb') }
+    let(:analyzer) { Fasterer::Analyzer.new(test_file_path) }
+    let(:file_traverser) { Fasterer::FileTraverser.new('.') }
+
+    before do
+      analyzer.scan
+    end
+
+    context "when print offenses" do
+      let(:explanation) { Fasterer::Offense::EXPLANATIONS[:for_loop_vs_each] }
+
+      it 'should print offense' do
+        match = "\e[0;31;49m#{test_file_path}:1\e[0m #{explanation}.\n\n"
+
+        expect { file_traverser.send(:output, analyzer) }.to output(match).to_stdout
+      end
+    end
+  end
 end

data/spec/support/output/sample_code.rb

@@ -0,0 +1,3 @@
+for number in [*1..100] do
+  number
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fasterer
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.5.0
 platform: ruby
 authors:
 - Damir Svrtan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-08 00:00:00.000000000 Z
+date: 2019-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -220,6 +220,7 @@ files:
 - spec/support/method_definition/simple_method.rb
 - spec/support/method_definition/simple_method_omitted_parenthesis.rb
 - spec/support/method_definition/simple_method_with_argument.rb
+- spec/support/output/sample_code.rb
 - spec/support/rescue_call/plain_rescue.rb
 - spec/support/rescue_call/rescue_with_class.rb
 - spec/support/rescue_call/rescue_with_class_and_variable.rb
@@ -325,6 +326,7 @@ test_files:
 - spec/support/method_definition/simple_method.rb
 - spec/support/method_definition/simple_method_omitted_parenthesis.rb
 - spec/support/method_definition/simple_method_with_argument.rb
+- spec/support/output/sample_code.rb
 - spec/support/rescue_call/plain_rescue.rb
 - spec/support/rescue_call/rescue_with_class.rb
 - spec/support/rescue_call/rescue_with_class_and_variable.rb