migratrix 0.0.7 → 0.0.9

data/lib/migratrix/exceptions.rb

@@ -4,5 +4,6 @@ module Migratrix
   class MigrationAlreadyExists < Exception; end
   class MigrationFileNotFound < Exception; end
   class MigrationNotDefined < Exception; end
+  class ExtractorSourceUndefined < Exception; end
 end
 

data/lib/migratrix/extractors/active_record.rb

@@ -0,0 +1,54 @@
+module Migratrix
+  module Extractors
+    # Extractor that expects to be pointed at an ActiveRecord class.
+    class ActiveRecord < Extractor
+      # TODO: Raise TypeError in initialize unless Extractor source is
+      # an ActiveRecord model
+
+      def source=(new_source)
+        raise TypeError.new(":source is of type must be an ActiveRecord model class (must inherit from ActiveRecord::Base)") unless is_ar?(new_source)
+        @source = new_source
+      end
+
+      def is_ar?(source)
+        source.is_a?(Class) && source.ancestors.include?(::ActiveRecord::Base)
+      end
+
+      def obtain_source(source)
+        raise ExtractorSourceUndefined unless source
+        raise TypeError.new(":source is of type must be an ActiveRecord model class (must inherit from ActiveRecord::Base)") unless is_ar?(source)
+        source
+      end
+
+      def handle_where(source, clause)
+        source.where(clause)
+      end
+
+      def handle_limit(source, clause)
+        source.limit(clause.to_i)
+      end
+
+      def handle_offset(source, clause)
+        source.offset(clause.to_i)
+      end
+
+      def handle_order(source, clause)
+        source.order(clause)
+      end
+
+      # Constructs the query
+      def to_query(source)
+        if source.is_a?(::ActiveRecord::Relation)
+          source.to_sql
+        else
+          source.handle_where(1).to_sql
+        end
+      end
+
+      def execute_extract(source)
+        source.all
+      end
+    end
+  end
+end
+

data/lib/migratrix/extractors/extractor.rb

@@ -0,0 +1,88 @@
+#require 'active_model/attribute_methods'
+
+module Migratrix
+  module Extractors
+    # base class for extraction
+    class Extractor
+      include ::Migratrix::Loggable
+#      include ActiveModel::AttributeMethods
+
+      attr_accessor :source, :options
+
+      def initialize(options={})
+        @options = options.deep_copy
+        self.source = options[:source] if options[:source]
+      end
+
+      def extract(options={})
+        options = @options.merge(options)
+
+        # TODO: Raise error if self.abstract? DANGER/NOTE that this is
+        # the "default strategy" for extraction, and may need to be
+        # extracted to a strategy object.
+
+        src = obtain_source(self.source)
+        src = handle_where(src, options["where"]) if options["where"]
+        src = handle_order(src, options["order"]) if options["order"]
+        src = handle_limit(src, options["limit"]) if options["limit"]
+        src = handle_offset(src, options["offset"]) if options["offset"]
+        execute_extract(src)
+      end
+
+      # = extraction filter methods
+      #
+      # The handle_* methods receive a source and return a source and
+      # must be chainable. For example, source might come in as an
+      # ActiveRecord::Base class (or as an ActiveRecord::Relation),
+      # and it will be returned as an ActiveRecord::Relation, which
+      # will not be expanded into an actual resultset until the
+      # execute_extract step is called. On the other hand, for a csv
+      # reader, the first self.source call might read all the rows of
+      # the table and hold them in memory, the where clause might
+      # filter it, the order clause might sort it, and the
+      # execute_extract might simply be a no-op returning the
+      # processed results.
+
+      # First step in extraction is to take the given source and turn
+      # it into something that the filter chain can used. The
+      # ActiveRecord extractor uses a legacy model class as its source
+      # so it can simply return its source. A CSV or Yaml extractor
+      # here might need to read the entire file contents and returns
+      # the full, unfiltered data source.
+      def obtain_source(source)
+        raise NotImplementedError
+      end
+
+      # Apply where clause to source, return new source.
+      def handle_where(source)
+        raise NotImplementedError
+      end
+
+      # Apply limit clause to source, return new source.
+      def handle_limit(source)
+        raise NotImplementedError
+      end
+
+      # Apply offset clause to source, return new source.
+      def handle_offset(source)
+        raise NotImplementedError
+      end
+
+      # Apply order clause to source, return new source.
+      def handle_order(source)
+        raise NotImplementedError
+      end
+
+      # Constructs the query, if applicable. May not exist or make
+      # sense for non-SQL and/or non-ActiveRecord extractors.
+      def to_query(source)
+        raise NotImplementedError
+      end
+
+      # Execute the extraction and return the result set.
+      def execute_extract(source)
+        raise NotImplementedError
+      end
+    end
+  end
+end

