Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Commit cd9a27d

Browse files
committed
disable after_commit callbacks
1 parent 9181dce commit cd9a27d

File tree

3 files changed

+6
-0
lines changed

3 files changed

+6
-0
lines changed

lib/replicate/active_record.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -306,6 +306,7 @@ def instance.run_callbacks(*args); yield; end
306306
def instance._run_save_callbacks(*args); yield; end
307307
def instance._run_create_callbacks(*args); yield; end
308308
def instance._run_update_callbacks(*args); yield; end
309+
def instance._run_commit_callbacks(*args); yield; end
309310
else
310311
# AR 2.x
311312
def instance.callback(*args)

replicate.gemspec

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ Gem::Specification.new do |s|
3535
s.test_files = s.files.select {|path| path =~ /^test\/.*_test.rb/}
3636
s.add_development_dependency 'activerecord', '~> 3.1'
3737
s.add_development_dependency 'sqlite3'
38+
s.add_development_dependency 'test_after_commit'
3839

3940
s.require_paths = %w[lib]
4041
end

test/active_record_test.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
dbfile = File.expand_path('../db', __FILE__)
1717
File.unlink dbfile if File.exist?(dbfile)
1818
ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => dbfile)
19+
require 'test_after_commit'
1920

2021
# load schema
2122
ActiveRecord::Migration.verbose = false
@@ -561,6 +562,9 @@ def test_loader_saves_without_callbacks
561562
# note when a record is saved with callbacks
562563
callbacks = false
563564
User.class_eval { after_save { callbacks = true } }
565+
User.class_eval { after_create { callbacks = true } }
566+
User.class_eval { after_update { callbacks = true } }
567+
User.class_eval { after_commit { callbacks = true } }
564568

565569
# check our assumptions
566570
user = User.create(:login => 'defunkt')

0 commit comments

Comments
 (0)