file_column_with_s3 0.1.1 → 0.1.2

data/Rakefile

@@ -1,4 +1,4 @@
-require 'bundler'  
+require 'bundler'
 Bundler::GemHelper.install_tasks
 
 task :default => [:test]
@@ -31,8 +31,8 @@ task :package => [:checkout_release, :release_docs] do
   sh "cd release; tar czf #{PKG_NAME}-#{PKG_VERSION}.tar.gz #{PKG_NAME}-#{PKG_VERSION}"
 end
 
+desc "Run all tests"
 task :test do
-  sh "cd test; ruby attachement_store_test.rb"
   sh "cd test; ruby file_column_test.rb"
   sh "cd test; ruby file_column_helper_test.rb"
   sh "cd test; ruby magick_test.rb"

data/lib/file_column.rb

@@ -1,7 +1,7 @@
 require 'fileutils'
 require 'tempfile'
 require 'magick_file_column'
-require 'file_column/attachement_store'
+require 'storage'
 
 module FileColumn # :nodoc:
   def self.append_features(base)
@@ -18,20 +18,22 @@ module FileColumn # :nodoc:
     end
   end
 
-  #todo: dir is not required for all type of store
-  def self.store(dir)
-    (@store_builder || AttachementStore::Builder.new(:filesystem)).build(dir)
+  def self.store(path_prefix, extra_opts={})
+    builder = @store_builder || Storage::Builder.new(:filesystem)
+    @build_opts ||= {}
+    builder.build(path_prefix, @build_opts.merge(extra_opts))
   end
 
-  def self.store=(args)
-    @store_builder = AttachementStore::Builder.new(*args)
+  def self.config_store(type, build_opts={})
+    @build_opts = build_opts
+    @store_builder = Storage::Builder.new(type)
   end
 
   def self.init_options(defaults, model, attr)
     options = defaults.dup
-    options[:store_dir] ||= File.join(options[:root_path], model, attr)
+    options[:store_dir] ||= File.join(model, attr)
     unless options[:store_dir].is_a?(Symbol)
-      options[:tmp_base_dir] ||= File.join(options[:store_dir], "tmp")
+      options[:tmp_base_dir] ||= File.join(options[:root_path], options[:store_dir], "tmp")
     end
     options[:base_url] ||= options[:web_root] + File.join(model, attr)
 
@@ -51,6 +53,9 @@ module FileColumn # :nodoc:
       @options_method = "#{attr}_options".to_sym
     end
 
+    def store
+      nil
+    end
 
     def assign(file)
       if file.is_a? File
@@ -122,10 +127,7 @@ module FileColumn # :nodoc:
     def store_dir
       if options[:store_dir].is_a? Symbol
         raise ArgumentError.new("'#{options[:store_dir]}' is not an instance method of class #{@instance.class.name}") unless @instance.respond_to?(options[:store_dir])
-
-        dir = File.join(options[:root_path], @instance.send(options[:store_dir]))
-        FileUtils.mkpath(dir) unless File.exists?(dir)
-        dir
+        @instance.send(options[:store_dir])
       else
         options[:store_dir]
       end
@@ -135,7 +137,7 @@ module FileColumn # :nodoc:
       if options[:tmp_base_dir]
         options[:tmp_base_dir]
       else
-        dir = File.join(store_dir, "tmp")
+        dir = File.join(options[:root_path], store_dir, "tmp")
         FileUtils.mkpath(dir) unless File.exists?(dir)
         dir
       end
@@ -223,7 +225,8 @@ module FileColumn # :nodoc:
     def store_upload(file)
       @tmp_dir = FileColumn.generate_temp_name
       @dir = File.join(tmp_base_dir, @tmp_dir)
-      FileUtils.mkdir(@dir)
+
+      FileUtils.mkdir_p(@dir)
 
       @filename = FileColumn::sanitize_filename(file.original_filename)
       local_file_path = File.join(tmp_base_dir,@tmp_dir,@filename)
@@ -356,22 +359,24 @@ module FileColumn # :nodoc:
   class PermanentUploadedFile < RealUploadedFile # :nodoc:
     def initialize(*args)
       super *args
-      @store = FileColumn.store(store_dir)
       @filename = @instance[@attr]
       @filename = nil if @filename.empty?
     end
 
+    def store
+      FileColumn.store(store_dir, {:root_path => options[:root_path] }) #root_path is only need for file system store, but pass it in will not hurt for other store
+    end
+
     def absolute_path(subdir=nil)
       if subdir