data/lib/migratrix/loggable.rb

@@ -7,12 +7,45 @@ module Migratrix
       def logger
         ::Migratrix::Migratrix.logger
       end
+      # If you skip the logger object, your class name will be prepended to the message.
+      def info(msg)
+        logger.info("#{self}: #{msg}")
+      end
+      def debug(msg)
+        logger.debug("#{self}: #{msg}")
+      end
+      def warn(msg)
+        logger.warn("#{self}: #{msg}")
+      end
+      def error(msg)
+        logger.error("#{self}: #{msg}")
+      end
+      def fatal(msg)
+        logger.fatal("#{self}: #{msg}")
+      end
     end
 
     module InstanceMethods
       def logger
         ::Migratrix::Migratrix.logger
       end
+
+      # If you skip the logger object, your class name will be prepended to the message.
+      def info(msg)
+        logger.info("#{self.class}: #{msg}")
+      end
+      def debug(msg)
+        logger.debug("#{self.class}: #{msg}")
+      end
+      def warn(msg)
+        logger.warn("#{self.class}: #{msg}")
+      end
+      def error(msg)
+        logger.error("#{self.class}: #{msg}")
+      end
+      def fatal(msg)
+        logger.fatal("#{self.class}: #{msg}")
+      end
     end
   end
 end

data/lib/migratrix/migration.rb

@@ -1,19 +1,40 @@
+require 'active_model/attribute_methods'
+
 module Migratrix
   # Superclass for all migrations. Migratrix COULD check to see that a
   # loaded migration inherits from this class, but hey, duck typing.
   class Migration
     include ::Migratrix::Loggable
+    include ActiveModel::AttributeMethods
 
+    cattr_accessor :extractor
     attr_accessor :options
 
     def initialize(options={})
-      # cannot make a deep copy of an IO stream (e.g. logger) so make a shallow copy of it and move it out of the way
       @options = options.deep_copy
     end
 
-    # Load this data from source
+    # Adds an extractor to the extractors chain.
+    def self.set_extractor(name, options={})
+      # klassify this name
+      raise NotImplementedError.new("Migratrix currently only supports ActiveRecord extractor.") unless name == :active_record
+      @@extractor = ::Migratrix::Extractors::ActiveRecord.new(options)
+    end
+
+    def extractor
+      @@extractor
+    end
+
+    # OKAY, NEW RULE: You get ONE Extractor per Migration. You're
+    # allowed to have multiple transform/load chains to the
+    # extraction, but extractors? ONE.
+
+    # default extraction method; simply assigns @extractor.extract to
+    # @extracted_items. If you override this method, you should
+    # populate @extracted_items if you want the default transform
+    # and/or load to work correctly.
     def extract
-      # run the chain of extractions
+      extractor.extract(options)
     end
 
     # Transforms source data into outputs
@@ -28,8 +49,15 @@ module Migratrix
     end
 
     # Perform the migration
+    # TODO: turn this into a strategy object. This pattern migrates
+    # everything in all one go, while the user may want to do a batch
+    # strategy. YAGNI: Rails 3 lets us defer the querying until we get
+    # to the transform step, and then it's batched for us under the
+    # hood. ...assuming, of course, we change the ActiveRecord
+    # extractor's execute_extract method to return source instead of
+    # all, but now the
     def migrate
-      extract
+      @extracted_items = extract
       transform
       load
     end

data/lib/migratrix/migratrix.rb

@@ -7,6 +7,10 @@ module Migratrix
     ::Migratrix::Migratrix.migrate(name, options)
   end
 
