sequel 4.30.0 → 4.31.0

data/spec/extensions/date_arithmetic_spec.rb

@@ -69,7 +69,7 @@ describe "date_arithmetic extension" do
     db = Sequel.mock
     db.extend_datasets do
       def date_add_sql_append(sql, da)
-        interval = ''
+        interval = String.new
         each_valid_interval_unit(da.interval, Sequel::SQL::DateAdd::DatasetMethods::DEF_DURATION_UNITS) do |value, sql_unit|
           interval << "#{value} #{sql_unit} "
         end

data/spec/extensions/dirty_spec.rb

@@ -2,7 +2,7 @@ require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
 
 describe "Sequel::Plugins::Dirty" do
   before do
-    @db = Sequel.mock(:fetch=>{:initial=>'i', :initial_changed=>'ic'}, :numrows=>1)
+    @db = Sequel.mock(:fetch=>{:initial=>'i'.dup, :initial_changed=>'ic'.dup}, :numrows=>1)
     @c = Class.new(Sequel::Model(@db[:c]))
     @c.plugin :dirty
     @c.columns :initial, :initial_changed, :missing, :missing_changed
@@ -135,9 +135,9 @@ describe "Sequel::Plugins::Dirty" do
 
   describe "with new instance" do
     before do
-      @o = @c.new(:initial=>'i', :initial_changed=>'ic')
-      @o.initial_changed = 'ic2'
-      @o.missing_changed = 'mc2'
+      @o = @c.new(:initial=>'i'.dup, :initial_changed=>'ic'.dup)
+      @o.initial_changed = 'ic2'.dup
+      @o.missing_changed = 'mc2'.dup
     end
 
     include dirty_plugin_specs
@@ -158,8 +158,8 @@ describe "Sequel::Plugins::Dirty" do
   describe "with existing instance" do
     before do
       @o = @c[1]
-      @o.initial_changed = 'ic2'
-      @o.missing_changed = 'mc2'
+      @o.initial_changed = 'ic2'.dup
+      @o.missing_changed = 'mc2'.dup
     end
 
     include dirty_plugin_specs

data/spec/extensions/force_encoding_spec.rb

@@ -10,7 +10,7 @@ describe "force_encoding plugin" do
   end
 
   it "should force encoding to given encoding on load" do
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     o = @c.load(:id=>1, :x=>s)
     o.x.must_equal 'blah'
@@ -18,7 +18,7 @@ describe "force_encoding plugin" do
   end
   
   it "should force encoding to given encoding when setting column values" do
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     o = @c.new(:x=>s)
     o.x.must_equal 'blah'
@@ -26,7 +26,7 @@ describe "force_encoding plugin" do
   end
   
   it "should work correctly when given a frozen string" do
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     s.freeze
     o = @c.new(:x=>s)
@@ -35,7 +35,7 @@ describe "force_encoding plugin" do
   end
   
   it "should have a forced_encoding class accessor" do
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     @c.forced_encoding = 'Windows-1258'
     o = @c.load(:id=>1, :x=>s)
@@ -44,7 +44,7 @@ describe "force_encoding plugin" do
   end
   
   it "should not force encoding if forced_encoding is nil" do
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     @c.forced_encoding = nil
     o = @c.load(:id=>1, :x=>s)
@@ -54,27 +54,27 @@ describe "force_encoding plugin" do
   
   it "should work correctly when subclassing" do
     c = Class.new(@c)
-    s = 'blah'
+    s = 'blah'.dup
     s.force_encoding('US-ASCII')
     o = c.load(:id=>1, :x=>s)
     o.x.must_equal 'blah'
     o.x.encoding.must_equal @e1
     
     c.plugin :force_encoding, 'UTF-16LE'
-    s = ''
+    s = String.new
     s.force_encoding('US-ASCII')
     o = c.load(:id=>1, :x=>s)
     o.x.must_equal ''
     o.x.encoding.must_equal Encoding.find('UTF-16LE')
     
     @c.plugin :force_encoding, 'UTF-32LE'
-    s = ''
+    s = String.new
     s.force_encoding('US-ASCII')
     o = @c.load(:id=>1, :x=>s)
     o.x.must_equal ''
     o.x.encoding.must_equal Encoding.find('UTF-32LE')
     
-    s = ''
+    s = String.new
     s.force_encoding('US-ASCII')
     o = c.load(:id=>1, :x=>s)
     o.x.must_equal ''
@@ -85,7 +85,7 @@ describe "force_encoding plugin" do
     o = @c.new
     ds = DB[:a]
     def ds.first
-      s = 'blah'
+      s = 'blah'.dup
       s.force_encoding('US-ASCII')
       {:id=>1, :x=>s}
     end
@@ -96,10 +96,10 @@ describe "force_encoding plugin" do
   end
   
   it "should work when refreshing model instances" do
-    o = @c.load(:id=>1, :x=>'as')
+    o = @c.load(:id=>1, :x=>'as'.dup)
     ds = DB[:a]
     def ds.first
-      s = 'blah'
+      s = 'blah'.dup
       s.force_encoding('US-ASCII')
       {:id=>1, :x=>s}
     end

data/spec/extensions/input_transformer_spec.rb

@@ -48,7 +48,7 @@ describe "Sequel::Plugins::InputTransformer" do
   it "should apply multiple input transformers in reverse order of their call" do
     @c.add_input_transformer(:add_bar){|v| v << 'bar'}
     @c.add_input_transformer(:add_foo){|v| v << 'foo'}
-    @o.name = ' name '
+    @o.name = ' name '.dup
     @o.name.must_equal 'raboof eman '
   end
 end

data/spec/extensions/migration_spec.rb

@@ -288,6 +288,14 @@ describe "Sequel::IntegerMigrator" do
     proc{Sequel::Migrator.apply(@db, "spec/files/duplicate_integer_migrations")}.must_raise(Sequel::Migrator::Error)
   end
 
+  it "should raise an error if there is an empty migration file" do
+    proc{Sequel::Migrator.apply(@db, "spec/files/empty_migration")}.must_raise(Sequel::Migrator::Error)
+  end
+
+  it "should raise an error if there is a migration file with multiple migrations" do
+    proc{Sequel::Migrator.apply(@db, "spec/files/double_migration")}.must_raise(Sequel::Migrator::Error)
+  end
+
   it "should add a column name if it doesn't already exist in the schema_info table" do
     @db.create_table(:schema_info){Integer :v}
     def @db.alter_table(*); end
@@ -474,6 +482,14 @@ describe "Sequel::TimestampMigrator" do
     Object.send(:remove_const, "CreateAlbums") if Object.const_defined?("CreateAlbums")
   end
   
+  it "should raise an error if there is an empty migration file" do
+    proc{Sequel::TimestampMigrator.apply(@db, "spec/files/empty_migration")}.must_raise(Sequel::Migrator::Error)
+  end
+
+  it "should raise an error if there is a migration file with multiple migrations" do
+    proc{Sequel::TimestampMigrator.apply(@db, "spec/files/double_migration")}.must_raise(Sequel::Migrator::Error)
+  end
+
   it "should handle migrating up or down all the way" do
     @dir = 'spec/files/timestamped_migrations'
     @m.apply(@db, @dir)

data/spec/extensions/modification_detection_spec.rb

@@ -3,7 +3,7 @@ require 'yaml'
 
 describe "serialization_modification_detection plugin" do
   before do
-    @ds = Sequel.mock(:fetch=>{:id=>1, :a=>'a', :b=>1, :c=>['a'], :d=>{'b'=>'c'}}, :numrows=>1, :autoid=>1)[:items]
+    @ds = Sequel.mock(:fetch=>{:id=>1, :a=>'a'.dup, :b=>1, :c=>['a'.dup], :d=>{'b'=>'c'.dup}}, :numrows=>1, :autoid=>1)[:items]
     @c = Class.new(Sequel::Model(@ds))
     @c.plugin :modification_detection
     @c.columns :a, :b, :c, :d
@@ -33,7 +33,7 @@ describe "serialization_modification_detection plugin" do
   end
   
   it "should not list a column twice" do
-    @o.a = 'b'
+    @o.a = 'b'.dup
     @o.a << 'a'
     @o.changed_columns.must_equal [:a]
   end

data/spec/extensions/nested_attributes_spec.rb

@@ -587,7 +587,7 @@ describe "NestedAttributes plugin" do
   it "should accept a :transform block that returns a changed attributes hash" do
     @Album.nested_attributes :tags, :transform=>proc{|parent, hash| hash[:name] << parent.name; hash }
     a = @Album.new(:name => 'Al')
-    a.set(:tags_attributes=>[{:name=>'T'}, {:name=>'T2'}])
+    a.set(:tags_attributes=>[{:name=>'T'.dup}, {:name=>'T2'.dup}])
     @db.sqls.must_equal []
     a.save
     check_sql_array("INSERT INTO albums (name) VALUES ('Al')",

data/spec/extensions/optimistic_locking_spec.rb

@@ -5,7 +5,7 @@ describe "optimistic_locking plugin" do
     @c = Class.new(Sequel::Model(:people)) do
     end
     h = {1=>{:id=>1, :name=>'John', :lock_version=>2}}
-    lv = @lv = "lock_version"
+    lv = @lv = "lock_version".dup
     @c.instance_dataset.numrows = @c.dataset.numrows = proc do |sql|
       case sql
       when /UPDATE people SET (name|#{lv}) = ('Jim'|'Bob'|\d+), (?:name|#{lv}) = ('Jim'|'Bob'|\d+) WHERE \(\(id = (\d+)\) AND \(#{lv} = (\d+)\)\)/

data/spec/extensions/pg_array_spec.rb

@@ -316,14 +316,9 @@ describe "pg_array extension" do
     Sequel::Postgres::PG_TYPES[1].call('{1}').class.must_equal(Sequel::Postgres::PGArray)
   end
 
-  it "should support registering custom types with :parser=>:json option" do
-    Sequel::Postgres::PGArray.register('foo', :oid=>2, :parser=>:json)
-    Sequel::Postgres::PG_TYPES[2].must_be_kind_of(Sequel::Postgres::PGArray::JSONCreator)
-  end
-
-  it "should support registering converters with :parser=>:json option and blocks" do
-    Sequel::Postgres::PGArray.register('foo', :oid=>4, :parser=>:json){|s| s * 2}
-    Sequel::Postgres::PG_TYPES[4].call('{{1, 2}, {3, 4}}').must_equal [[2, 4], [6, 8]]
+  it "should support registering converters with blocks" do
+    Sequel::Postgres::PGArray.register('foo', :oid=>4){|s| s.to_i * 2}
+    Sequel::Postgres::PG_TYPES[4].call('{{1,2},{3,4}}').must_equal [[2, 4], [6, 8]]
   end
 
   it "should support registering custom types with :array_type option" do

data/spec/files/double_migration/001_create_sessions.rb

@@ -0,0 +1,9 @@
+class CreateSessions < Sequel::Migration
+  def up
+    create_table(:sm1111){Integer :smc1}
+  end
+  
+  def down
+    drop_table(:sm1111)
+  end
+end

data/spec/files/double_migration/002_create_nodes.rb

@@ -0,0 +1,19 @@
+Class.new(Sequel::Migration) do
+  def up
+    create_table(:sm2222){Integer :smc2}
+  end
+    
+  def down
+    drop_table(:sm2222)
+  end
+end
+
+Class.new(Sequel::Migration) do
+  def up
+    create_table(:sm2443){Integer :smc2}
+  end
+    
+  def down
+    drop_table(:sm2443)
+  end
+end

data/spec/files/double_migration/003_3_create_users.rb

@@ -0,0 +1,4 @@
+Sequel.migration do
+  up{create_table(:sm3333){Integer :smc3}}
+  down{drop_table(:sm3333)}
+end

data/spec/files/empty_migration/001_create_sessions.rb

@@ -0,0 +1,9 @@
+class CreateSessions < Sequel::Migration
+  def up
+    create_table(:sm1111){Integer :smc1}
+  end
+  
+  def down
+    drop_table(:sm1111)
+  end
+end

data/spec/files/empty_migration/003_3_create_users.rb

@@ -0,0 +1,4 @@
+Sequel.migration do
+  up{create_table(:sm3333){Integer :smc3}}
+  down{drop_table(:sm3333)}
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sequel
 version: !ruby/object:Gem::Version
-  version: 4.30.0
+  version: 4.31.0
 platform: ruby
 authors:
 - Jeremy Evans
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-04 00:00:00.000000000 Z
+date: 2016-02-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -231,6 +231,7 @@ extra_rdoc_files:
 - doc/release_notes/4.28.0.txt
 - doc/release_notes/4.29.0.txt
 - doc/release_notes/4.30.0.txt
+- doc/release_notes/4.31.0.txt
 files:
 - CHANGELOG
 - MIT-LICENSE
@@ -350,6 +351,7 @@ files:
 - doc/release_notes/4.29.0.txt
 - doc/release_notes/4.3.0.txt
 - doc/release_notes/4.30.0.txt
+- doc/release_notes/4.31.0.txt
 - doc/release_notes/4.4.0.txt
 - doc/release_notes/4.5.0.txt
 - doc/release_notes/4.6.0.txt
@@ -779,11 +781,17 @@ files:
 - spec/files/convert_to_timestamp_migrations/003_3_create_users.rb
 - spec/files/convert_to_timestamp_migrations/1273253850_create_artists.rb
 - spec/files/convert_to_timestamp_migrations/1273253852_create_albums.rb
+- spec/files/double_migration/001_create_sessions.rb
+- spec/files/double_migration/002_create_nodes.rb
+- spec/files/double_migration/003_3_create_users.rb
 - spec/files/duplicate_integer_migrations/001_create_alt_advanced.rb
 - spec/files/duplicate_integer_migrations/001_create_alt_basic.rb
 - spec/files/duplicate_timestamped_migrations/1273253849_create_sessions.rb
 - spec/files/duplicate_timestamped_migrations/1273253853_create_nodes.rb
 - spec/files/duplicate_timestamped_migrations/1273253853_create_users.rb
+- spec/files/empty_migration/001_create_sessions.rb
+- spec/files/empty_migration/002_create_nodes.rb
+- spec/files/empty_migration/003_3_create_users.rb
 - spec/files/integer_migrations/001_create_sessions.rb
 - spec/files/integer_migrations/002_create_nodes.rb
 - spec/files/integer_migrations/003_3_create_users.rb
@@ -867,7 +875,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: The Database Toolkit for Ruby