-        @store.absolute_path(File.join(relative_path_prefix, subdir, @filename))
+        store.absolute_path(File.join(relative_path_prefix, subdir, @filename))
       else
-        @store.absolute_path(File.join(relative_path_prefix, @filename))
+        store.absolute_path(File.join(relative_path_prefix, @filename))
       end
     end
 
     def move_from(local_dir, just_uploaded)
-      @store.upload_dir(relative_path_prefix, local_dir)
-
+      store.upload_dir(relative_path_prefix, local_dir)
       @just_uploaded = just_uploaded
     end
 
@@ -390,7 +395,6 @@ module FileColumn # :nodoc:
 
     def assign_temp(temp_path)
       return nil if temp_path.nil? or temp_path.empty?
-
       temp = clone_as TempUploadedFile
       temp.parse_temp_path(temp_path)
       temp
@@ -401,7 +405,7 @@ module FileColumn # :nodoc:
     end
 
     def delete_files
-      @store.clear
+      store.delete(relative_path_prefix)
     end
 
     private
@@ -541,7 +545,7 @@ module FileColumn # :nodoc:
   #
   # For setting a static storage_dir that doesn't change with respect to a particular
   # instance, you assign <tt>:storage_dir</tt> a String representing a directory
-  # as an absolute path.
+  # as a relative path under root_path.
   #
   # If you need more fine-grained control over the storage directory, you
   # can use the name of a callback-method as a symbol for the
@@ -642,13 +646,13 @@ module FileColumn # :nodoc:
         result
       end
 
+      private state_method
+
       define_method "file_column_relative_path_prefix" do
         raise RuntimeError.new("Trying to access file_column, but primary key got lost.") if self.id.to_s.empty?
-        File.join(*("%08d" % self.id).scan(/..../))
+        options[:use_safe_path] ? File.join(*("%08d" % self.id).scan(/..../)) : self.id.to_s
       end
 
-      private state_method
-
       define_method attr do |*args|
         send(state_method).absolute_path *args
       end
@@ -702,6 +706,7 @@ module FileColumn # :nodoc:
         send(state_method).just_uploaded?
       end
 
+
       # this creates a closure keeping a reference to my_options
       # right now that's the only way we store the options. We
       # might use a class attribute as well
@@ -709,6 +714,30 @@ module FileColumn # :nodoc:
         my_options
       end
 
+      define_method "#{attr}_download_url" do |context_path, *args|
+        state = send(state_method)
+        relative_path = state.relative_path(*args)
+        return nil unless relative_path
+
+        store = state.store
+        return store.url_for(relative_path) if store && store.respond_to?(:url_for)
+
+        url = ""
+        url << context_path if context_path
+        url << "/"
+        url << state.options[:base_url] << "/"
+        url << relative_path
+      end
+
+      define_method "#{attr}_copy_to" do |local_path, *args|
+        state = send(state_method)
+        relative_path = state.relative_path(*args)
+        return nil unless relative_path
+        store = state.store
+        FileUtils.mkdir_p(File.dirname(local_path))
+        store.copy(relative_path, local_path)
+      end
+
       private after_save_method, after_destroy_method
 
       FileColumn::MagickExtension::file_column(self, attr, my_options) if options[:magick]
@@ -725,7 +754,8 @@ module FileColumn # :nodoc:
 
   def self.sanitize_filename(filename)
     filename = File.basename(filename.gsub("\\", "/")) # work-around for IE
-    filename.gsub!(/[^a-zA-Z0-9\.\-\+_]/,"_")
+    # cannot use \w or \W since Rails enables unicode $KCODE flag
+    filename.gsub!(/[^A-Za-z0-9_\-\.]/, "_")
     filename = "_#{filename}" if filename =~ /^\.+$/
     filename = "unnamed" if filename.size == 0
     filename

data/lib/file_column_helper.rb

@@ -3,7 +3,7 @@
 # automatically included into ActionView::Base, thereby making this module's
 # methods available in all your views.
 module FileColumnHelper
-  
+
   # Use this helper to create an upload field for a file_column attribute. This will generate
   # an additional hidden field to keep uploaded files during form-redisplays. For example,
   # when called with
@@ -26,7 +26,7 @@ module FileColumnHelper
     result = ActionView::Helpers::InstanceTag.new(object.dup, method.to_s+"_temp", self).to_input_field_tag("hidden", {})
     result << ActionView::Helpers::InstanceTag.new(object.dup, method, self).to_input_field_tag("file", options)
   end
-  
+
   # Creates an URL where an uploaded file can be accessed. When called for an Entry object with
   # id 42 (stored in <tt>@entry</tt>) like this
   #
@@ -71,15 +71,10 @@ module FileColumnHelper
         subdir = options
       end
     end
-    
-    relative_path = object.send("#{method}_relative_path", subdir)
-    return nil unless relative_path
 
-    url = ""
-    url << ActionController::Base.relative_url_root.to_s if absolute
-    url << "/"
-    url << object.send("#{method}_options")[:base_url] << "/"
-    url << relative_path
+    context_path = absolute ? ActionController::Base.relative_url_root.to_s : nil
+    object.send("#{method}_download_url", context_path, subdir)
+
   end
 
   # Same as +url_for_file_colum+ but allows you to access different versions
@@ -115,7 +110,7 @@ module FileColumnHelper
   #
   # and
   #
-  #    <%= url_for_image_column @entry, "image", 
+  #    <%= url_for_image_column @entry, "image",
   #       :size => "50x50", :crop => "1:1", :name => "thumb" %>
   #
   # will produce something like this:

data/lib/magick_file_column.rb

@@ -17,7 +17,7 @@ module FileColumn # :nodoc:
           options[:magick][:versions].each_pair do |version, version_options|
             next if version_options[:lazy]
             dirname = version_options[:name]
-            FileUtils.mkdir File.join(@dir, dirname)
+            FileUtils.mkdir_p File.join(@dir, dirname)
             transform_image(img, version_options, absolute_path(dirname))
           end
         end
@@ -55,7 +55,7 @@ module FileColumn # :nodoc:
           return nil
         end
         dirname = version_options[:name]
-        FileUtils.mkdir File.join(@dir, dirname)
+        FileUtils.mkdir_p File.join(@dir, dirname)
         transform_image(img, version_options, absolute_path(dirname))
       end
 

data/s3_env.example

@@ -1,5 +1,5 @@
 #!/bin/sh
 
-export S3_ACCESS_KEY_ID=<key-id>
-export S3_SECRET_ACCESS_KEY=<access-key>
+export AWS_ACCESS_KEY_ID=<key-id>
+export AWS_SECRET_ACCESS_KEY=<access-key>
 export S3_BUCKET_NAME=file-column-attachement-test

data/test/abstract_unit.rb

@@ -64,3 +64,39 @@ class Test::Unit::TestCase
 
   alias_method :f, :file_path
 end
+
+# provid a dummy storage implementation for tests
+
+class InMemoryWithUrlStorageStore
+  def initialize(path_prefix, options)
+    @path_prefix = path_prefix
+    @storage = {}
+  end
+  def upload(path, file)
+    @storage[absolute_path(path) + "/" + File.basename(file)] = File.read(file)
+  end
+
+  def upload_dir(path, local_dir)
+    Dir[File.join(local_dir, "*")].each do |f|
+      upload(path, f)
+    end
+  end
+
+  def exists?(path)
+    File.key?(path)
+  end
+
+  def url_for(path)
+    "store generated url for #{path}"
+  end
+
+  def absolute_path(path)
+    File.join(@path_prefix, path)
+  end
+
+  def clear
+    @storage = {}
+  end
+end
+
+Storage.add_store_class(:in_memory_with_url, InMemoryWithUrlStorageStore)

data/test/file_column_helper_test.rb

@@ -6,9 +6,19 @@ class UrlForFileColumnTest < Test::Unit::TestCase
 
   def setup
     Entry.file_column :image
+    ActionController::Base.relative_url_root = nil
     @request = RequestMock.new
   end
 
+  def test_should_use_store_generated_url_if_file_store_configured_can_do_it
+    FileColumn.config_store(:in_memory_with_url)
+    @e = Entry.new(:image => upload(f("skanthak.png")))
+    assert @e.save
+    assert_equal "store generated url for #{@e.image_relative_path}", url_for_file_column("e", "image")
+  ensure
+    FileColumn.config_store(:filesystem)
+  end
+
   def test_url_for_file_column_with_temp_entry
     @e = Entry.new(:image => upload(f("skanthak.png")))
     url = url_for_file_column("e", "image")
