pliny 0.9.0 → 0.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cc0fc3ea6bd2bf5b8c5b6acdcf1cabf32ea781d5
-  data.tar.gz: 68bcfd4977381a765954e04884680dfaefdb5b4e
+  metadata.gz: eb6f83c7b5d986610339642f0aa5ea69471dbc80
+  data.tar.gz: 961db5cf37d94f149b0b1fce4fe4f05f0a715f84
 SHA512:
-  metadata.gz: 9a7c5e06960e5076cf66d5f3951acdba7e19152d1294dbc79c6946333f701bf574b9f520d0d3fe52256a5586cf8ebdb279870785f16e6db5b66d2601d880ff0c
-  data.tar.gz: 67a0baee927f357155076aa8466d4358abd1b0ef6d68dec6c9db69853db8b31ef042162a50b8945cad12965411dd41b697bd495ca70bb7accb3e8dcb3ce7e4e5
+  metadata.gz: 47578679bc614de3f1b6e99a22a53bd5cb5468d84c41b822c7c7036f60fac8cc5d8aa5b83109282f037b1cce1685359a751d6a17df17501c1a40a0ae7ff25107
+  data.tar.gz: f2a0a12c64767f6c82d9a1430780238b882099407177f96f024d7972d6cac76f13e0cb9ed501b93c1ee46502c0aaa37683575cec8629813749dac01420fad68c

data/lib/pliny/db_support.rb

@@ -25,16 +25,19 @@ module Pliny
       end
     end
 
-    def migrate
-      Sequel::Migrator.apply(db, "./db/migrate")
+    def migrate(target=nil)
+      Sequel::Migrator.apply(db, "./db/migrate", target)
     end
 
     def rollback
+      return unless db.tables.include?(:schema_migrations)
+      return unless current = db[:schema_migrations].order(Sequel.desc(:filename)).first
+
       migrations = Dir["./db/migrate/*.rb"].map { |f| File.basename(f).to_i }.sort
-      current = db[:schema_migrations].order(Sequel.desc(:filename)).first[:filename].to_i
-      target = 0 # by default, rollback everything
-      if i = migrations.index(current)
-        target = migrations[i - 1] || 0
+      target     = 0 # by default, rollback everything
+      index      = migrations.index(current[:filename].to_i)
+      if index > 0
+        target = migrations[index - 1]
       end
       Sequel::Migrator.apply(db, "./db/migrate", target)
     end

data/lib/pliny/version.rb

@@ -1,3 +1,3 @@
 module Pliny
-  VERSION = "0.9.0"
+  VERSION = "0.9.1"
 end

data/spec/db_support_spec.rb

@@ -40,26 +40,33 @@ describe Pliny::DbSupport do
 
   describe "#rollback" do
     before do
-      t = Time.now
-      File.open("#{@path}/db/migrate/#{(t-3).to_i}_first.rb", "w") do |f|
+      @t = Time.now
+      File.open("#{@path}/db/migrate/#{(@t-2).to_i}_first.rb", "w") do |f|
         f.puts "Sequel.migration { change { create_table(:first) } }"
       end
 
-      File.open("#{@path}/db/migrate/#{(t-2).to_i}_second.rb", "w") do |f|
+      File.open("#{@path}/db/migrate/#{(@t-1).to_i}_second.rb", "w") do |f|
         f.puts "Sequel.migration { change { create_table(:second) } }"
       end
-
-      File.open("#{@path}/db/migrate/#{(t-1).to_i}_third.rb", "w") do |f|
-        f.puts "Sequel.migration { change { create_table(:third) } }"
-      end
     end
 
-    it "reverts one migration" do
+    it "reverts migrations" do
       support.migrate
-      support.rollback
       assert_equal [:first, :schema_migrations, :second], DB.tables.sort
       support.rollback
       assert_equal [:first, :schema_migrations], DB.tables.sort
+      support.rollback
+      assert_equal [:schema_migrations], DB.tables.sort
+    end
+
+    it "handle blank databases (no schema_migrations table)" do
+      support.rollback # noop
+    end
+
+    it "handles databases not migrated (schema_migrations is empty)" do
+      support.migrate (@t-2).to_i
+      support.rollback # destroy table, leave schema_migrations
+      support.rollback # should noop
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pliny
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
 platform: ruby
 authors:
 - Brandur Leach