undestroy 0.0.1 → 0.0.2

data/test/unit/binding/active_record_test.rb

@@ -0,0 +1,188 @@
+require 'assert'
+
+module Undestroy::Binding::ActiveRecord::Test
+
+  class Base < Undestroy::Test::Base
+    desc 'Binding::ActiveRecord class'
+    subject { Undestroy::Binding::ActiveRecord }
+
+    setup do
+      @model = Class.new(Undestroy::Test::ARMain)
+      @model.table_name = 'foobar'
+      @model.connection.create_table :foobar, :force => true
+      @model.connection.create_table :archive_foobar, :force => true
+    end
+
+    teardown do
+      @model.connection.execute 'drop table if exists foobar'
+      @model.connection.execute 'drop table if exists archive_foobar'
+      Undestroy::Config.instance_variable_set(:@config, nil)
+      Undestroy::Test::Fixtures::Archive.reset
+    end
+  end
+
+  class BasicInstance < Base
+    desc 'basic instance'
+    subject { Undestroy::Binding::ActiveRecord.new @model }
+
+    should have_accessors :config, :model
+  end
+
+  class AddClassMethod < Base
+    desc 'add class method'
+
+    setup do
+      @model.class_eval do
+        undef_method :undestroy_model_binding if respond_to?(:undestroy_model_binding)
+        remove_possible_method :undestroy_model_binding=
+        class << self
+          undef_method :undestroy_model_binding
+          undef_method :undestroy
+        end
+      end
+      @model._destroy_callbacks = []
+    end
+
+    should "add class_attr called `undestroy_model_binding`" do
+      subject.add @model
+      assert_respond_to :undestroy_model_binding, @model
+      assert_respond_to :undestroy_model_binding=, @model
+    end
+
+    should "add undestroy class method to AR::Base initializing this binding" do
+      subject.add @model
+      assert_respond_to :undestroy, @model
+
+      @model.undestroy :fields => {}
+
+      assert_instance_of subject, @model.undestroy_model_binding
+      assert_equal Hash.new, @model.undestroy_model_binding.config.fields
+    end
+
+    should "add before_destroy callback when undestroy called" do
+      subject.add @model
+      archive_class = Undestroy::Test::Fixtures::Archive
+      @model.undestroy :internals => { :archive => archive_class }
+
+      callback = @model._destroy_callbacks.first
+      assert callback, "No destroy callbacks defined"
+      assert_equal :before, callback.kind
+      assert_instance_of Proc, callback.raw_filter
+
+      instance = @model.new
+      instance.instance_eval(&callback.raw_filter)
+      assert_equal [[:run]], archive_class.data[:calls]
+    end
+
+    should "only add callbacks once" do
+      subject.add @model
+      @model.undestroy
+      @model.undestroy
+      assert_equal 1, @model._destroy_callbacks.size
+    end
+
+    should "allow adding to other classes" do
+      new_model = Class.new(@model)
+      subject.add(new_model)
+      assert_respond_to :undestroy_model_binding, new_model
+      assert_not_respond_to :undestroy_model_binding, @model
+    end
+  end
+
+  class InitMethod < Base
+    desc 'init method'
+
+    should "require first argument and set it to model" do
+      assert_raises(ArgumentError) { subject.new }
+      binding = subject.new @model
+      assert_equal @model, binding.model
+    end
+
+    should "validate model argument is AR::Base" do
+      assert_raises(ArgumentError) { subject.new Class.new }
+    end
+
+    should "accept optional hash of config options" do
+      assert_not_raises { subject.new @model, {} }
+    end
+
+    should "set config attr to new config object" do
+      binding = subject.new @model
+      assert_instance_of Undestroy::Config, binding.config
+    end
+
+    should "merge config options onto global config" do
+      Undestroy::Config.configure do |config|
+        config.fields = {}
+      end
+      assert_equal Hash.new, subject.new(@model).config.fields
+      assert_not_equal subject.new(@model).config.object_id, Undestroy::Config.config.object_id
+      assert_equal({ :foo => :bar }, subject.new(@model, :fields => { :foo => :bar }).config.fields)
+    end
+
+    should "set config.source_class to value of model" do
+      binding = subject.new(@model)
+      assert_equal @model, binding.config.source_class
+    end
+
+    should "default :table_name to 'archive_{source.table_name}'" do
+      @model.table_name = :foobar
+      binding = subject.new(@model)
+      assert_equal 'archive_foobar', binding.config.table_name
+    end
+
+    should "create a target_class if none provided" do
+      binding = subject.new(@model)
+      assert binding.config.target_class.ancestors.include?(ActiveRecord::Base)
+    end
+
+    should "use target_class if provided" do
+      target = Class.new(Undestroy::Test::ARAlt)
+      target.table_name = "target_class_test"
+      binding = subject.new(@model, :target_class => target)
+
+      assert_equal target, binding.config.target_class
+      assert_equal 'target_class_test', binding.config.target_class.table_name
+      assert_equal 'tmp/alt.db', binding.config.target_class.connection_config[:database]
+    end
+
+    should "validate target_class is AR::Base if provided" do
+      target = Class.new
+      assert_raises(ArgumentError) { subject.new(@model, :target_class => target) }
+    end
+
+    should "set target class's table_name to :table_name attr" do
+      binding = subject.new(@model, :table_name => "foo_foo_archive")
+      assert_equal "foo_foo_archive", binding.config.target_class.table_name
+    end
+
+    should "set target class's parent class to :abstract_class attr" do
+      @model.table_name = :foobar
+      binding = subject.new(@model, :abstract_class => Undestroy::Test::ARAlt)
+      assert_equal 'tmp/alt.db', binding.config.target_class.connection_config[:database]
+    end
+
+  end
+
+  class BeforeDestroy < Base
+    desc 'before_destroy method'
+    subject { @binding ||= Undestroy::Binding::ActiveRecord.new(@model) }
+
+    should "exist and require one param" do
+      assert_respond_to :before_destroy, subject
+      assert_equal 1, subject.method(:before_destroy).arity
+    end
+
+    should "instantiate config[:archive] instance with config and model instance and call `run`" do
+      test_class = Undestroy::Test::Fixtures::Archive
+      ar_source = Undestroy::Test::Fixtures::ARFixture.new
+      subject.config.internals[:archive] = test_class
+      subject.before_destroy(ar_source)
+
+      assert_equal({ :config => subject.config, :source => ar_source }, test_class.data[:args])
+      assert_equal [[:run]], test_class.data[:calls]
+    end
+  end
+
+end
+

