sequel 4.49.0 → 5.0.0

data/spec/extensions/column_conflicts_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "column_conflicts plugin" do
   before do

data/spec/extensions/column_select_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::ColumnSelect" do
   def set_cols(*cols)
@@ -63,13 +63,13 @@ describe "Sequel::Plugins::ColumnSelect" do
     @Album.dataset.sql.must_equal 'SELECT name, artist FROM albums'
   end
 
-  deprecated "should not add a explicit column selection on existing dataset with multiple tables" do
+  it "should work with implicit subqueries used for joined datasets" do
     @Album.dataset = @Album.db.from(:a1, :a2)
     @Album.plugin :column_select
-    @Album.dataset.sql.must_equal 'SELECT * FROM a1, a2'
+    @Album.dataset.sql.must_equal "SELECT a1.id, a1.a, a1.b, a1.c FROM (SELECT * FROM a1, a2) AS a1"
 
     @Album.dataset = @Album.db.from(:a1).cross_join(:a2)
-    @Album.dataset.sql.must_equal 'SELECT * FROM a1 CROSS JOIN a2'
+    @Album.dataset.sql.must_equal "SELECT a1.id, a1.a, a1.b, a1.c FROM (SELECT * FROM a1 CROSS JOIN a2) AS a1"
   end
 
   it "should add a explicit column selection on existing dataset with a subquery" do

data/spec/extensions/columns_introspection_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), 'spec_helper')
+require_relative "spec_helper"
 
 Sequel.extension :columns_introspection
 

data/spec/extensions/columns_updated_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::ColumnsUpdated" do
   before do

data/spec/extensions/composition_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Composition plugin" do
   before do
@@ -9,12 +9,6 @@ describe "Composition plugin" do
     DB.reset
   end
   
-  deprecated "should allow access to composition_module" do
-    @c.composition_module.must_be_kind_of Module
-    @c.composition_module = v = Module.new
-    @c.composition_module.must_equal v
-  end
-
   it ".composition should add compositions" do
     @o.wont_respond_to(:date)
     @c.composition :date, :mapping=>[:year, :month, :day]

data/spec/extensions/connection_expiration_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 connection_expiration_specs = shared_description do
   describe "connection expiration" do
@@ -109,13 +109,13 @@ end
 
 describe "Sequel::ConnectionExpiration with threaded pool" do
   before do
-    @db = Sequel.mock
+    @db = Sequel.mock(:test=>false)
   end
   include connection_expiration_specs
 end
 describe "Sequel::ConnectionExpiration with sharded threaded pool" do
   before do
-    @db = Sequel.mock(:servers=>{})
+    @db = Sequel.mock(:test=>false, :servers=>{})
   end
   include connection_expiration_specs
 end

data/spec/extensions/connection_validator_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 connection_validator_specs = shared_description do
   describe "connection validator" do
@@ -114,13 +114,13 @@ end
 
 describe "Sequel::ConnectionValidator with threaded pool" do
   before do
-    @db = Sequel.mock
+    @db = Sequel.mock(:test=>false)
   end
   include connection_validator_specs
 end
 describe "Sequel::ConnectionValidator with sharded threaded pool" do
   before do
-    @db = Sequel.mock(:servers=>{})
+    @db = Sequel.mock(:test=>false, :servers=>{})
   end
   include connection_validator_specs
 end

data/spec/extensions/constraint_validations_plugin_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::ConstraintValidations" do
   def model_class(opts={})

data/spec/extensions/constraint_validations_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "constraint_validations extension" do
   def parse_insert(s)

data/spec/extensions/core_refinements_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 if (RUBY_VERSION >= '2.0.0' && RUBY_ENGINE == 'ruby') # || (RUBY_VERSION >= '2.3.0' && RUBY_ENGINE == 'jruby')
 Sequel.extension :core_refinements, :pg_array, :pg_hstore, :pg_row, :pg_range, :pg_row_ops, :pg_range_ops, :pg_array_ops, :pg_hstore_ops, :pg_json, :pg_json_ops
@@ -525,6 +525,4 @@ describe "Postgres extensions integration" do
     @db.literal((1..2).pg_range(:int4range)).must_equal "int4range(1,2,'[]')"
   end
 end
-else
-  skip_warn "core_refinements extension: only works on ruby 2.0+"
 end

data/spec/extensions/csv_serializer_spec.rb

@@ -1,11 +1,7 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
-
-begin
-  csv_lib = RUBY_VERSION < '1.9' ? 'fastercsv' : 'csv'
-  require csv_lib
-rescue LoadError => e
-  skip_warn "csv_serializer plugin: can't load #{csv_lib} (#{e.class}: #{e})"
-else
+require_relative "spec_helper"
+
+require 'csv' 
+
 describe "Sequel::Plugins::CsvSerializer" do
   before do
     artist = @Artist = Class.new(Sequel::Model(:artists))
@@ -185,4 +181,3 @@ describe "Sequel::Plugins::CsvSerializer" do
     @Album.csv_serializer_opts[:only].frozen?.must_equal true
   end
 end
-end

data/spec/extensions/current_datetime_timestamp_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), 'spec_helper')
+require_relative "spec_helper"
 
 describe "current_datetime_timestamp extension" do
   before do

data/spec/extensions/dataset_associations_spec.rb

@@ -1,9 +1,10 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::DatasetAssociations" do
   before do
     @db = Sequel.mock(:host=>'postgres')
     @db.extend_datasets do
+      def quote_identifiers?; false end
       def supports_window_functions?; true; end
       def supports_distinct_on?; true; end
     end

data/spec/extensions/dataset_source_alias_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "dataset_source_alias extension" do
   before do

data/spec/extensions/date_arithmetic_spec.rb

@@ -1,10 +1,10 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 asd = begin
   require 'active_support/duration'
   true
-rescue LoadError => e
-  skip_warn "date_arithmetic extension (partial): can't load active_support/duration (#{e.class}: #{e})"
+rescue LoadError
+  warn "Skipping some tests of date_arithmetic extension: can't load active_support/duration"
   false
 end
 

data/spec/extensions/def_dataset_method_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe Sequel::Model, ".def_dataset_method" do
   before do

data/spec/extensions/defaults_setter_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::DefaultsSetter" do
   before do
@@ -9,7 +9,7 @@ describe "Sequel::Plugins::DefaultsSetter" do
     @c.instance_variable_set(:@db_schema, {:a=>{}})
     @c.plugin :defaults_setter
     @c.columns :a
-    @pr = proc{|x| meta_def(db, :schema){|*| [[:id, {:primary_key=>true}], [:a, {:ruby_default => x, :primary_key=>false}]]}; c.dataset = c.dataset; c}
+    @pr = proc{|x| db.define_singleton_method(:schema){|*| [[:id, {:primary_key=>true}], [:a, {:ruby_default => x, :primary_key=>false}]]}; c.dataset = c.dataset; c}
   end
   after do
     Sequel.datetime_class = Time

data/spec/extensions/delay_add_association_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::DelayAddAssociation" do
   before do
@@ -16,7 +16,7 @@ describe "Sequel::Plugins::DelayAddAssociation" do
     @o.add_c(@c.load(:id=>2, :name=>'b'))
     @db.sqls.must_equal []
     @o.save
-    @db.sqls.must_equal ["INSERT INTO cs (name) VALUES ('a')", "UPDATE cs SET c_id = 1 WHERE (id = 2)", "SELECT * FROM cs WHERE (id = 1) LIMIT 1"]
+    @db.sqls.must_equal ["INSERT INTO cs (name) VALUES ('a')", "SELECT * FROM cs WHERE (id = 1) LIMIT 1", "UPDATE cs SET c_id = 1 WHERE (id = 2)"]
   end
 
   it "should immediately reflect changes in cached association" do
@@ -55,12 +55,11 @@ describe "Sequel::Plugins::DelayAddAssociation" do
     @o.add_c(:name=>'b')
     @db.sqls.must_equal []
     @o.save
-    sqls = @db.sqls
-    sqls.length.must_equal 4
-    sqls[0].must_equal "INSERT INTO cs (name) VALUES ('a')"
-    ["INSERT INTO cs (name, c_id) VALUES ('b', 1)", "INSERT INTO cs (c_id, name) VALUES (1, 'b')"].must_include sqls[1]
-    sqls[2].must_equal "SELECT * FROM cs WHERE (id = 2) LIMIT 1"
-    sqls[3].must_equal "SELECT * FROM cs WHERE (id = 1) LIMIT 1"
+    @db.sqls.must_equal [
+      "INSERT INTO cs (name) VALUES ('a')",
+      "SELECT * FROM cs WHERE (id = 1) LIMIT 1",
+      "INSERT INTO cs (name, c_id) VALUES ('b', 1)",
+      "SELECT * FROM cs WHERE (id = 2) LIMIT 1"]
   end
 
   it "should work when passing in primary keys" do
@@ -69,6 +68,6 @@ describe "Sequel::Plugins::DelayAddAssociation" do
     @o.add_c(2)
     @db.sqls.must_equal ["SELECT * FROM cs WHERE (id = 2) LIMIT 1"]
     @o.save
-    @db.sqls.must_equal ["INSERT INTO cs (name) VALUES ('a')", "UPDATE cs SET c_id = 1 WHERE (id = 2)", "SELECT * FROM cs WHERE (id = 1) LIMIT 1"]
+    @db.sqls.must_equal ["INSERT INTO cs (name) VALUES ('a')", "SELECT * FROM cs WHERE (id = 1) LIMIT 1", "UPDATE cs SET c_id = 1 WHERE (id = 2)"]
   end
 end

data/spec/extensions/dirty_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::Dirty" do
   before do

data/spec/extensions/duplicate_columns_handler_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 mod = shared_description do
   it "should take action depending on :on_duplicate_columns if 2 or more columns have the same name" do

data/spec/extensions/eager_each_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::EagerEach" do
   before do
@@ -57,6 +57,6 @@ describe "Sequel::Plugins::EagerEach" do
   end
 
   it "should not attempt to eager load when getting the columns" do
-    ds = @c.eager(:children).with_extend{def all; raise; end}.columns!.must_equal [:id, :parent_id]
+    @c.eager(:children).with_extend{def all; raise; end}.columns!.must_equal [:id, :parent_id]
   end
 end

data/spec/extensions/empty_array_consider_nulls_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "filter_having extension" do
   before do

data/spec/extensions/error_splitter_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "Sequel::Plugins::ErrorSplitter" do
   before do

data/spec/extensions/error_sql_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "error_sql extension" do
   before do

data/spec/extensions/eval_inspect_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 Sequel.extension :eval_inspect
 

data/spec/extensions/finder_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe Sequel::Model, ".finder" do
   before do

data/spec/extensions/force_encoding_spec.rb

@@ -1,5 +1,5 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
-if RUBY_VERSION >= '1.9.0'
+require_relative "spec_helper"
+
 describe "force_encoding plugin" do
   before do
     @c = Class.new(Sequel::Model)
@@ -124,6 +124,3 @@ describe "force_encoding plugin" do
     o.x.encoding.must_equal @e1
   end
 end 
-else
-  skip_warn "force_encoding plugin: only works on ruby 1.9+"
-end

data/spec/extensions/freeze_datasets_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe "freeze_datasets extension" do
   before do

data/spec/extensions/graph_each_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 describe Sequel::Dataset, " graphing" do
   before do
@@ -76,7 +76,7 @@ describe Sequel::Dataset, " graphing" do
   end
 
   it "#graph_each should only include the columns selected with #set_graph_aliases and #add_graph_aliases, if called" do
-    @db.fetch = [nil, [{:x=>2,:y=>3}], nil, [{:x=>2}], [{:x=>2, :q=>18}]]
+    @db.fetch = [[{:x=>2,:y=>3}], [{:x=>2}], [{:x=>2, :q=>18}]]
 
     @ds1.graph(:lines, :x=>:id).set_graph_aliases(:x=>[:points, :x], :y=>[:lines, :y]).all.must_equal [{:points=>{:x=>2}, :lines=>{:y=>3}}]
     ds = @ds1.graph(:lines, :x=>:id).set_graph_aliases(:x=>[:points, :x])
@@ -86,17 +86,17 @@ describe Sequel::Dataset, " graphing" do
   end
 
   it "#graph_each should correctly map values when #set_graph_aliases is used with a third argument for each entry" do
-    @db.fetch = [nil, {:x=>2,:y=>3}]
+    @db.fetch = [{:x=>2,:y=>3}]
     @ds1.graph(:lines, :x=>:id).set_graph_aliases(:x=>[:points, :z1, 2], :y=>[:lines, :z2, Sequel.function(:random)]).all.must_equal [{:points=>{:z1=>2}, :lines=>{:z2=>3}}]
   end
 
   it "#graph_each should correctly map values when #set_graph_aliases is used with a single argument for each entry" do
-    @db.fetch = [nil, {:x=>2,:y=>3}]
+    @db.fetch = [{:x=>2,:y=>3}]
     @ds1.graph(:lines, :x=>:id).set_graph_aliases(:x=>[:points], :y=>[:lines]).all.must_equal [{:points=>{:x=>2}, :lines=>{:y=>3}}]
   end
 
   it "#graph_each should correctly map values when #set_graph_aliases is used with a symbol for each entry" do
-    @db.fetch = [nil, {:x=>2,:y=>3}]
+    @db.fetch = [{:x=>2,:y=>3}]
     @ds1.graph(:lines, :x=>:id).set_graph_aliases(:x=>:points, :y=>:lines).all.must_equal [{:points=>{:x=>2}, :lines=>{:y=>3}}]
   end
 

data/spec/extensions/hook_class_methods_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(File.expand_path(__FILE__)), "spec_helper")
+require_relative "spec_helper"
 
 model_class = proc do |klass, &block|
   c = Class.new(klass)
@@ -140,39 +140,6 @@ describe Sequel::Model, "hook_class_methods plugin" do
     adds.must_equal ['456']
   end
   
-  deprecated "should stop processing if a before hook returns false" do
-    flag = true
-    adds = []
-    
-    a = model_class.call Sequel::Model do
-      before_save{adds << 'cruel'; flag}
-      before_save{adds << 'blah'; flag}
-    end
-    
-    a.new.before_save
-    adds.must_equal ['blah', 'cruel']
-
-    # chain should not break on nil
-    adds = []
-    flag = nil
-    a.new.before_save
-    adds.must_equal ['blah', 'cruel']
-    
-    adds = []
-    flag = false
-    a.new.before_save
-    adds.must_equal ['blah']
-    
-    b = Class.new(a)
-    b.class_eval do
-      before_save{adds << 'mau'}
-    end
-    
-    adds = []
-    b.new.before_save
-    adds.must_equal ['mau', 'blah']
-  end
-
   it "should stop processing if a before hook calls cancel_action" do
     flag = true
     adds = []
@@ -226,25 +193,12 @@ describe "Model#before_create && Model#after_create" do
     DB.sqls.must_equal ['BLAH before', 'INSERT INTO items (x) VALUES (2)', 'BLAH after']
   end
 
-  deprecated ".create should cancel the save and raise an error if before_create returns false and raise_on_save_failure is true" do
-    @c.before_create{false}
-    proc{@c.create(:x => 2)}.must_raise(Sequel::HookFailed)
-    DB.sqls.must_equal []
-  end
-
   it ".create should cancel the save and raise an error if before_create calls cancel_action and raise_on_save_failure is true" do
     @c.before_create{cancel_action}
     proc{@c.create(:x => 2)}.must_raise(Sequel::HookFailed)
     DB.sqls.must_equal []
   end
 
-  deprecated ".create should cancel the save and return nil if before_create returns false and raise_on_save_failure is false" do
-    @c.before_create{false}
-    @c.raise_on_save_failure = false
-    @c.create(:x => 2).must_be_nil
-    DB.sqls.must_equal []
-  end
-
   it ".create should cancel the save and return nil if before_create calls cancel_action and raise_on_save_failure is false" do
     @c.before_create{cancel_action}
     @c.raise_on_save_failure = false
@@ -269,25 +223,12 @@ describe "Model#before_update && Model#after_update" do
     DB.sqls.must_equal ['BLAH before', 'UPDATE items SET x = 123 WHERE (id = 2233)', 'BLAH after']
   end
 
-  deprecated "#save should cancel the save and raise an error if before_update returns false and raise_on_save_failure is true" do
-    @c.before_update{false}
-    proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and raise an error if before_update calls cancel_action and raise_on_save_failure is true" do
     @c.before_update{cancel_action}
     proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
     DB.sqls.must_equal []
   end
 
-  deprecated "#save should cancel the save and return nil if before_update returns false and raise_on_save_failure is false" do
-    @c.before_update{false}
-    @c.raise_on_save_failure = false
-    @c.load(:id => 2233).save.must_be_nil
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and return nil if before_update calls cancel_action and raise_on_save_failure is false" do
     @c.before_update{cancel_action}
     @c.raise_on_save_failure = false
@@ -320,25 +261,12 @@ describe "Model#before_save && Model#after_save" do
     DB.sqls.must_equal ['BLAH before', 'INSERT INTO items (x) VALUES (2)', 'BLAH after']
   end
 
-  deprecated "#save should cancel the save and raise an error if before_save returns false and raise_on_save_failure is true" do
-    @c.before_save{false}
-    proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and raise an error if before_save calls cancel_action and raise_on_save_failure is true" do
     @c.before_save{cancel_action}
     proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
     DB.sqls.must_equal []
   end
 
-  deprecated "#save should cancel the save and return nil if before_save returns false and raise_on_save_failure is false" do
-    @c.before_save{false}
-    @c.raise_on_save_failure = false
-    @c.load(:id => 2233).save.must_be_nil
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and return nil if before_save calls cancel_action and raise_on_save_failure is false" do
     @c.before_save{cancel_action}
     @c.raise_on_save_failure = false
@@ -363,25 +291,12 @@ describe "Model#before_destroy && Model#after_destroy" do
     DB.sqls.must_equal ['BLAH before', "DELETE FROM items WHERE id = 2233", 'BLAH after']
   end
 
-  deprecated "#destroy should cancel the destroy and raise an error if before_destroy returns false and raise_on_save_failure is true" do
-    @c.before_destroy{false}
-    proc{@c.load(:id => 2233).destroy}.must_raise(Sequel::HookFailed)
-    DB.sqls.must_equal []
-  end
-
   it "#destroy should cancel the destroy and raise an error if before_destroy calls cancel_action and raise_on_save_failure is true" do
     @c.before_destroy{cancel_action}
     proc{@c.load(:id => 2233).destroy}.must_raise(Sequel::HookFailed)
     DB.sqls.must_equal []
   end
 
-  deprecated "#destroy should cancel the destroy and return nil if before_destroy returns false and raise_on_save_failure is false" do
-    @c.before_destroy{false}
-    @c.raise_on_save_failure = false
-    @c.load(:id => 2233).destroy.must_be_nil
-    DB.sqls.must_equal []
-  end
-
   it "#destroy should cancel the destroy and return nil if before_destroy calls cancel_action and raise_on_save_failure is false" do
     @c.before_destroy{cancel_action}
     @c.raise_on_save_failure = false
@@ -430,25 +345,12 @@ describe "Model#before_validation && Model#after_validation" do
     DB.sqls.must_equal ['BLAH before', 'BLAH after']
   end
 
-  deprecated "#save should cancel the save and raise an error if before_validation returns false and raise_on_save_failure is true" do
-    @c.before_validation{false}
-    proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and raise an error if before_validation calls cancel_action and raise_on_save_failure is true" do
     @c.before_validation{cancel_action}
     proc{@c.load(:id => 2233).save}.must_raise(Sequel::HookFailed)
     DB.sqls.must_equal []
   end
 
-  deprecated "#save should cancel the save and return nil if before_validation returns false and raise_on_save_failure is false" do
-    @c.before_validation{false}
-    @c.raise_on_save_failure = false
-    @c.load(:id => 2233).save.must_be_nil
-    DB.sqls.must_equal []
-  end
-
   it "#save should cancel the save and return nil if before_validation calls cancel_action and raise_on_save_failure is false" do
     @c.before_validation{cancel_action}
     @c.raise_on_save_failure = false
@@ -457,52 +359,6 @@ describe "Model#before_validation && Model#after_validation" do
   end
 end
 
-describe "Model transaction hooks" do
-  before do
-    DB.reset
-
-    @c = model_class.call(Sequel::Model(:items)) do
-      columns :x
-      after_save {DB << "AS"}
-      after_destroy {DB << "AD"}
-      self.use_transactions = true
-    end
-  end
-  
-  deprecated "should call after_commit or after_rollback depending on whether the transaction commits or rolls back" do
-    @c.after_commit{DB << 'AC'}
-    @c.after_rollback{DB << 'AR'}
-    m = @c.load(:id => 2233, :x=>123)
-
-    m.save
-    DB.sqls.must_equal ['BEGIN', 'UPDATE items SET x = 123 WHERE (id = 2233)', 'AS', 'COMMIT', 'AC']
-
-    @c.db.transaction(:rollback=>:always){m.save}
-    DB.sqls.must_equal ['BEGIN', 'UPDATE items SET x = 123 WHERE (id = 2233)', 'AS', 'ROLLBACK', 'AR']
-
-    @c.db.transaction do
-      m.save
-      DB.sqls.must_equal ['BEGIN', 'UPDATE items SET x = 123 WHERE (id = 2233)', 'AS']
-    end
-  end
-  
-  deprecated "should call after_destroy_commit or after_destroy_rollback depending on whether the transaction commits or rolls back" do
-    @c.after_destroy_commit {DB << 'ADC'}
-    @c.after_destroy_rollback{DB << 'ADR'}
-
-    @c.load(:id => 2233).destroy
-    DB.sqls.must_equal ['BEGIN', 'DELETE FROM items WHERE id = 2233', 'AD', 'COMMIT', 'ADC']
-
-    @c.db.transaction(:rollback=>:always){@c.load(:id => 2233).destroy}
-    DB.sqls.must_equal ['BEGIN', 'DELETE FROM items WHERE id = 2233', 'AD', 'ROLLBACK', 'ADR']
-
-    @c.db.transaction do
-      @c.load(:id => 2233).destroy
-      DB.sqls.must_equal ['BEGIN', 'DELETE FROM items WHERE id = 2233', 'AD']
-    end
-  end
-end
-
 describe "Model.has_hooks?" do
   before do
     @c = model_class.call(Sequel::Model(:items))
@@ -522,52 +378,3 @@ describe "Model.has_hooks?" do
     @d.has_hooks?(:before_save).must_equal false
   end
 end
-
-describe "Model#add_hook_type" do
-  before do
-    deprecated do
-      class ::Foo < Sequel::Model(:items)
-        plugin :hook_class_methods
-        add_hook_type :before_bar, :after_bar
-
-        def bar
-          return :b if before_bar == false
-          return :a if after_bar == false
-          true
-        end
-      end
-      @f = Class.new(Foo)
-    end
-  end
-  after do
-    Object.send(:remove_const, :Foo)
-  end
-
-  deprecated "should have before_bar and after_bar class methods" do
-    @f.must_respond_to(:before_bar)
-    @f.must_respond_to(:before_bar)
-  end
-
-  deprecated "should have before_bar and after_bar instance methods" do
-    @f.new.must_respond_to(:before_bar)
-    @f.new.must_respond_to(:before_bar)
-  end
-
-  deprecated "it should return true for bar when before_bar and after_bar hooks are returing true" do
-    a = 1
-    @f.before_bar { a += 1}
-    @f.new.bar.must_equal true
-    a.must_equal 2
-    @f.after_bar { a *= 2}
-    @f.new.bar.must_equal true
-    a.must_equal 6
-  end
-
-  deprecated "it should return nil for bar when before_bar and after_bar hooks are returing false" do
-    @f.new.bar.must_equal true
-    @f.after_bar { false }
-    @f.new.bar.must_equal :a
-    @f.before_bar { false }
-    @f.new.bar.must_equal :b
-  end
-end