+  def self.reload_migration(name)
+    ::Migratrix::Migratrix.reload_migration(name)
+  end
+
   def self.logger
     ::Migratrix::Migratrix.logger
   end
@@ -15,6 +19,10 @@ module Migratrix
     ::Migratrix::Migratrix.logger = new_logger
   end
 
+  def self.log_to(stream)
+    ::Migratrix::Migratrix.log_to(stream)
+  end
+
   class Migratrix
     include ::Migratrix::Loggable
 
@@ -35,9 +43,7 @@ module Migratrix
       klass_name = migration_name(name)
       unless loaded?(klass_name)
         raise MigrationAlreadyExists.new("Migratrix cannot instantiate class Migratrix::#{klass_name} because it already exists") if ::Migratrix.const_defined?(klass_name)
-        filename = migrations_path + "#{name}_migration.rb"
-        raise MigrationFileNotFound.new("Migratrix cannot find migration file #{filename}") unless File.exists?(filename)
-        load filename
+        reload_migration name
         raise MigrationNotDefined.new("Expected migration file #{filename} to define Migratrix::#{klass_name} but it did not") unless ::Migratrix.const_defined?(klass_name)
         register_migration(klass_name, "Migratrix::#{klass_name}".constantize)
       end
@@ -59,7 +65,13 @@ module Migratrix
     end
 
     def valid_options
-      %w(limit where)
+      %w(limit offset order where)
+    end
+
+    def reload_migration(name)
+      filename = migrations_path + "#{name}_migration.rb"
+      raise MigrationFileNotFound.new("Migratrix cannot find migration file #{filename}") unless File.exists?(filename)
+      load filename
     end
 
     # ----------------------------------------------------------------------
@@ -75,6 +87,10 @@ module Migratrix
       logger
     end
 
+    def self.log_to(stream)
+      self.logger = self.create_logger(stream)
+    end
+
     def self.init_logger
       return Rails.logger if Rails.logger
       @@logger = create_logger($stdout)
@@ -106,7 +122,7 @@ module Migratrix
     end
 
     def registered_migrations
-      @registered_migrations ||= {}
+      @@registered_migrations ||= {}
     end
     # End MigrationRegistry
     # ----------------------------------------------------------------------

data/lib/migratrix.rb

@@ -7,7 +7,7 @@ module Migratrix
   EXT=Pathname.new(__FILE__).dirname + "patches"
 
   def self.default_migrations_path
-    Rails.root + 'lib/migrations'
+    Rails.root + 'db/legacy'
   end
 
   require EXT + 'string_ext'
@@ -18,4 +18,7 @@ module Migratrix
   require APP + 'migration'
   require APP + 'migratrix'
 
+  require APP + 'extractors/extractor'
+  require APP + 'extractors/active_record'
 end
+

data/spec/lib/migratrix/extractors/active_record_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+require 'active_record'
+
+class TestModel < ::ActiveRecord::Base
+#  self.abstract_class = true
+end
+
+class TestActiveRecordExtractor < Migratrix::Extractors::ActiveRecord
+end
+
+describe Migratrix::Extractors::ActiveRecord do
+  let(:extractor) { TestActiveRecordExtractor.new }
+  describe "sanity check cat" do
+    it "is sanity checked" do
+      Migratrix::Extractors::Extractor.should_not be_nil
+      TestActiveRecordExtractor.should_not be_nil
+    end
+  end
+
+  describe ".new" do
+    it "raises TypeError unless source is Active" do
+      lambda { TestActiveRecordExtractor.new :source => Object }.should raise_error(TypeError)
+    end
+  end
+
+  describe "#source=" do
+    it "raises TypeError unless source is Active" do
+      lambda { extractor.source = Object }.should raise_error(TypeError)
+    end
+  end
+
+  describe "#obtain_source" do
+    it "raises ExtractorSourceUndefined unless source is defined" do
+      lambda { extractor.extract }.should raise_error(Migratrix::ExtractorSourceUndefined)
+    end
+
+    it "returns the legacy ActiveRecord class" do
+      extractor.source = TestModel
+      extractor.obtain_source(TestModel).should == TestModel
+    end
+  end
+end
+

data/spec/lib/migratrix/extractors/extractor_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+class TestExtractor < Migratrix::Extractors::Extractor
+end
+
+describe Migratrix::Extractors::Extractor do
+  describe "sanity check cat" do
+    it "is sanity checked" do
+      Migratrix::Extractors::Extractor.should_not be_nil
+      TestExtractor.should_not be_nil
+    end
+  end
+
+  describe "unimplemented method" do
+    let(:base_extractor) { Migratrix::Extractors::Extractor.new }
+    [:obtain_source, :handle_where, :handle_limit, :handle_offset, :handle_order, :to_query, :execute_extract].each do |method|
+      describe "#{method}" do
+        it "raises NotImplementedError" do
+          lambda { base_extractor.send(method, nil) }.should raise_error(NotImplementedError)
+        end
+      end
+    end
+  end
+
+  describe "#extract (default strategy)" do
+    describe "with no options" do
+      let(:extractor) { TestExtractor.new }
+      it "should call handle_source and execute_extract only" do
+        extractor.should_receive(:obtain_source).with(nil).and_return(13)
+        extractor.should_receive(:execute_extract).with(13).and_return(64)
+        extractor.extract.should == 64
+      end
+    end
+
+    describe "with all options" do
+      let(:extractor) { TestExtractor.new "where" => 1, "order" => 2, "limit" => 3, "offset" => 4 }
+      it "should call entire handler chain" do
+        extractor.should_receive(:obtain_source).with(nil).and_return("A")
+        extractor.should_receive(:handle_where).with("A", 1).and_return("B")
+        extractor.should_receive(:handle_order).with("B", 2).and_return("C")
+        extractor.should_receive(:handle_limit).with("C", 3).and_return("D")
+        extractor.should_receive(:handle_offset).with("D", 4).and_return("E")
+        extractor.should_receive(:execute_extract).with("E").and_return("BONG")
+        extractor.extract.should == "BONG"
+      end
+    end
+
+    describe "with overridden options" do
+      let(:extractor) { TestExtractor.new  }
+      let(:overrides) { {"where" => 5, "order" => 6, "limit" => 7, "offset" => 8 } }
+      it "should call entire handler chain" do
+        extractor.should_receive(:obtain_source).with(nil).and_return("A")
+        extractor.should_receive(:handle_where).with("A", 5).and_return("B")
+        extractor.should_receive(:handle_order).with("B", 6).and_return("C")
+        extractor.should_receive(:handle_limit).with("C", 7).and_return("D")
+        extractor.should_receive(:handle_offset).with("D", 8).and_return("E")
+        extractor.should_receive(:execute_extract).with("E").and_return("BONG")
+        extractor.extract(overrides).should == "BONG"
+      end
+    end
+  end
+end
+

data/spec/lib/migratrix/loggable_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+# class SomeLoggableThing
+#   include Migratrix::Loggable
+# end
+
+shared_examples_for "loggable" do
+  # Your spec must define loggable and loggable_name!
+  #  let(:loggable) { SomeLoggableThing.new }
+  let(:loggable_name) { loggable.class }
+  let(:buffer) { StringIO.new }
+  let(:logger) { Migratrix::Migratrix.create_logger(buffer) }
+
+  before do
+    Timecop.freeze(Time.local(2011, 6, 28, 3, 14, 15))
+  end
+
+  after do
+    Timecop.return
+  end
+
+  describe "your shared loggable specs" do
+    it "must define let(:loggable) to somehting that includes Migratrix::Loggable" do
+      loggable.class.ancestors.should include(Migratrix::Loggable)
+    end
+    it "must define let(:loggable_name) to the name of the class that will use the logger methods" do
+      loggable_name.should_not be_nil
+    end
+  end
+
+  describe "instance" do
+    def spec_correct_instance_log_message(method)
+      token = method.to_s.upcase[0]
+      with_logger(logger) do
+        loggable.send(method, "This is a test #{method} message")
+      end
+      buffer.string.should == "#{token} 2011-06-28 03:14:15: #{loggable_name}: This is a test #{method} message\n"
+    end
+
+    it "#logger can log" do
+      with_logger(logger) do
+        loggable.logger.info("This is a test")
+      end
+      buffer.string.should == "I 2011-06-28 03:14:15: This is a test\n"
+    end
+
+    describe "#info" do
+      it "logs with class name" do
+        with_logger(logger) do
+          loggable.info("This is a test")
+        end
+        buffer.string.should == "I 2011-06-28 03:14:15: #{loggable_name}: This is a test\n"
+      end
+    end
+
+    [:info, :debug, :warn, :error, :fatal].each do |log_level|
+      describe ".#{log_level}" do
+        it "logs #{log_level} message with class name" do
+          spec_correct_instance_log_message log_level
+        end
+      end
+    end
+  end
+
+  describe "class" do
+    def spec_correct_class_log_message(method)
+      token = method.to_s.upcase[0]
+      with_logger(logger) do
+        loggable.class.send(method, "This is a test #{method} message")
+      end
+      buffer.string.should == "#{token} 2011-06-28 03:14:15: #{loggable_name}: This is a test #{method} message\n"
+    end
+
+    it ".logger can log" do
+      with_logger(logger) do
+        loggable.class.logger.info("This is a test")
+      end
+      buffer.string.should == "I 2011-06-28 03:14:15: This is a test\n"
+    end
+
+    [:info, :debug, :warn, :error, :fatal].each do |log_level|
+      describe ".#{log_level}" do
+        it "logs #{log_level} message with class name" do
+          spec_correct_class_log_message log_level
+        end
+      end
+    end
+  end
+
+end