data/test/unit/config_test.rb

@@ -0,0 +1,146 @@
+require 'assert'
+
+class Undestroy::Config::Test
+
+  class Base < Undestroy::Test::Base
+    desc 'Undestroy::Config class'
+    subject { Undestroy::Config }
+
+    teardown do
+      Undestroy::Config.instance_variable_set(:@config, nil)
+    end
+  end
+
+  class ConfigureClassMethod < Base
+    desc 'configure class method'
+
+    should "exist" do
+      assert subject.respond_to?(:configure)
+    end
+
+    should "call the block with an instance of Config" do
+      called = false
+      subject.configure do |object|
+        called = true
+        assert_instance_of subject, object
+      end
+      assert called, "Block was not called"
+    end
+
+    should "store result in global config" do
+      subject.configure do |object|
+        object.table_name = "FOO"
+      end
+      config = subject.instance_variable_get(:@config)
+      assert config
+      assert_equal "FOO", config.table_name
+    end
+  end
+
+  class ConfigClassMethod < Base
+    desc 'return the global Config'
+
+    should "exist" do
+      assert subject.respond_to?(:config)
+    end
+
+    should "return Config object" do
+      assert_instance_of subject, subject.config
+    end
+
+    should "return the same Config object" do
+      assert_equal subject.config.object_id, subject.config.object_id
+    end
+  end
+
+  class BasicInstance < Base
+    desc 'basic instance'
+    subject { Undestroy::Config.new }
+
+    should have_accessors :table_name, :abstract_class, :fields, :migrate
+    should have_accessors :source_class, :target_class, :internals
+  end
+
+  class InitMethod < Base
+    desc 'init method'
+
+    should "default migrate to true" do
+      config = subject.new
+      assert config.migrate
+    end
+
+    should "default fields to delayed deleted_at" do
+      config = subject.new
+      assert_equal [:deleted_at], config.fields.keys
+      assert_instance_of Proc, config.fields[:deleted_at]
+      assert_instance_of Time, config.fields[:deleted_at].call
+      assert Time.now - config.fields[:deleted_at].call < 1
+    end
+
+    should "default internals to internal classes" do
+      config = subject.new
+      assert config.internals
+      assert_instance_of Hash, config.internals
+      assert_equal Undestroy::Archive, config.internals[:archive]
+      assert_equal Undestroy::Transfer, config.internals[:transfer]
+    end
+
+    should "set config options using provided hash" do
+      config = subject.new :table_name => "foo",
+        :abstract_class => "test_archive",
+        :target_class => "foo",
+        :fields => {},
+        :migrate => false
+
+      assert_equal "foo", config.table_name
+      assert_equal "foo", config.target_class
+      assert_equal "test_archive", config.abstract_class
+      assert_equal Hash.new, config.fields
+      assert_equal false, config.migrate
+    end
+
+  end
+
+  class MergeMethod < Base
+    desc 'merge method'
+
+    should "accept config option and return merged config options" do
+      config1 = subject.new :abstract_class => 'foo', :migrate => false
+      config2 = subject.new :abstract_class => 'bar', :fields => {}, :internals => {}
+      config3 = config1.merge(config2)
+
+      assert_equal 'bar', config3.abstract_class
+      assert_equal true, config3.migrate
+      assert_equal Hash.new, config3.fields
+      assert_equal Hash.new, config3.internals
+      assert_equal 'foo', config1.abstract_class
+    end
+  end
+
+
+  class PrimitiveFields < Base
+    desc 'primitive_fields method'
+    subject { @config ||= Undestroy::Config.new }
+
+    should "exist" do
+      assert_respond_to :primitive_fields, subject
+    end
+
+    should "require 1 parameter" do
+      assert_equal 1, subject.method(:primitive_fields).arity
+    end
+
+    should "return fields with any procs evaled" do
+      assert_instance_of Hash, subject.primitive_fields(1)
+      assert_instance_of Time, subject.primitive_fields(1)[:deleted_at]
+    end
+
+    should "pass argument in to proc" do
+      val = {}
+      subject.fields = { :test => proc { |arg| val[:arg] = arg } }
+      subject.primitive_fields("FOOO!")
+      assert_equal "FOOO!", val[:arg]
+    end
+  end
+
+end

