rails_best_practices 0.3.15 → 0.3.16

data/VERSION

@@ -1 +1 @@
-0.3.15
+0.3.16

data/lib/rails_best_practices.rb

@@ -2,4 +2,51 @@ require 'rails_best_practices/checks'
 require 'rails_best_practices/core'
 
 module RailsBestPractices
+  
+  class <<self
+    def analyze_files(dir = '.', options = {})
+      files = expand_dirs_to_files(dir)
+      files = model_first_sort(files)
+      files = add_duplicate_migrations(files)
+      ['vendor', 'spec', 'test', 'stories'].each do |pattern|
+        files = ignore_files(files, "#{pattern}/") unless options[pattern]
+      end
+      files
+    end
+  
+    def expand_dirs_to_files *dirs
+      extensions = ['rb', 'erb', 'haml', 'builder']
+
+      dirs.flatten.map { |p|
+        if File.directory? p
+          Dir[File.join(p, '**', "*.{#{extensions.join(',')}}")]
+        else
+          p
+        end
+      }.flatten
+    end
+
+    # for law_of_demeter_check
+    def model_first_sort files
+      files.sort { |a, b|
+        if a =~ /models\/.*rb/
+          -1
+        elsif b =~ /models\/.*rb/
+          1
+        else
+          a <=> b
+        end
+      }
+    end
+
+    # for always_add_db_index_check
+    def add_duplicate_migrations files
+      migration_files = files.select { |file| file.index("db/migrate") }
+      (files << migration_files).flatten
+    end
+
+    def ignore_files files, pattern
+      files.reject { |file| file.index(pattern) }
+    end
+  end
 end

data/lib/rails_best_practices/command.rb

@@ -1,39 +1,5 @@
 require 'optparse'
 
-def expand_dirs_to_files *dirs
-  extensions = ['rb', 'erb', 'haml', 'builder']
-
-  dirs.flatten.map { |p|
-    if File.directory? p
-      Dir[File.join(p, '**', "*.{#{extensions.join(',')}}")]
-    else
-      p
-    end
-  }.flatten
-end
-
-# for law_of_demeter_check
-def model_first_sort files
-  files.sort { |a, b|
-    if a =~ /models\/.*rb/
-      -1
-    elsif b =~ /models\/.*rb/
-      1
-    else
-      a <=> b
-    end
-  }
-end
-
-# for always_add_db_index_check
-def add_duplicate_migrations files
-  migration_files = files.select { |file| file.index("db/migrate") }
-  (files << migration_files).flatten
-end
-
-def ignore_files files, pattern
-  files.reject { |file| file.index(pattern) }
-end
 
 options = {}
 OptionParser.new do |opts|
@@ -60,12 +26,7 @@ end
 runner = RailsBestPractices::Core::Runner.new
 runner.set_debug if options['debug']
 
-files = expand_dirs_to_files(ARGV)
-files = model_first_sort(files)
-files = add_duplicate_migrations(files)
-['vendor', 'spec', 'test', 'stories'].each do |pattern|
-  files = ignore_files(files, "#{pattern}/") unless options[pattern]
-end
+files = RailsBestPractices::analyze_files(ARGV, options)
 files.each { |file| runner.check_file(file) }
 
 runner.errors.each {|error| puts error}

data/rails_best_practices.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{rails_best_practices}
-  s.version = "0.3.15"
+  s.version = "0.3.16"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Richard Huang"]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: rails_best_practices
 version: !ruby/object:Gem::Version 
-  version: 0.3.15
+  version: 0.3.16
 platform: ruby
 authors: 
 - Richard Huang