flextures.rb in flextures-3.1.0

- old
+ new

@@ -1,19 +1,18 @@
 # encoding: utf-8
 
 require 'ostruct'
 require 'csv'
-
 require "flextures/flextures_base_config"
 require "flextures/flextures_extension_modules"
 require "flextures/flextures_factory"
 
 module Flextures
-  # ActiveRecord Model is created that guess by table_name
+  # guessing ActiveRecord Model name by table_name and create
   # @params [String|Symbol] table_name
   # @params [ActiveRecord::Base] model class
-  def self.create_model table_name
+  def self.create_model( table_name )
     # when Model is defined in FactoryFilter
     a = ->{
       f = Factory::FACTORIES[table_name.to_sym]
       f && f[:model]
     }
@@ -24,30 +23,30 @@
       rescue => e
         nil
       end
     }
     # when cannot guess Model name
-    c = ->{ 
+    c = ->{
       Class.new(ActiveRecord::Base){ |o| o.table_name=table_name }
     }
     a.call || b.call || c.call
   end
 
   # load configuration file, if file is exist
   def self.init_load
     if defined?(Rails) and Rails.root
       [
-        "#{Rails.root.to_path}/config/flextures.config.rb",
-        "#{Rails.root.to_path}/config/flextures.factory.rb",
-      ].each { |fn| load(fn) if File.exist?(fn) }
+        File.join( Rails.root.to_path,"/config/flextures.config.rb" ),
+        File.join( Rails.root.to_path,"/config/flextures.factory.rb" ),
+      ].each { |fn| File.exist?(fn) && load(fn) }
     end
   end
 
   # @return [Array] flextures useable table names
   def self.deletable_tables
     tables = ActiveRecord::Base.connection.tables
-    Flextures::Config.ignore_tables.each { |name| tables.delete name }
+    Flextures::Config.ignore_tables.each { |name| tables.delete( name.to_s ) }
     tables
   end
 
   # initialize table data
   def self.init_tables
@@ -59,11 +58,11 @@
       rescue => e
       end
     end
   end
 
-  def self.delete_tables *tables
+  def self.delete_tables( *tables )
     tables.each do |name|
       # if 'name' variable is 'database view', raise error
       begin
         Class.new(ActiveRecord::Base){ |o| o.table_name= name }.delete_all
       rescue StandaraError => e
@@ -71,11 +70,11 @@
     end
   end
 
   # It is debug method to use like 'tab' method
   # @params [Proc] dumper write dump information
-  def self.table_tap &dumper
+  def self.table_tap( &dumper )
     tables = Flextures::deletable_tables
     tables.each do |name|
       # if 'name' variable is 'database view', raise error
       begin
         klass = Class.new(ActiveRecord::Base){ |o| o.table_name= name; }
@@ -83,14 +82,14 @@
       rescue StandaraError => e
       end
     end
   end
 
-  # parse arguments functions
+  # parse arguments functions.
   module ARGS
     # parse rake ENV parameters
-    def self.parse option={}
+    def self.parse( option={} )
       table_names = []
       if v = (ENV["T"] or ENV["TABLE"])
         table_names = v.split(',').map{ |name| { table: name, file: name } }
       end
       if v = (ENV["M"] or ENV["MODEL"])
@@ -109,34 +108,33 @@
           [ table_names.first.merge( file: names.first ) ]
         else
           names.map{ |name| { table: name, file: name } }
         end
       }
-      # parse filename define parameters
+      # parse filename and define parameters.
       table_names = fixtures_args_parser.call ENV["FIXTURES"] if ENV["FIXTURES"]
       table_names = fixtures_args_parser.call ENV["FILE"]     if ENV["FILE"]
       table_names = fixtures_args_parser.call ENV["F"]        if ENV["F"]
 
-      table_names = table_names.map{ |option| option.merge dir: ENV["DIR"] } if ENV["DIR"]
-      table_names = table_names.map{ |option| option.merge dir: ENV["D"]   } if ENV["D"]
+      table_names = table_names.map{ |option| option.merge( dir: ENV["DIR"] ) } if ENV["DIR"]
+      table_names = table_names.map{ |option| option.merge( dir: ENV["D"] )   } if ENV["D"]
 
-      table_names = table_names.map{ |option| option.merge minus: ENV["MINUS"].to_s.split(",") } if ENV["MINUS"]
-      table_names = table_names.map{ |option| option.merge plus:  ENV["PLUS"].to_s.split(",")  } if ENV["PLUS"]
+      table_names = table_names.map{ |option| option.merge( minus: ENV["MINUS"].to_s.split(",") ) } if ENV["MINUS"]
+      table_names = table_names.map{ |option| option.merge( plus:  ENV["PLUS"].to_s.split(",") )  } if ENV["PLUS"]
 
-      table_names = table_names.map{ |option| option.merge silent: true }   if ENV["OPTION"].to_s.split(",").include?("silent")
-      table_names = table_names.map{ |option| option.merge unfilter: true } if ENV["OPTION"].to_s.split(",").include?("unfilter")
-      table_names = table_names.map{ |option| option.merge strict: true }   if ENV["OPTION"].to_s.split(",").include?("strict")
-      table_names = table_names.map{ |option| option.merge stair: true }    if ENV["OPTION"].to_s.split(",").include?("stair")
+      table_names = table_names.map{ |option| option.merge( silent: true ) }   if ENV["OPTION"].to_s.split(",").include?("silent")
+      table_names = table_names.map{ |option| option.merge( unfilter: true ) } if ENV["OPTION"].to_s.split(",").include?("unfilter")
+      table_names = table_names.map{ |option| option.merge( strict: true ) }   if ENV["OPTION"].to_s.split(",").include?("strict")
+      table_names = table_names.map{ |option| option.merge( stair: true ) }    if ENV["OPTION"].to_s.split(",").include?("stair")
 
-      # if mode is 'read mode' and file is not exist value is not return
-      table_names.select! &exist if option[:mode] && option[:mode] == 'read'
+      # if mode is 'read mode' and file is not exist, value is not return.
+      table_names.select!(&exist) if option[:mode] && option[:mode] == 'read'
       table_names
     end
 
     # check exist filename block
     def self.exist
       return->(name){ File.exists?( File.join( Config.fixture_load_directory, "#{name}.csv") ) or
                       File.exists?( File.join( Config.fixture_load_directory, "#{name}.yml") ) }
     end
   end
 end
-