data/spec/lib/{migration_spec.rb → migratrix/migration_spec.rb}

@@ -6,6 +6,7 @@ class Migratrix::TestMigration < Migratrix::Migration
 end
 
 describe Migratrix::Migration do
+  let(:migration) { Migratrix::TestMigration.new }
   let(:loggable) { Migratrix::TestMigration.new }
   it_should_behave_like "loggable"
 
@@ -22,8 +23,6 @@ describe Migratrix::Migration do
   end
 
   describe "#migrate" do
-    let(:migration) { Migratrix::TestMigration.new }
-
     it "delegates to extract, transform, and load" do
       migration.should_receive(:extract).once
       migration.should_receive(:transform).once
@@ -31,7 +30,30 @@ describe Migratrix::Migration do
       migration.migrate
     end
   end
-end
 
+  describe "with mock active_record extractor" do
+    let(:mock_extractor) { mock("Extractor", :extract => 43)}
+    before do
+      Migratrix::Extractors::ActiveRecord.should_receive(:new).with({ :source => Object}).and_return(mock_extractor)
+      Migratrix::TestMigration.class_eval "set_extractor :active_record, :source => Object"
+    end
+
+    describe ".set_extractor" do
+      it "sets the class accessor for extractor" do
+        Migratrix::TestMigration.extractor.should == mock_extractor
+      end
+
+      it "also sets convenience instance method for extractor" do
+        Migratrix::TestMigration.new.extractor.should == mock_extractor
+      end
+    end
+
+    describe "#extract" do
+      it "delegates to extractor" do
+        migration.extract.should == 43
+      end
+    end
+  end
+end
 
 

data/spec/lib/{migratrix_spec.rb → migratrix/migratrix_spec.rb}

@@ -34,8 +34,8 @@ describe Migratrix::Migratrix do
   end
 
   describe ".migrations_path" do
-    it "uses ./lib/migrations by default" do
-      migratrix.migrations_path.should == ROOT + "lib/migrations"
+    it "uses ./db/legacy by default" do
+      migratrix.migrations_path.should == ROOT + "db/legacy"
     end
 
     it "can be overridden" do
@@ -46,7 +46,7 @@ describe Migratrix::Migratrix do
 
   describe "#valid_options" do
     it "returns the valid set of option keys" do
-      migratrix.valid_options.should == ["limit", "where"]
+      migratrix.valid_options.should == ["limit", "offset", "order", "where"]
     end
   end
 
@@ -99,26 +99,43 @@ describe Migratrix::Migratrix do
     end
   end
 
-  describe ".logger=" do
+  describe "with logger as a singleton" do
     let (:migration) { migratrix.create_migration :marbles }
     let (:buffer) { StringIO.new }
 
