mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
CodeStatistic will now ignore hidden files:
- Modify the default regex to not match hidden files
This commit is contained in:
parent
5465224892
commit
ef233bba9a
2 changed files with 15 additions and 2 deletions
|
@ -33,7 +33,7 @@ class CodeStatistics #:nodoc:
|
|||
Hash[@pairs.map{|pair| [pair.first, calculate_directory_statistics(pair.last)]}]
|
||||
end
|
||||
|
||||
def calculate_directory_statistics(directory, pattern = /.*\.(rb|js|coffee|rake)$/)
|
||||
def calculate_directory_statistics(directory, pattern = /^(?!\.).*?\.(rb|js|coffee|rake)$/)
|
||||
stats = CodeStatisticsCalculator.new
|
||||
|
||||
Dir.foreach(directory) do |file_name|
|
||||
|
|
|
@ -4,7 +4,7 @@ require 'rails/code_statistics'
|
|||
class CodeStatisticsTest < ActiveSupport::TestCase
|
||||
def setup
|
||||
@tmp_path = File.expand_path(File.join(File.dirname(__FILE__), 'fixtures', 'tmp'))
|
||||
@dir_js = File.expand_path(File.join(File.dirname(__FILE__), 'fixtures', 'tmp', 'lib.js'))
|
||||
@dir_js = File.join(@tmp_path, 'lib.js')
|
||||
FileUtils.mkdir_p(@dir_js)
|
||||
end
|
||||
|
||||
|
@ -17,4 +17,17 @@ class CodeStatisticsTest < ActiveSupport::TestCase
|
|||
@code_statistics = CodeStatistics.new(['tmp dir', @tmp_path])
|
||||
end
|
||||
end
|
||||
|
||||
test 'ignores hidden files' do
|
||||
File.write File.join(@tmp_path, '.example.rb'), <<-CODE
|
||||
def foo
|
||||
puts 'foo'
|
||||
end
|
||||
CODE
|
||||
|
||||
assert_nothing_raised do
|
||||
CodeStatistics.new(['hidden file', @tmp_path])
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue