couch-migrate 1.1.2 → 1.1.3

data/Gemfile

@@ -2,9 +2,10 @@ source "http://rubygems.org"
 gem 'couchrest'
 
 group :development do
-  gem "rspec", "~> 2.3.0"
+  gem "rspec", "~> 2.6.0"
   gem "yard", "~> 0.6.0"
   gem "bundler", "~> 1.0.0"
   gem "jeweler", "~> 1.6.4"
   gem "rcov", ">= 0"
+  gem 'fakefs'
 end

data/Gemfile.lock

@@ -6,6 +6,7 @@ GEM
       multi_json (~> 1.0.0)
       rest-client (~> 1.6.1)
     diff-lcs (1.1.3)
+    fakefs (0.4.0)
     git (1.2.5)
     jeweler (1.6.4)
       bundler (~> 1.0)
@@ -17,14 +18,14 @@ GEM
     rcov (0.9.10)
     rest-client (1.6.7)
       mime-types (>= 1.16)
-    rspec (2.3.0)
-      rspec-core (~> 2.3.0)
-      rspec-expectations (~> 2.3.0)
-      rspec-mocks (~> 2.3.0)
-    rspec-core (2.3.1)
-    rspec-expectations (2.3.0)
+    rspec (2.6.0)
+      rspec-core (~> 2.6.0)
+      rspec-expectations (~> 2.6.0)
+      rspec-mocks (~> 2.6.0)
+    rspec-core (2.6.4)
+    rspec-expectations (2.6.0)
       diff-lcs (~> 1.1.2)
-    rspec-mocks (2.3.0)
+    rspec-mocks (2.6.0)
     yard (0.6.8)
 
 PLATFORMS
@@ -33,7 +34,8 @@ PLATFORMS
 DEPENDENCIES
   bundler (~> 1.0.0)
   couchrest
+  fakefs
   jeweler (~> 1.6.4)
   rcov
-  rspec (~> 2.3.0)
+  rspec (~> 2.6.0)
   yard (~> 0.6.0)

data/README.rdoc

@@ -115,6 +115,14 @@ db:seed" could provide.
       end
     end
 