+    def spec_all_loggers_are(this_logger)
+      Migratrix.logger.should == this_logger
+      Migratrix::Migratrix.logger.should == this_logger
+      migratrix.logger.should == this_logger
+      migration.logger.should == this_logger
+      Migratrix::MarblesMigration.logger.should == this_logger
+    end
+
     before do
       reset_migratrix! migratrix
       migratrix.migrations_path = SPEC + "fixtures/migrations"
     end
 
-    it "sets logger globally across all Migratrices, the Migratrix module, Migrators and Models" do
-      logger = Migratrix::Migratrix.create_logger(buffer)
-      Migratrix::Migratrix.logger = logger
-      with_logger(logger) do
-        Migratrix.logger.should == logger
-        Migratrix::Migratrix.logger.should == logger
-        migratrix.logger.should == logger
-        migration.logger.should == logger
-        Migratrix::MarblesMigration.logger.should == logger
+    describe ".logger=" do
+      it "sets logger globally across all Migratrices, the Migratrix module, Migrators and Models" do
+        logger = Migratrix::Migratrix.create_logger(buffer)
+        with_logger(logger) do
+          Migratrix::Migratrix.logger = logger
+          spec_all_loggers_are logger
+        end
+      end
+    end
+    describe ".log_to" do
+      it "sets logger globally across all Migratrices, the Migratrix module, Migrators and Models" do
+        logger = Migratrix::Migratrix.create_logger(buffer)
+        with_logger(logger) do
+          Migratrix::Migratrix.should_receive(:create_logger).with(buffer).once.and_return(logger)
+          Migratrix.log_to buffer
+          spec_all_loggers_are logger
+        end
       end
     end
   end
+
 end
 

data/spec/lib/migratrix_module_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe Migratrix do
+  describe "sanity check kitty" do
+    it "is sanity checked" do
+      Migratrix.should_not be_nil
+      Migratrix.class.should == Module
+    end
+  end
+
+  describe "convenience delegator methods" do
+    def spec_delegates_to_migratrix(method, *args)
+      if args.size > 0
+        Migratrix::Migratrix.should_receive(method).with(*args).once
+      else
+        Migratrix::Migratrix.should_receive(method).once
+      end
+      Migratrix.send(method, *args)
+    end
+
+    describe ".migrate!" do
+      let (:migratrix) { Migratrix::Migratrix.new }
+
+      before do
+        reset_migratrix! migratrix
+        Migratrix::Migratrix.stub!(:new).and_return(migratrix)
+        migratrix.migrations_path = SPEC + "fixtures/migrations"
+      end
+
+      it "delegates to Migratrix::Migratrix" do
+        Migratrix.migrate! :marbles
+        Migratrix::MarblesMigration.should be_migrated
+      end
+    end
+
+    describe ".logger" do
+      it "delegates to Migratrix::Migratrix" do
+        spec_delegates_to_migratrix :logger
+      end
+    end
+
+    describe ".logger=" do
+      let (:logger) { Logger.new(StringIO.new) }
+      it "delegates to Migratrix::Migratrix" do
+        spec_delegates_to_migratrix :logger=, logger
+      end
+    end
+
+    describe ".log_to" do
+      let (:buffer) { StringIO.new }
+      it "delegates to Migratrix::Migratrix" do
+        spec_delegates_to_migratrix :log_to, buffer
+      end
+    end
+
+    describe ".reload_migration" do
+      it "delegates to Migratrix::Migratrix" do
+        spec_delegates_to_migratrix :reload_migration, :marbles
+      end
+    end
+  end
+end
+

data/spec/{patches/string_ext_spec.rb → lib/patches/object_ext_spec.rb}

@@ -19,8 +19,5 @@ describe Object do
         big_hash[:struct1].object_id.should == big_hash[:struct2].object_id
       end
     end
-
-#     it "returns a completely cloned, fully deep copy of the object" do
-#     end
   end
 end

data/spec/{patches/object_ext_spec.rb → lib/patches/string_ext_spec.rb}

@@ -28,4 +28,6 @@ describe String do
       "socks".should_not be_singular
     end
   end
