sequel 3.34.1 → 3.35.0

data/spec/extensions/pg_auto_parameterize_spec.rb

@@ -3,8 +3,9 @@ require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
 describe "pg_auto_parameterize extension" do
   before do
     @db = Sequel.connect('mock://postgres', :quote_identifiers=>false)
-    @db.extend Sequel::Postgres::AutoParameterize::DatabaseMethods
     @db.synchronize{|c| def c.escape_bytea(v) v*2 end}
+    @db.extend_datasets{def use_cursor(*) clone end}
+    @db.extend Sequel::Postgres::AutoParameterize::DatabaseMethods
   end
 
   it "should automatically parameterize queries strings, blobs, numerics, dates, and times" do
@@ -44,10 +45,6 @@ describe "pg_auto_parameterize extension" do
 
     @db[:table].insert(:a=>1).should == 1
     @db.sqls.should == ['INSERT INTO table (a) VALUES ($1::int4) RETURNING id -- args: [1]']
-
-    @db.server_version = 80000
-    @db[:table].insert(:a=>1).should == 1
-    @db.sqls.should == ['INSERT INTO table (a) VALUES ($1::int4) -- args: [1]']
   end
 
   it "should not automatically paramiterize if no_auto_parameterize is used" do
@@ -61,4 +58,8 @@ describe "pg_auto_parameterize extension" do
   it "should show args with string when inspecting SQL " do
     @db[:table].filter(:a=>1).sql.inspect.should == '"SELECT * FROM table WHERE (a = $1::int4); [1]"'
   end
+
+  it "should not auto parameterize when using cursors" do
+    @db[:table].filter(:a=>1).use_cursor.opts[:no_auto_parameterize].should be_true
+  end
 end

data/spec/extensions/schema_dumper_spec.rb

@@ -508,7 +508,7 @@ END_MIG
       ["double precision", "timestamp with time zone", "timestamp without time zone",
        "time with time zone", "time without time zone", "character varying(20)"] +
       %w"nvarchar ntext smalldatetime smallmoney binary varbinary nchar" +
-      ["timestamp(6) without time zone", "timestamp(6) with time zone", "int(12) unsigned", 'bigint unsigned', 'tinyint(3) unsigned']
+      ["timestamp(6) without time zone", "timestamp(6) with time zone", "int(12) unsigned", 'bigint unsigned', 'tinyint(3) unsigned', 'identity', 'int identity']
     @d.meta_def(:schema) do |t, *o|
       i = 0
       types.map{|x| [:"c#{i+=1}", {:db_type=>x, :allow_null=>true}]}
@@ -581,6 +581,8 @@ create_table(:x) do
   Integer :c64
   Bignum :c65
   Integer :c66
+  Integer :c67
+  Integer :c68
 end
 END_MIG
   end

data/spec/extensions/xml_serializer_spec.rb

@@ -157,6 +157,19 @@ describe "Sequel::Plugins::XmlSerializer" do
     Album.array_from_xml(Album.dataset.to_xml(:only=>:name)).should == [Album.load(:name=>@album.name)]
   end
 
+  it "should have to_xml dataset method respect an :array option" do
+    a = Album.load(:id=>1, :name=>'RF', :artist_id=>3)
+    Album.array_from_xml(Album.to_xml(:array=>[a])).should == [a]
+
+    a.associations[:artist] = artist = Artist.load(:id=>3, :name=>'YJM')
+    Album.array_from_xml(Album.to_xml(:array=>[a], :include=>:artist)).first.artist.should == artist
+
+    artist.associations[:albums] = [a]
+    x = Artist.array_from_xml(Artist.to_xml(:array=>[artist], :include=>:albums))
+    x.should == [artist]
+    x.first.albums.should == [a]
+  end
+
   it "should raise an error if the dataset does not have a row_proc" do
     proc{Album.dataset.naked.to_xml}.should raise_error(Sequel::Error)
   end

data/spec/files/{transactionless_migrations → transaction_specified_migrations}/001_create_alt_basic.rb

@@ -1,4 +1,4 @@
 Sequel.migration do
-  no_transaction
+  transaction
   change{create_table(:sm11111){Integer :smc1}}
 end

data/spec/integration/associations_test.rb

@@ -608,7 +608,7 @@ describe "Sequel::Model Simple Associations" do
       @els = {:eager_limit_strategy=>:correlated_subquery}
     end
     it_should_behave_like "eager limit strategies"
-  end unless Sequel.guarded?(:mysql, :db2, :oracle)
+  end unless Sequel.guarded?(:mysql, :db2, :oracle, :h2)
 
   specify "should handle many_to_one associations with same name as :key" do
     Album.def_column_alias(:artist_id_id, :artist_id)
@@ -693,12 +693,10 @@ describe "Sequel::Model Simple Associations" do
     @album.artist(proc{|ds| ds.exclude(:id=>@artist.id)}).should == nil
     @album.artist(proc{|ds| ds.filter(:id=>@artist.id)}).should == @artist
 
-    if RUBY_VERSION >= '1.8.7'
-      @artist.albums{|ds| ds.exclude(:id=>@album.id)}.should == []
-      @artist.albums{|ds| ds.filter(:id=>@album.id)}.should == [@album]
-      @album.artist{|ds| ds.exclude(:id=>@artist.id)}.should == nil
-      @album.artist{|ds| ds.filter(:id=>@artist.id)}.should == @artist
-    end
+    @artist.albums{|ds| ds.exclude(:id=>@album.id)}.should == []
+    @artist.albums{|ds| ds.filter(:id=>@album.id)}.should == [@album]
+    @album.artist{|ds| ds.exclude(:id=>@artist.id)}.should == nil
+    @album.artist{|ds| ds.filter(:id=>@artist.id)}.should == @artist
   end
   
   specify "should handle dynamic callbacks for eager loading via eager and eager_graph" do

data/spec/integration/dataset_test.rb

@@ -291,15 +291,26 @@ end
 
 describe Sequel::Database do
   specify "should correctly escape strings" do
-    INTEGRATION_DB.get("\\dingo".as(:a)) == "\\dingo"
-  end
-
-  specify "should correctly escape strings with quotes" do
-    INTEGRATION_DB.get("\\'dingo".as(:a)) == "\\'dingo"
+    ["\\\n",
+     "\\\\\n",
+     "\\\r\n",
+     "\\\\\r\n",
+     "\\\\\n\n", 
+     "\\\\\r\n\r\n",
+     "\\dingo",
+     "\\'dingo",
+     "\\\\''dingo",
+    ].each do |str|
+      INTEGRATION_DB.get(str.as(:a)).should == str
+      str = "1#{str}1"
+      INTEGRATION_DB.get(str.as(:a)).should == str
+      str = "#{str}#{str}"
+      INTEGRATION_DB.get(str.as(:a)).should == str
+    end
   end
 
-  specify "should properly escape binary data" do
-    INTEGRATION_DB.get("\1\2\3".to_sequel_blob.as(:a)) == "\1\2\3"
+  cspecify "should properly escape binary data", [:odbc], [:jdbc, :hsqldb], [:swift], :oracle do
+    INTEGRATION_DB.get("\1\2\3".to_sequel_blob.cast(File).as(:a)).should == "\1\2\3"
   end
 
   specify "should have a working table_exists?" do
@@ -1115,6 +1126,13 @@ describe "Sequel::Dataset DSL support" do
     @ds.get{a.sql_number ^ b ^ 1}.to_i.should == 7
   end
   
+  specify "should work with the modulus operator" do
+    @ds.insert(3, 5)
+    @ds.get{a.sql_number % 4}.to_i.should == 3
+    @ds.get{b.sql_number % 4}.to_i.should == 1
+    @ds.get{a.sql_number % 4 % 2}.to_i.should == 1
+  end
+  
   specify "should work with inequality operators" do
     @ds.insert(10, 11)
     @ds.insert(11, 11)

data/spec/integration/plugin_test.rb

@@ -914,7 +914,7 @@ if INTEGRATION_DB.dataset.supports_cte? and !Sequel.guarded?(:db2)
       nodes[0].associations.fetch(:parent, 1).should == nil
       nodes[1].associations[:parent].should == @aa
       nodes[1].associations[:parent].associations[:parent].should == @a
-      nodes[1].associations[:parent].associations[:parent].associations.fetch(:parent, 1) == nil
+      nodes[1].associations[:parent].associations[:parent].associations.fetch(:parent, 1).should == nil
       nodes[2].associations[:parent].should == @aaaa
       nodes[2].associations[:parent].associations[:parent].should == @aaa
       nodes[2].associations[:parent].associations[:parent].associations[:parent].should == @aa

data/spec/integration/schema_test.rb

@@ -239,7 +239,7 @@ describe "Database schema modifiers" do
     @ds = @db[:items]
   end
   after do
-    @db.drop_table?(:items)
+    @db.drop_table?(:items, :items2)
   end
 
   specify "should create tables correctly" do
@@ -250,6 +250,21 @@ describe "Database schema modifiers" do
     @ds.columns!.should == [:number]
   end
   
+  specify "should create tables from select statements correctly" do
+    @db.create_table!(:items){Integer :number}
+    @ds.insert([10])
+    @db.create_table(:items2, :as=>@db[:items])
+    @db.schema(:items2, :reload=>true).map{|x| x.first}.should == [:number]
+    @db[:items2].columns.should == [:number]
+    @db[:items2].all.should == [{:number=>10}]
+  end
+  
+  specify "should handle create table in a rolled back transaction" do
+    @db.drop_table?(:items)
+    @db.transaction(:rollback=>:always){@db.create_table(:items){Integer :number}}
+    @db.table_exists?(:items).should be_false
+  end if INTEGRATION_DB.supports_transactional_ddl?
+  
   describe "join tables" do
     after do
       @db.drop_join_table(:cat_id=>:cats, :dog_id=>:dogs) if @db.table_exists?(:cats_dogs)

data/spec/model/associations_spec.rb

@@ -382,7 +382,7 @@ describe Sequel::Model, "many_to_one" do
     MODEL_DB.sqls.should == ["SELECT * FROM nodes WHERE ((nodes.id = 234) AND (name > 'M')) LIMIT 1"]
   end
   
-  it "should use a block given to the association method as a callback on ruby 1.8.7+" do
+  it "should use a block given to the association method as a callback" do
     @c2.many_to_one :parent, :class => @c2
 
     d = @c2.create(:id => 1)
@@ -391,7 +391,7 @@ describe Sequel::Model, "many_to_one" do
     d.associations[:parent] = 42
     d.parent{|ds| ds.filter{name > 'M'}}.should_not == 42 
     MODEL_DB.sqls.should == ["SELECT * FROM nodes WHERE ((nodes.id = 234) AND (name > 'M')) LIMIT 1"]
-  end if RUBY_VERSION >= '1.8.7'
+  end
   
   it "should have the setter add to the reciprocal one_to_many cached association list if it exists" do
     @c2.many_to_one :parent, :class => @c2

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sequel
 version: !ruby/object:Gem::Version
-  version: 3.34.1
+  version: 3.35.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-02 00:00:00.000000000 Z
+date: 2012-05-01 00:00:00.000000000 Z
 dependencies: []
 description: The Database Toolkit for Ruby
 email: code@jeremyevans.net
@@ -40,6 +40,7 @@ extra_rdoc_files:
 - doc/testing.rdoc
 - doc/schema_modification.rdoc
 - doc/transactions.rdoc
+- doc/thread_safety.rdoc
 - doc/release_notes/1.0.txt
 - doc/release_notes/1.1.txt
 - doc/release_notes/1.3.txt
@@ -93,6 +94,7 @@ extra_rdoc_files:
 - doc/release_notes/3.32.0.txt
 - doc/release_notes/3.33.0.txt
 - doc/release_notes/3.34.0.txt
+- doc/release_notes/3.35.0.txt
 files:
 - MIT-LICENSE
 - CHANGELOG
@@ -164,6 +166,7 @@ files:
 - doc/release_notes/3.32.0.txt
 - doc/release_notes/3.33.0.txt
 - doc/release_notes/3.34.0.txt
+- doc/release_notes/3.35.0.txt
 - doc/sharding.rdoc
 - doc/sql.rdoc
 - doc/validations.rdoc
@@ -172,6 +175,7 @@ files:
 - doc/testing.rdoc
 - doc/schema_modification.rdoc
 - doc/transactions.rdoc
+- doc/thread_safety.rdoc
 - spec/adapters/firebird_spec.rb
 - spec/adapters/informix_spec.rb
 - spec/adapters/mssql_spec.rb
@@ -185,9 +189,9 @@ files:
 - spec/core/dataset_spec.rb
 - spec/core/database_spec.rb
 - spec/core/expression_filters_spec.rb
+- spec/core/schema_spec.rb
 - spec/core/object_graph_spec.rb
 - spec/core/schema_generator_spec.rb
-- spec/core/schema_spec.rb
 - spec/core/spec_helper.rb
 - spec/core/version_spec.rb
 - spec/core/mock_adapter_spec.rb
@@ -264,6 +268,7 @@ files:
 - spec/extensions/select_remove_spec.rb
 - spec/extensions/static_cache_spec.rb
 - spec/extensions/query_literals_spec.rb
+- spec/extensions/dirty_spec.rb
 - spec/extensions/columns_introspection_spec.rb
 - spec/integration/associations_test.rb
 - spec/integration/database_test.rb
@@ -333,10 +338,10 @@ files:
 - spec/files/reversible_migrations/003_reversible.rb
 - spec/files/reversible_migrations/004_reversible.rb
 - spec/files/reversible_migrations/005_reversible.rb
-- spec/files/transaction_migrations/001_create_alt_basic.rb
-- spec/files/transaction_migrations/002_create_basic.rb
-- spec/files/transactionless_migrations/001_create_alt_basic.rb
-- spec/files/transactionless_migrations/002_create_basic.rb
+- spec/files/transaction_specified_migrations/001_create_alt_basic.rb
+- spec/files/transaction_specified_migrations/002_create_basic.rb
+- spec/files/transaction_unspecified_migrations/001_create_alt_basic.rb
+- spec/files/transaction_unspecified_migrations/002_create_basic.rb
 - lib/sequel.rb
 - lib/sequel/adapters/ado.rb
 - lib/sequel/adapters/ado/mssql.rb
@@ -368,7 +373,6 @@ files:
 - lib/sequel/adapters/mysql.rb
 - lib/sequel/adapters/odbc.rb
 - lib/sequel/adapters/odbc/mssql.rb
-- lib/sequel/adapters/odbc/db2.rb
 - lib/sequel/adapters/openbase.rb
 - lib/sequel/adapters/oracle.rb
 - lib/sequel/adapters/postgres.rb
@@ -503,6 +507,7 @@ files:
 - lib/sequel/plugins/prepared_statements_associations.rb
 - lib/sequel/plugins/dataset_associations.rb
 - lib/sequel/plugins/many_to_one_pk_lookup.rb
+- lib/sequel/plugins/dirty.rb
 - lib/sequel/timezones.rb
 - lib/sequel/version.rb
 - lib/sequel/ast_transformer.rb
@@ -527,7 +532,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.8.4
+      version: 1.8.7
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:

data/lib/sequel/adapters/odbc/db2.rb

@@ -1,17 +0,0 @@
-Sequel.require 'adapters/shared/db2'
-
-module Sequel
-  module ODBC
-    # Database and Dataset instance methods for DB2 specific
-    # support via ODBC.
-    module DB2
-      module DatabaseMethods
-        include ::Sequel::DB2::DatabaseMethods
-      end
-      
-      class Dataset < ODBC::Dataset
-        include ::Sequel::DB2::DatasetMethods
-      end
-    end
-  end
-end