+== Note
+
+This gem requires Ruby 1.9 only because of the use of
+"require_relative". In the true spirit of open source, if you would like
+to use this gem with Ruby 1.8, please fork it, fix those requires, enjoy
+using it, and then submit a pull request. One gold star will be awarded
+to the person who does this.
+
 == Thanks
 
 Thanks to OpenLogic (http://openlogic.com), a great company that

data/VERSION

@@ -1 +1 @@
-1.1.2
+1.1.3

data/couch-migrate.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{couch-migrate}
-  s.version = "1.1.2"
+  s.version = "1.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Greg Edwards"]
-  s.date = %q{2011-09-28}
+  s.date = %q{2011-10-04}
   s.description = %q{A simple migration system for CouchDB.}
   s.email = %q{greg@greglearns.com}
   s.extra_rdoc_files = [
@@ -26,7 +26,7 @@ Gem::Specification.new do |s|
     "Rakefile",
     "VERSION",
     "couch-migrate.gemspec",
-    "db/migrate/1_test.rb",
+    "db/migrate/1_example.rb",
     "lib/couch-migrate.rb",
     "lib/couch_migrate/base_executer.rb",
     "lib/couch_migrate/base_migrater.rb",
@@ -46,6 +46,8 @@ Gem::Specification.new do |s|
     "spec/couch_migrate/couch_persisted_list_spec.rb",
     "spec/couch_migrate/file_migrater_spec.rb",
     "spec/couch_migrate/file_persisted_list_spec.rb",
+    "spec/couch_migrate/shared_persisted_list.rb",
+    "spec/couch_migrate/shared_sequential_migrations.rb",
     "spec/spec_helper.rb"
   ]
   s.homepage = %q{https://github.com/gedwards/couch-migrate}
@@ -59,26 +61,29 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<couchrest>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, ["~> 2.3.0"])
+      s.add_development_dependency(%q<rspec>, ["~> 2.6.0"])
       s.add_development_dependency(%q<yard>, ["~> 0.6.0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6.4"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<fakefs>, [">= 0"])
     else
       s.add_dependency(%q<couchrest>, [">= 0"])
-      s.add_dependency(%q<rspec>, ["~> 2.3.0"])
+      s.add_dependency(%q<rspec>, ["~> 2.6.0"])
       s.add_dependency(%q<yard>, ["~> 0.6.0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
       s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<fakefs>, [">= 0"])
     end
   else
     s.add_dependency(%q<couchrest>, [">= 0"])
-    s.add_dependency(%q<rspec>, ["~> 2.3.0"])
+    s.add_dependency(%q<rspec>, ["~> 2.6.0"])
     s.add_dependency(%q<yard>, ["~> 0.6.0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
     s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<fakefs>, [">= 0"])
   end
 end
 

data/lib/couch_migrate/base_migrater.rb

@@ -1,5 +1,6 @@
 require_relative 'base_executer'
 require_relative 'base_persisted_list'
+require 'pathname'
 
 module CouchMigrate
   class BaseMigrater
@@ -87,7 +88,7 @@ module CouchMigrate
 
     def filter_and_sort(arr= [])
       # discard invalid formats, then sort numerically by first number, then alphabetically for remainder
-      format = /(\d+)_(.*)\.rb/
+      format = /^(\d+)_(.*)\.rb$/
       arr.map do |e|
         name = Pathname.new(e).basename.to_s
         match = format.match(name)

data/lib/couch_migrate/couch_persisted_list.rb

@@ -28,7 +28,6 @@ module CouchMigrate
     def cleanup
       @list = []
       write
-      self
     end
 
   end

data/lib/couch_migrate/file_persisted_list.rb

@@ -3,9 +3,9 @@ require 'yaml'
 
 module CouchMigrate
   class FilePersistedList < BasePersistedList
-    def initialize(directory="db/migrate")
+    def initialize(meta_file_directory="db/migrate")
       super()
-      path = Pathname.new(directory)
+      path = Pathname.new(meta_file_directory)
       path.mkpath
       @path = path+"meta.yml"
       read
@@ -26,7 +26,8 @@ module CouchMigrate
     end
 
     def cleanup
-      @path.delete rescue nil
+      File.delete(@path.to_s) rescue nil
+      # @path.delete rescue nil # fakefs does not handle this correctly
     end
 
   end

data/spec/couch_migrate/base_migrater_spec.rb

@@ -1,8 +1,49 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
+require 'pathname'
+require 'fakefs/spec_helpers'
+
+describe CouchMigrate::BaseMigrater, "sorting and filtering" do
+
+  it "ignores files that do not follow the migrationOrder_migration_title format" do
+    names = [
+      '2_name.rb', #valid
+      'a_bad_name.rb', #invalid
+      '.#1_migration_a.rb', #invalid
+      '3_name.rb', #valid
+    ]
+    subject.raw_migrations(names)
+    subject.raw_migrations.should == ['2_name.rb', '3_name.rb']
+  end
+
+  it "sorts on migration numbers" do
+    subject.raw_migrations(['2_name.rb','1_name.rb', '3_name.rb'])
+    subject.pending_migrations.should == ['1_name.rb','2_name.rb', '3_name.rb']
+  end
+
+  it "sorts on title if migration numbers are the same" do
+    subject.raw_migrations(['2_a_name.rb','2_b_name.rb'])
+    subject.raw_migrations.should == ['2_a_name.rb','2_b_name.rb']
+  end
+
+  it "sorts on title if migration numbers are the same (version 2)" do
+    subject.raw_migrations(['2_b_name.rb','2_a_name.rb'])
+    subject.raw_migrations.should == ['2_a_name.rb','2_b_name.rb']
+  end
+
+  it "sorts migration numbers numerically, not alphabetically" do
+    subject.raw_migrations(['10_name.rb','2_name.rb'])
+    subject.raw_migrations.should == ['2_name.rb', '10_name.rb']
+  end
+
+  it "sorts migration numbers numerically, not alphabetically (version 2)" do
+    subject.raw_migrations(['2_name.rb','10_name.rb'])
+    subject.raw_migrations.should == ['2_name.rb', '10_name.rb']
+  end
+
+end
 
-describe "BaseMigrater", "#migrate" do
-  subject { CouchMigrate::BaseMigrater.new }
+describe CouchMigrate::BaseMigrater, "#migrate" do
 
   let(:file_name_1){ "1_migration_a.rb" }
   let(:file_name_2){ "2_migration_b.rb" }
@@ -14,24 +55,25 @@ describe "BaseMigrater", "#migrate" do
     subject.raw_migrations([file_name_1, file_name_2]).raw_migrations.should == [file_name_1,file_name_2]
   end
 
+  it "does not execute migrations that have already been run" do
+    subject.raw_migrations([file_name_1, file_name_2]).completed_migrations([file_name_1]).pending_migrations.should == [file_name_2]
+  end
+
   context "interacting with the file system" do
-    let(:path){ Pathname.new("spec/tmp") }
-    let(:file_1){ path + file_name_1 }
-    let(:file_2){ path + file_name_2 }
-    let(:file_3){ path + file_name_3 }
-
-    before(:all) do
-      path.mkpath
-      File.open(file_3,"w"){|f| f << "up do end" }
-      File.open(file_2,"w"){|f| f << "up do raise '2 simulated failure' end" }
-      File.open(file_1,"w"){|f| f << "up do end" }
-    end
+    include FakeFS::SpecHelpers
+
+    let(:path){ Pathname.new("spec/tmp").tap{|p| p.mkpath} }
+
+    before(:each) do
+      # create migration files
+      [
+        [file_name_1, "up do end" ],
+        [file_name_2, "up do raise '2 simulated failure' end" ],
+        [file_name_3, "up do end" ],
+      ].each do |file_name, content|
+        File.open(path+file_name, 'w') {|f| f << content}
+      end
 
-    after(:all) do
-      file_1.delete
-      file_2.delete
-      file_3.delete
-      path.rmdir rescue nil
     end
 
     it "reads the db/migrations directory to generate a list of potential migrations to run" do
@@ -41,51 +83,14 @@ describe "BaseMigrater", "#migrate" do
 
     it "stops when a migration file exits with an error, not processing any subsequent files afterwards" do
       subject.directory(path).pending_migrations.should == [file_name_1, file_name_2, file_name_3]
+
       subject.migrate(:quiet, :up).should == {success: [file_name_1], failed: [file_name_2]}
+
       subject.failed_migration.should == file_name_2
       subject.pending_migrations.should == [file_name_2, file_name_3]
     end
 
   end
 
-  it "does not execute migrations that have already been run" do
-    subject.raw_migrations([file_name_1, file_name_2]).completed_migrations([file_name_1]).pending_migrations.should == [file_name_2]
-  end
-
-end
-
-describe "BaseMigrater", "sorting and filtering" do
-  subject { CouchMigrate::BaseMigrater.new }
-
-  it "sorts on migration numbers" do
-    subject.raw_migrations(['2_name.rb','1_name.rb', '3_name.rb'])
-    subject.pending_migrations.should == ['1_name.rb','2_name.rb', '3_name.rb']
-  end
-
-  it "sorts on title if migration numbers are the same" do
-    subject.raw_migrations(['2_a_name.rb','2_b_name.rb'])
-    subject.raw_migrations.should == ['2_a_name.rb','2_b_name.rb']
-  end
-
-  it "sorts on title if migration numbers are the same (version 2)" do
-    subject.raw_migrations(['2_b_name.rb','2_a_name.rb'])
-    subject.raw_migrations.should == ['2_a_name.rb','2_b_name.rb']
-  end
-
-  it "sorts migration numbers numerically, not alphabetically" do
-    subject.raw_migrations(['10_name.rb','2_name.rb'])
-    subject.raw_migrations.should == ['2_name.rb', '10_name.rb']
-  end
-
-  it "sorts migration numbers numerically, not alphabetically (version 2)" do
-    subject.raw_migrations(['2_name.rb','10_name.rb'])
-    subject.raw_migrations.should == ['2_name.rb', '10_name.rb']
-  end
-
-  it "ignores files that do not have follow the migrationOrder_migration_title format" do
-    subject.raw_migrations(['2_name.rb','a_bad_name.rb'])
-    subject.raw_migrations.should == ['2_name.rb']
-  end
-
 end
 

data/spec/couch_migrate/base_persisted_list_spec.rb

@@ -1,44 +1,11 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
+require_relative 'shared_persisted_list'
 
 describe CouchMigrate::BasePersistedList do
 
-  it "starts off with an empty value/list" do
-    subject.get.should == []
+  it_should_behave_like "a_persisted_list" do
+    subject { CouchMigrate::BasePersistedList.new }
   end
-
-  it "can set values" do
-    arr = [1,2,3]
-    subject.set(arr).get.should == arr
-  end
-
-  it "can append values" do
-    arr = [1,2,3]
-    more = [4,5]
-    expected = [1,2,3,4,5]
-
-    subject.set(arr).add(more)
-    subject.get.should == expected
-  end
-
-  it "can remove values" do
-    arr = [1,2,3]
-    remove = [2,3]
-    expected = [1]
-
-    subject.set(arr).remove(remove)
-    subject.get.should == expected
-  end
-
-  it "can be reset" do
-    subject.set([1,2,3]).reset
-    subject.get.should == []
-  end
-
-  it "is chainable" do
-    subject.set([1]).add([2]).get.should == [1,2]
-    subject.set([1]).add([2]).reset.get.should == []
-  end
-
 end
 

data/spec/couch_migrate/couch_migrater_spec.rb

@@ -1,84 +1,20 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
+require_relative 'shared_sequential_migrations'
 
-describe "CouchMigrater", "#migrate" do
+describe CouchMigrate::CouchMigrater, "#migrate" do
 
-
-  context "interacting with the file system" do
+  it_should_behave_like "sequential_migrations" do
     subject { CouchMigrate::CouchMigrater }
-    let(:db_path) { 'http://127.0.0.1:5984/couch-migrate_test' }
-    let(:path){ Pathname.new("spec/tmp") }
-
-    let(:file_name_1){ "1_migration_a.rb" }
-    let(:file_name_2){ "2_migration_b.rb" }
-    let(:file_name_3){ "3_migration_b.rb" }
-    let(:file_path_1){ path + file_name_1 }
-    let(:file_path_2){ path + file_name_2 }
-    let(:file_path_3){ path + file_name_3 }
-
-    before(:each) do
-      migrater.reset # clear pre-existing migration meta data
-      path.mkpath
-      file_path_1.delete rescue nil
-      file_path_2.delete rescue nil
-      file_path_3.delete rescue nil
-    end
 
-    after(:each) do
-      # migrater.reset # remove meta file
-      file_path_1.delete rescue nil
-      file_path_2.delete rescue nil
-      file_path_3.delete rescue nil
-      path.rmdir rescue nil
-    end
+    let(:path){ Pathname.new("spec/tmp") }
+    let(:db_path) { 'http://127.0.0.1:5984/couch-migrate_test' }
 
     def migrater
       # allows new migrations to be picked up
-      res = subject.new(db_path, path)
-      res
-    end
-
-    it "handles a sequence of migration added over time" do
-      params = [:up, :quiet]
-      down_params = [:down, :quiet]
-
-      # should succeed
-      File.open(file_path_1,"w"){|f| f << "up do end" }
-      migrater.migrate(*params).should == {success: [file_name_1]}
-
-      # should do nothing
-      migrater.migrate(*params).should == {}
-
-      # should fail
-      File.open(file_path_2,"w"){|f| f << "up do raise '2 simulated failure' end" }
-      migrater.migrate(*params).should == {success: [], failed: [file_name_2]}
-
-      # should pass now
-      File.open(file_path_2,"w"){|f| f << "up do end" }
-      migrater.migrate(*params).should == {success: [file_name_2]}
-
-      # should succeed
-      File.open(file_path_3,"w"){|f| f << "up do end" }
-      migrater.migrate(*params).should == {success: [file_name_3]}
-
-      # should do nothing
-      migrater.migrate(*params).should == {}
-
-      # resetting the migration causes all to be run next time
-      migrater.reset
-      migrater.migrate(*params).should == {success: [file_name_1, file_name_2, file_name_3]}
-
-      # Down * 3
-      migrater.migrate(*down_params).should == {success: [file_name_3]}
-      migrater.migrate(*down_params).should == {success: [file_name_2]}
-      migrater.migrate(*down_params).should == {success: [file_name_1]}
-
-      # Down does nothing
-      migrater.migrate(*down_params).should == {}
+      subject.new(db_path, path)
     end
-
   end
 
 end
 
-

data/spec/couch_migrate/couch_persisted_list_spec.rb

@@ -1,40 +1,12 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
+require_relative 'shared_persisted_list'
 
 describe CouchMigrate::CouchPersistedList do
-  let(:db_path) { 'http://127.0.0.1:5984/couch-migrate_test' }
-  subject { CouchMigrate::CouchPersistedList.new(db_path) }
 
-  before(:each) do
-    subject.reset
+  it_should_behave_like "a_persisted_list" do
+    let(:db_path) { 'http://127.0.0.1:5984/couch-migrate_test' }
+    subject { CouchMigrate::CouchPersistedList.new(db_path).tap{|l| l.reset} }
   end
-
-  it "starts off with an empty value/list" do
-    subject.get.should == []
-  end
-
-  it "can set values" do
-    arr = [1,2,3]
-    subject.set(arr).get.should == arr
-  end
-
-  it "can append values" do
-    arr = [1,2,3]
-    more = [4,5]
-    expected = [1,2,3,4,5]
-
-    subject.set(arr).add(more)
-    subject.get.should == expected
-  end
-
-  it "can be reset" do
-    subject.set([1,2,3]).reset
-    subject.get.should == []
-  end
-
-  it "is chainable" do
-    subject.set([1]).add([2]).get.should == [1,2]
-    subject.set([1]).add([2]).reset.get.should == []
-  end
-
 end
+

data/spec/couch_migrate/file_migrater_spec.rb

@@ -1,86 +1,17 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
+require_relative 'shared_sequential_migrations'
 
-describe "FileMigrater", "#migrate" do
+describe CouchMigrate::FileMigrater, "#migrate" do
 
-
-  context "interacting with the file system" do
+  it_should_behave_like "sequential_migrations" do
     subject { CouchMigrate::FileMigrater }
     let(:path){ Pathname.new("spec/tmp") }
 
-    let(:file_name_1){ "1_migration_a.rb" }
-    let(:file_name_2){ "2_migration_b.rb" }
-    let(:file_name_3){ "3_migration_b.rb" }
-    let(:file_path_1){ path + file_name_1 }
-    let(:file_path_2){ path + file_name_2 }
-    let(:file_path_3){ path + file_name_3 }
-
-    before(:each) do
-      migrater.reset # clear pre-existing migration meta data
-      path.mkpath
-      file_path_1.delete rescue nil
-      file_path_2.delete rescue nil
-      file_path_3.delete rescue nil
-    end
-
-    after(:each) do
-      migrater.reset # remove meta file
-      file_path_1.delete rescue nil
-      file_path_2.delete rescue nil
-      file_path_3.delete rescue nil
-      path.rmdir rescue nil
-    end
-
     def migrater
       # allows new migrations to be picked up
       subject.new(path)
     end
-
-    it "handles a sequence of migration added over time" do
-      up_params = [:up, :quiet]
-      down_params = [:down, :quiet]
-
-      # should succeed
-      File.open(file_path_1,"w"){|f| f << "up do end" } 
-      migrater.migrate(*up_params).should == {success: [file_name_1]}
-
-      # should do nothing
-      migrater.migrate(*up_params).should == {}
-
-      # should fail
-      File.open(file_path_2,"w"){|f| f << "up do raise '2 simulated failure' end" }
-      migrater.migrate(*up_params).should == {success: [], failed: [file_name_2]}
-
-      # should pass now
-      File.open(file_path_2,"w"){|f| f << "up do end" }
-      migrater.migrate(*up_params).should == {success: [file_name_2]}
-
-      # Down
-      migrater.migrate(*down_params).should == {success: [file_name_2]}
-
-      # up
-      migrater.migrate(*up_params).should == {success: [file_name_2]}
-
-      # should succeed
-      File.open(file_path_3,"w"){|f| f << "up do end" }
-      migrater.migrate(*up_params).should == {success: [file_name_3]}
-
-      # should do nothing
-      migrater.migrate(*up_params).should == {}
-
-      # resetting the migration causes all to be run next time
-      migrater.reset
-      migrater.migrate(*up_params).should == {success: [file_name_1, file_name_2, file_name_3]}
-
-      # Down * 3
-      migrater.migrate(*down_params).should == {success: [file_name_3]}
-      migrater.migrate(*down_params).should == {success: [file_name_2]}
-      migrater.migrate(*down_params).should == {success: [file_name_1]}
-
-      # Down does nothing
-      migrater.migrate(*down_params).should == {}
-    end
-
   end
 
 end

data/spec/couch_migrate/file_persisted_list_spec.rb

@@ -1,42 +1,13 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'couch-migrate'
 require 'pathname'
+require_relative 'shared_persisted_list'
 
 describe CouchMigrate::FilePersistedList do
-  let(:path){ Pathname.new("spec/tmp") }
-  subject { CouchMigrate::FilePersistedList.new(path) }
 
-  after(:all) do
-    Pathname.new(path).delete
+  it_should_behave_like "a_persisted_list" do
+    let(:path){ Pathname.new("spec/tmp") }
+    subject { CouchMigrate::FilePersistedList.new(path) }
   end
-
-  it "starts off with an empty value/list" do
-    subject.get.should == []
-  end
-
-  it "can set values" do
-    arr = [1,2,3]
-    subject.set(arr).get.should == arr
-  end
-
-  it "can append values" do
-    arr = [1,2,3]
-    more = [4,5]
-    expected = [1,2,3,4,5]
-
-    subject.set(arr).add(more)
-    subject.get.should == expected
-  end
-
-  it "can be reset" do
-    subject.set([1,2,3]).reset
-    subject.get.should == []
-  end
-
-  it "is chainable" do
-    subject.set([1]).add([2]).get.should == [1,2]
-    subject.set([1]).add([2]).reset.get.should == []
-  end
-
 end
 

data/spec/couch_migrate/shared_persisted_list.rb

@@ -0,0 +1,46 @@
+require 'fakefs/spec_helpers'
+
+shared_examples_for "a_persisted_list" do
+  include FakeFS::SpecHelpers
+
+  subject { raise "subject must return a new instance" }
+
+  it "starts off with an empty value/list" do
+    subject.get.should == []
+  end
+
+  it "can set values" do
+    arr = [1,2,3]
+    subject.set(arr).get.should == arr
+  end
+
+  it "can append values" do
+    arr = [1,2,3]
+    more = [4,5]
+    expected = [1,2,3,4,5]
+
+    subject.set(arr).add(more)
+    subject.get.should == expected
+  end
+
+  it "can remove values" do
+    arr = [1,2,3]
+    remove = [2,3]
+    expected = [1]
+
+    subject.set(arr).remove(remove)
+    subject.get.should == expected
+  end
+
+  it "can be reset" do
+    subject.set([1,2,3]).reset
+    subject.get.should == []
+  end
+
+  it "is chainable" do
+    subject.set([1]).add([2]).get.should == [1,2]
+    subject.set([1]).add([2]).reset.get.should == []
+  end
+
+end
+

data/spec/couch_migrate/shared_sequential_migrations.rb

@@ -0,0 +1,84 @@
+require 'pathname'
+require 'fakefs/spec_helpers'
+
+shared_examples_for "sequential_migrations" do
+  let(:path){ raise "a path must be defined by the grouping using these shared examples" }
+
+  include FakeFS::SpecHelpers
+
+  def migrater
+    # allows new migrations to be picked up
+    raise "Groups that include shared examples for sequential migrations must implement a migrater function that returns a new migration object"
+  end
+
+  context "sequential usage over time" do
+
+    let(:file_name_1){ "1_migration_a.rb" }
+    let(:file_name_2){ "2_migration_b.rb" }
+    let(:file_name_3){ "3_migration_b.rb" }
+
+    before(:each) do
+      path.mkpath
+      migrater.reset # clear pre-existing migration meta data
+    end
+
+    def create_migration(file_name, content)
+      File.open(path+file_name,"w"){|f| f << content }
+    end
+
+    it "handles a sequence of migration added over time" do
+      up_params = [:up, :quiet]
+      down_params = [:down, :quiet]
+
+      # should succeed
+      create_migration(file_name_1, "up do end" )
+      migrater.raw_migrations.should == [file_name_1]
+      migrater.completed_migrations.should == []
+      migrater.pending_migrations.should == [file_name_1]
+      migrater.migrate(*up_params).should == {success: [file_name_1]}
+
+      # should do nothing
+      migrater.migrate(*up_params).should == {}
+
+      # should fail
+      create_migration(file_name_2, "up do raise '2 simulated failure' end" )
+      migrater.migrate(*up_params).should == {success: [], failed: [file_name_2]}
+
+      # should pass now
+      create_migration(file_name_2, "up do end" )
+      migrater.migrate(*up_params).should == {success: [file_name_2]}
+
+      # Down
+      migrater.migrate(*down_params).should == {success: [file_name_2]}
+
+      # up
+      migrater.migrate(*up_params).should == {success: [file_name_2]}
+
+      # should succeed
+      create_migration(file_name_3, "up do end" )
+      migrater.migrate(*up_params).should == {success: [file_name_3]}
+
+      # should do nothing
+      migrater.migrate(*up_params).should == {}
+
+      # resetting the migration causes all to be run next time
+      migrater.reset
+      migrater.pending_migrations.should == [file_name_1, file_name_2, file_name_3]
+      migrater.migrate(*up_params).should == {success: [file_name_1, file_name_2, file_name_3]}
+
+      # Down * 3
+      migrater.migrate(*down_params).should == {success: [file_name_3]}
+      migrater.migrate(*down_params).should == {success: [file_name_2]}
+      migrater.migrate(*down_params).should == {success: [file_name_1]}
+
+      # Down does nothing after going down as far as possible
+      migrater.migrate(*down_params).should == {}
+
+      # Now, back up
+      migrater.pending_migrations.should == [file_name_1, file_name_2, file_name_3]
+      migrater.migrate(*up_params).should == {success: [file_name_1, file_name_2, file_name_3]}
+
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'rspec'
+require 'fakefs/safe'
 
 # Requires supporting files with custom matchers and macros, etc,
 # in ./support/ and its subdirectories.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: couch-migrate
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,12 +9,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-09-28 00:00:00.000000000 -06:00
+date: 2011-10-04 00:00:00.000000000 -06:00
 default_executable: 
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
-  requirement: &2158537840 !ruby/object:Gem::Requirement
+  requirement: &2154429960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,21 +22,21 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2158537840
+  version_requirements: *2154429960
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2158536380 !ruby/object:Gem::Requirement
+  requirement: &2154429080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.3.0
+        version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *2158536380
+  version_requirements: *2154429080
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &2158534860 !ruby/object:Gem::Requirement
+  requirement: &2154428340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -44,10 +44,10 @@ dependencies:
         version: 0.6.0
   type: :development
   prerelease: false
-  version_requirements: *2158534860
+  version_requirements: *2154428340
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &2158513340 !ruby/object:Gem::Requirement
+  requirement: &2154427760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -55,10 +55,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *2158513340
+  version_requirements: *2154427760
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &2158512000 !ruby/object:Gem::Requirement
+  requirement: &2154427160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -66,10 +66,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *2158512000
+  version_requirements: *2154427160
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &2158510080 !ruby/object:Gem::Requirement
+  requirement: &2154426620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -77,7 +77,18 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2158510080
+  version_requirements: *2154426620
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: &2154425960 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *2154425960
 description: A simple migration system for CouchDB.
 email: greg@greglearns.com
 executables: []
@@ -95,7 +106,7 @@ files:
 - Rakefile
 - VERSION
 - couch-migrate.gemspec
-- db/migrate/1_test.rb
+- db/migrate/1_example.rb
 - lib/couch-migrate.rb
 - lib/couch_migrate/base_executer.rb
 - lib/couch_migrate/base_migrater.rb
@@ -115,6 +126,8 @@ files:
 - spec/couch_migrate/couch_persisted_list_spec.rb
 - spec/couch_migrate/file_migrater_spec.rb
 - spec/couch_migrate/file_persisted_list_spec.rb
+- spec/couch_migrate/shared_persisted_list.rb
+- spec/couch_migrate/shared_sequential_migrations.rb
 - spec/spec_helper.rb
 has_rdoc: true
 homepage: https://github.com/gedwards/couch-migrate
@@ -132,7 +145,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 126001030709164033
+      hash: 3382144699293927769
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: