Merge pull request #242 from charlotte-ruby/performance-deletion
Performance deletion
This commit is contained in:
commit
72b379908f
|
@ -15,9 +15,9 @@ module Impressionist
|
|||
def define_association
|
||||
has_many(:impressions,
|
||||
:as => :impressionable,
|
||||
:dependent => :destroy)
|
||||
:dependent => :delete_all)
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
end
|
||||
end
|
||||
|
|
|
@ -9,7 +9,7 @@ module Impressionist
|
|||
def is_impressionable(options={})
|
||||
many(:impressions,
|
||||
:as => :impressionable,
|
||||
:dependent => :destroy)
|
||||
:dependent => :delete_all)
|
||||
|
||||
@impressionist_cache_options = options
|
||||
end
|
||||
|
|
|
@ -57,14 +57,14 @@ describe Impression do
|
|||
@article.impressionist_count(:filter=>:session_hash).should eq 7
|
||||
end
|
||||
|
||||
# tests :dependent => :destroy
|
||||
# tests :dependent => :delete_all
|
||||
it "should delete impressions on deletion of impressionable" do
|
||||
#impressions_count = Impression.all.size
|
||||
a = Article.create
|
||||
i = a.impressions.create
|
||||
a.destroy
|
||||
a.destroyed?.should be_true
|
||||
i.destroyed?.should be_true
|
||||
Impression.exists?(i.id).should be_false
|
||||
end
|
||||
|
||||
end
|
||||
|
|
Loading…
Reference in New Issue