+
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: migratrix
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-15 00:00:00.000000000Z
+date: 2011-10-16 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trollop
-  requirement: &2157814860 !ruby/object:Gem::Requirement
+  requirement: &2153508480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2157814860
+  version_requirements: *2153508480
 description: Migratrix, a Rails legacy database migration tool supporting multiple
   strategies, including arbitrary n-ary migrations (1->n, n->1, n->m), arbitrary inputs
   and outputs (ActiveRecord, bare SQL, CSV) and migration logging
@@ -38,6 +38,8 @@ files:
 - lib/migratrix.rb
 - lib/migratrix/active_record_migration_helpers.rb
 - lib/migratrix/exceptions.rb
+- lib/migratrix/extractors/active_record.rb
+- lib/migratrix/extractors/extractor.rb
 - lib/migratrix/loggable.rb
 - lib/migratrix/migration.rb
 - lib/migratrix/migratrix.rb
@@ -45,13 +47,15 @@ files:
 - lib/patches/object_ext.rb
 - lib/patches/string_ext.rb
 - spec/fixtures/migrations/marbles_migration.rb
-- spec/lib/loggable_spec.rb
-- spec/lib/migration_spec.rb
-- spec/lib/migrator_spec.rb
-- spec/lib/migratrix_spec.rb
-- spec/migratrix_module_spec.rb
-- spec/patches/object_ext_spec.rb
-- spec/patches/string_ext_spec.rb
+- spec/lib/migratrix/extractors/active_record_spec.rb
+- spec/lib/migratrix/extractors/extractor_spec.rb
+- spec/lib/migratrix/loggable_spec.rb
+- spec/lib/migratrix/migration_spec.rb
+- spec/lib/migratrix/migrator_spec.rb
+- spec/lib/migratrix/migratrix_spec.rb
+- spec/lib/migratrix_module_spec.rb
+- spec/lib/patches/object_ext_spec.rb
+- spec/lib/patches/string_ext_spec.rb
 - spec/spec_helper.rb
 - MIT-LICENSE
 homepage: http://github.com/dbrady/migratrix/

data/spec/lib/loggable_spec.rb

@@ -1,32 +0,0 @@
-require 'spec_helper'
-
-# class SomeLoggableThing
-#   include Migratrix::Loggable
-# end
-
-shared_examples_for "loggable" do
-#  let(:loggable) { SomeLoggableThing.new }
-  let(:buffer) { StringIO.new }
-  let(:logger) { Migratrix::Migratrix.create_logger(buffer) }
-
-  before do
-    Timecop.freeze(Time.local(2011, 6, 28, 3, 14, 15))
-  end
-
-  after do
-    Timecop.return
-  end
-
-  it "is loggable" do
-    loggable.class.ancestors.should include(Migratrix::Loggable)
-  end
-
-  describe "instance" do
-    it "can log" do
-      with_logger(logger) do
-        loggable.logger.info("This is a test")
-      end
-      buffer.string.should == "I 2011-06-28 03:14:15: This is a test\n"
-    end
-  end
-end

data/spec/migratrix_module_spec.rb

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-describe Migratrix do
-  describe "sanity check kitty" do
-    it "is sanity checked" do
-      Migratrix.should_not be_nil
-      Migratrix.class.should == Module
-    end
-  end
-
-  describe ".migrate!" do
-    let (:migratrix) { Migratrix::Migratrix.new }
-
-    before do
-      reset_migratrix! migratrix
-      Migratrix::Migratrix.stub!(:new).and_return(migratrix)
-      migratrix.migrations_path = SPEC + "fixtures/migrations"
-    end
-
-    it "delegates to Migratrix::Migratrix" do
-      Migratrix.migrate! :marbles
-      Migratrix::MarblesMigration.should be_migrated
-    end
-  end
-
-  describe ".logger" do
-    it "delegates to Migratrix::Migratrix" do
-      Migratrix::Migratrix.should_receive(:logger).and_return nil
-      Migratrix.logger
-    end
-  end
-
-  describe ".logger=" do
-    let (:logger) { Logger.new(StringIO.new) }
-    it "delegates to Migratrix::Migratrix" do
-      Migratrix::Migratrix.should_receive(:logger=).with(logger).and_return nil
-      Migratrix.logger = logger
-    end
-  end
-end
-