data/test/unit/transfer_test.rb

@@ -0,0 +1,65 @@
+require 'assert'
+
+module Undestroy::Transfer::Test
+
+  class Base < Undestroy::Test::Base
+    desc 'Undestroy::Transfer class'
+    subject { Undestroy::Transfer }
+  end
+
+  class BasicInstance < Base
+    subject { Undestroy::Transfer.new :klass => Undestroy::Test::Fixtures::ARFixture }
+    desc 'basic instance'
+
+    should have_accessors :target
+  end
+
+  class InitMethod < Base
+    desc 'init method'
+
+    setup do
+      @fields = {
+        :id => 123,
+        :name => "Foo",
+        :description => "Foo Description"
+      }
+      @transfer = subject.new :klass => Undestroy::Test::Fixtures::ARFixture, :fields => @fields
+    end
+
+    should "raise ArgumentError if no :klass key" do
+      assert_raises(ArgumentError) { subject.new }
+    end
+
+    should "default :fields to empty hash" do
+      assert_not_raises { subject.new :klass => Undestroy::Test::Fixtures::ARFixture }
+    end
+
+    should "create :klass instance with :fields" do
+      assert_instance_of Undestroy::Test::Fixtures::ARFixture, @transfer.target
+      assert_equal @fields[:id], @transfer.target[:id]
+      assert_equal @fields[:name], @transfer.target[:name]
+      assert_equal @fields[:description], @transfer.target[:description]
+    end
+
+  end
+
+  class RunMethod < Base
+    desc 'run method'
+
+    setup do
+      @fields = {
+        :id => 1,
+        :name => "Bar",
+        :a_field => "some_thing"
+      }
+      @transfer = subject.new :klass => Undestroy::Test::Fixtures::ARFixture, :fields => @fields
+    end
+
+    should "call save on built record" do
+      @transfer.run
+      self.assert @transfer.target.saved?
+    end
+  end
+
+end
+

data/undestroy.gemspec

@@ -1,5 +1,6 @@
 # -*- encoding: utf-8 -*-
-require File.expand_path('../lib/undestroy/version', __FILE__)
+$: << File.expand_path('../lib', __FILE__)
+require 'undestroy/version'
 
 Gem::Specification.new do |gem|
   gem.authors       = ["Travis Petticrew"]

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: undestroy
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 27
   prerelease: 
   segments: 
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  - 2
+  version: 0.0.2
 platform: ruby
 authors: 
 - Travis Petticrew
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-12 00:00:00 Z
+date: 2012-03-15 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   type: :runtime
@@ -73,14 +73,30 @@ extra_rdoc_files: []
 
 files: 
 - .gitignore
+- ARCH.md
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - lib/undestroy.rb
+- lib/undestroy/archive.rb
+- lib/undestroy/binding.rb
+- lib/undestroy/binding/active_record.rb
+- lib/undestroy/config.rb
+- lib/undestroy/transfer.rb
 - lib/undestroy/version.rb
+- lib/undestroy/with_binding.rb
+- lib/undestroy/without_binding.rb
+- test/fixtures/active_record_models.rb
+- test/fixtures/ar.rb
+- test/fixtures/archive.rb
 - test/helper.rb
-- test/unit/undestroy_test.rb
+- test/integration/active_record_test.rb
+- test/unit/archive_test.rb
+- test/unit/binding/active_record_test.rb
+- test/unit/config_test.rb
+- test/unit/transfer_test.rb
+- tmp/.gitkeep
 - undestroy.gemspec
 homepage: ""
 licenses: []
@@ -116,5 +132,12 @@ signing_key:
 specification_version: 3
 summary: Allow AR records to be undestroyed by archiving their data in a seperate table when destroying the original data
 test_files: 
+- test/fixtures/active_record_models.rb
+- test/fixtures/ar.rb
+- test/fixtures/archive.rb
 - test/helper.rb
-- test/unit/undestroy_test.rb
+- test/integration/active_record_test.rb
+- test/unit/archive_test.rb
+- test/unit/binding/active_record_test.rb
+- test/unit/config_test.rb
+- test/unit/transfer_test.rb