@@ -26,7 +36,6 @@ class UrlForFileColumnTest < Test::Unit::TestCase
   def test_url_for_file_column_works_with_symbol
     @e = Entry.new(:image => upload(f("skanthak.png")))
     assert @e.save
-
     url = url_for_file_column(:e, :image)
     assert_equal "/entry/image/#{@e.file_column_relative_path_prefix}/skanthak.png", url
   end
@@ -52,7 +61,7 @@ class UrlForFileColumnTest < Test::Unit::TestCase
   end
 end
 
-class UrlForFileColumnTest < Test::Unit::TestCase
+class UrlForFileColumnWithContextPathTest < Test::Unit::TestCase
   include FileColumnHelper
   include ActionView::Helpers::AssetTagHelper
   include ActionView::Helpers::TagHelper
@@ -75,7 +84,8 @@ class UrlForFileColumnTest < Test::Unit::TestCase
   def test_with_image_tag
     e = Entry.new(:image => upload(f("skanthak.png")))
     html = image_tag url_for_file_column(e, "image")
-    url = html.scan(/src=\"(.+)\?.*\"/).first.first
+
+    url = html.scan(/src=\"([^?]+)\?*.*\"/).first.first
 
     assert_match IMAGE_URL, url
   end

data/test/file_column_test.rb

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 require File.dirname(__FILE__) + '/abstract_unit'
 
 require File.dirname(__FILE__) + '/fixtures/entry'
@@ -39,12 +40,14 @@ class FileColumnTest < Test::Unit::TestCase
     assert_equal "__foo", FileColumn::sanitize_filename('`*foo')
     assert_equal "foo.txt", FileColumn::sanitize_filename('c:\temp\foo.txt')
     assert_equal "_.", FileColumn::sanitize_filename(".")
+    assert_equal "a_b.jpg", FileColumn::sanitize_filename("a+b.jpg")
   end
 
+
   def test_default_options
     e = Entry.new
-    assert_match %r{/public/entry/image}, e.image_options[:store_dir]
-    assert_match %r{/public/entry/image/tmp}, e.image_options[:tmp_base_dir]
+    assert_match %r{entry/image}, e.image_options[:store_dir]
+    assert_match %r{entry/image/tmp}, e.image_options[:tmp_base_dir]
   end
 
   def test_assign_without_save_with_tempfile
@@ -582,6 +585,13 @@ class FileColumnTest < Test::Unit::TestCase
     e["id"] = nil
     assert_raise(RuntimeError) { e.image }
   end
+
+  def test_copy_file_to_local_dir
+    e = Entry.new(:image => upload(f("skanthak.png")))
+    assert e.save
+    e.image_copy_to('/tmp/file_column_test/copy_dir/skanthak.png')
+    assert File.exist?('/tmp/file_column_test/copy_dir/skanthak.png')
+  end
 end
 
 # Tests for moving temp dir to permanent dir
@@ -650,3 +660,43 @@ class FileColumnMoveTest < Test::Unit::TestCase
   end
 
 end
+
+
+# Tests for moving temp dir to permanent dir
+class FileColumnDownloadUrlTest < Test::Unit::TestCase
+
+
+  def test_default_download_url_is_relative
+    Entry.file_column :image
+    e = Entry.new(:image => upload(f("skanthak.png")))
+    assert e.save
+
+    assert_equal "/entry/image/#{e.id}/skanthak.png", e.image_download_url(nil)
+  end
+
+  def test_download_url_with_specified_url_base
+    Entry.file_column :image, :base_url => 'foo'
+    e = Entry.new(:image => upload(f("skanthak.png")))
+    assert e.save
+
+    assert_equal "/foo/#{e.id}/skanthak.png", e.image_download_url(nil)
+  end
+
+  def test_download_url_with_specified_subdir
+    Entry.file_column :image
+    e = Entry.new(:image => upload(f("skanthak.png")))
+    assert e.save
+
+    assert_equal "/entry/image/#{e.id}/foo/skanthak.png", e.image_download_url(nil, 'foo')
+  end
+
+
+  def test_should_use_store_generated_url_if_storage_configured_can_do_it
+    FileColumn.config_store(:in_memory_with_url)
+    e = Entry.new(:image => upload(f("skanthak.png")))
+    assert e.save
+    assert_equal "store generated url for #{e.image_relative_path}", e.image_download_url(nil)
+  ensure
+    FileColumn.config_store(:filesystem)
+  end
+end

data/test/magick_test.rb

@@ -1,5 +1,4 @@
 begin
-  require 'RMagick'
   require File.dirname(__FILE__) + '/abstract_unit'
   require File.dirname(__FILE__) + '/fixtures/entry'
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: file_column_with_s3
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 31
   prerelease: 
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - Mingle SaaS team
@@ -15,13 +15,31 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-09-20 00:00:00 Z
+date: 2013-08-19 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  version_requirements: &id001 !ruby/object:Gem::Requirement 
+  name: mingle-storage
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 29
+        segments: 
+        - 0
+        - 0
+        - 1
+        version: 0.0.1
+  type: :runtime
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: rails
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ~>
       - !ruby/object:Gem::Version 
         hash: 19
         segments: 
@@ -29,12 +47,12 @@ dependencies:
         - 3
         - 8
         version: 2.3.8
-  prerelease: false
   type: :runtime
-  name: rails
-  requirement: *id001
+  version_requirements: *id002
 - !ruby/object:Gem::Dependency 
-  version_requirements: &id002 !ruby/object:Gem::Requirement 
+  name: sqlite3
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -43,12 +61,12 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  prerelease: false
   type: :development
-  name: sqlite3
-  requirement: *id002
+  version_requirements: *id003
 - !ruby/object:Gem::Dependency 
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  name: rmagick
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -57,10 +75,22 @@ dependencies:
         segments: 
         - 0
         version: "0"
+  type: :development
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
+  name: aws-sdk
   prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
   type: :development
-  name: rmagick
-  requirement: *id003
+  version_requirements: *id005
 description: 
 email: mingle.saas@thoughtworks.com
 executables: []
@@ -71,7 +101,6 @@ extra_rdoc_files: []
 
 files: 
 - Rakefile
-- lib/file_column/attachement_store.rb
 - lib/file_column.rb
 - lib/file_column_helper.rb
 - lib/file_compat.rb
@@ -80,7 +109,6 @@ files:
 - lib/test_case.rb
 - lib/validations.rb
 - test/abstract_unit.rb
-- test/attachement_store_test.rb
 - test/connection.rb
 - test/file_column_helper_test.rb
 - test/file_column_test.rb
@@ -124,7 +152,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: File attachment library for ruby

data/lib/file_column/attachement_store.rb

@@ -1,93 +0,0 @@
-module FileColumn
-  module AttachementStore
-    begin
-      require 'aws-sdk'
-      class S3Store
-        def initialize(options)
-          s3 = AWS::S3.new(:access_key_id => options[:access_key_id],
-                           :secret_access_key => options[:secret_access_key])
-          @bucket = s3.buckets[options[:bucket_name]]
-        end
-
-        def upload(path, local_file)
-          @bucket.objects.create('/' + path + "/" + File.basename(local_file), File.read(local_file))
-        end
-
-        def upload_dir(path, local_dir)
-          @bucket.objects.with_prefix("/" + path).delete_all
-          Dir[File.join(local_dir, "*")].each do |f|
-            upload(path, f)
-          end
-        end
-
-        def read(path)
-          @bucket.objects["/" + path ].read
-        end
-
-        def exists?(path)
-          @bucket.objects['/' + path].exists?
-        end
-
-        def clear
-          @bucket.clear!
-        end
-      end
-    rescue LoadError => e
-      puts "Warning: can not load aws-sdk gem, s3 file store disabled"
-    end
-
-    class FilesystemStore
-      def initialize(dir)
-        @dir = dir
-        FileUtils.mkdir_p @dir
-      end
-
-      def read(path)
-        File.read(absolute_path(path))
-      end
-
-      def upload(path, local_file)
-        FileUtils.mkdir_p(absolute_path(path))
-        FileUtils.mv(local_file, absolute_path(path))
-      end
-
-      def upload_dir(path, local_dir)
-        FileUtils.rm_rf(absolute_path(path))
-        Dir[File.join(local_dir, "*")].each do |f|
-          upload(path, f)
-        end
-      end
-
-      #todo: this should be interface that retrive a lazy file object
-      def absolute_path(*relative_paths)
-        File.join(@dir, *relative_paths)
-      end
-
-      def exists?(file_path)
-        File.exists?(absolute_path(file_path))
-      end
-
-      def clear
-        FileUtils.rm_rf @dir
-      end
-    end
-
-    class Builder
-      def initialize(*build_opts)
-        @type, *@build_opts = *build_opts
-      end
-
-      def build(dir=nil)
-        args = @build_opts
-        args += [dir] if @type == :filesystem
-        store_class.new(*args)
-      end
-
-      private
-
-      def store_class
-        ActiveSupport::Inflector.constantize("FileColumn::AttachementStore::#{ActiveSupport::Inflector.camelize(@type)}Store")
-      end
-    end
-  end
-end

data/test/attachement_store_test.rb

@@ -1,96 +0,0 @@
-require File.dirname(__FILE__) + '/abstract_unit'
-require 'active_support/test_case'
-
-class AttachementStoreTest < Test::Unit::TestCase
-  extend Test::Unit::Assertions
-
-  STORE_DIR = File.dirname(__FILE__)+"/public/entry"
-  STORE_BUILD_OPTS = [[:filesystem]]
-  if !ENV["S3_ACCESS_KEY_ID"].blank?
-    STORE_BUILD_OPTS << [:s3, {
-                           :access_key_id => ENV["S3_ACCESS_KEY_ID"],
-                           :secret_access_key => ENV["S3_SECRET_ACCESS_KEY"],
-                           :bucket_name => ENV["S3_BUCKET_NAME"]}]
-  end
-
-  def teardown
-    FileColumn.store(STORE_DIR).clear
-    FileUtils.rm_rf("/tmp/file_column_test")
-  end
-
-  def self.store_test(test_name, store_type, *store_building_args, &block)
-    define_method(test_name + "_for_#{store_type}_store") do
-      FileColumn.store = store_type, *store_building_args
-      yield
-    end
-  end
-
-
-  STORE_BUILD_OPTS.each do |store_type, *rest_args|
-    store_test "test_build_right_store", store_type, *rest_args do
-      assert  FileColumn.store("/tmp/attachements").class.name.include?(ActiveSupport::Inflector.camelize(store_type))
-    end
-
-    store_test "test_upload_local_file", store_type, *rest_args do
-      file =  "/tmp/file_column_test/abc"
-      FileUtils.mkdir_p(File.dirname(file))
-      FileUtils.touch(file)
-      store = FileColumn.store(STORE_DIR)
-      store.upload("x/y/z", file)
-      assert !store.exists?("x/abc")
-      assert store.exists?("x/y/z/abc")
-      assert_equal "", store.read("x/y/z/abc")
-    end
-
-    store_test "test_upload_with_same_name_replace_file", store_type, *rest_args do
-      file =  "/tmp/file_column_test/abc"
-      FileUtils.mkdir_p(File.dirname(file))
-      File.open(file, "w+") { |f| f << "123" }
-
-      store = FileColumn.store(STORE_DIR)
-      store.upload("x/y/z", file)
-
-      assert_equal "123", store.read("x/y/z/abc")
-
-      File.open(file, "w+") { |f| f << "456" }
-      store.upload("x/y/z", file)
-
-      assert_equal "456", store.read("x/y/z/abc")
-    end
-
-    store_test "test_upload_local_dir", store_type, *rest_args do
-      local_dir = "/tmp/file_column_test"
-      FileUtils.mkdir_p(local_dir)
-      FileUtils.touch(File.join(local_dir, "a"))
-      FileUtils.touch(File.join(local_dir, "b"))
-
-      store = FileColumn.store(STORE_DIR)
-      store.upload_dir("x/y/z", local_dir)
-
-      assert store.exists?("x/y/z/a")
-      assert store.exists?("x/y/z/b")
-    end
-
-
-    store_test "test_upload_local_dir_with_replace_files", store_type, *rest_args do
-
-      local_dir = "/tmp/file_column_test/old"
-      FileUtils.mkdir_p(local_dir)
-      FileUtils.touch(File.join(local_dir, "a"))
-
-      store = FileColumn.store(STORE_DIR)
-      store.upload_dir("x/y/z", local_dir)
-
-      local_dir = "/tmp/file_column_test/new"
-      FileUtils.mkdir_p(local_dir)
-      FileUtils.touch(File.join(local_dir, "b"))
-
-      store = FileColumn.store(STORE_DIR)
-      store.upload_dir("x/y/z", local_dir)
-
-      assert store.exists?("x/y/z/b")
-      assert !store.exists?("x/y/z/a")
-    end
-
-  end
-end