Do not implicitly mark recordss of has_many :through as readonly but do mark habtm records as readonly (eventually only on join tables without rich attributes).

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4107 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

1 files changed, 12 insertions, 6 deletions

diff --git a/activerecord/test/readonly_test.rb b/activerecord/test/readonly_test.rb
index 55f5cbe4ca..584eb735fe 100755
--- a/activerecord/test/readonly_test.rb
+++ b/activerecord/test/readonly_test.rb
@@ -3,6 +3,8 @@ require 'fixtures/post'
 require 'fixtures/comment'
 require 'fixtures/developer'
 require 'fixtures/project'
+require 'fixtures/reader'
+require 'fixtures/person'
 
 # Dummy class methods to test implicit association scoping.
 def Comment.foo() find :first end
@@ -50,19 +52,23 @@ class ReadOnlyTest < Test::Unit::TestCase
   def test_habtm_find_readonly
     dev = Developer.find(1)
     assert !dev.projects.empty?
-    dev.projects.each { |p| assert !p.readonly? }
-    dev.projects.find(:all) { |p| assert !p.readonly? }
-    dev.projects.find(:all, :readonly => true) { |p| assert p.readonly? }
+    assert dev.projects.all?(&:readonly?)
+    assert dev.projects.find(:all).all?(&:readonly?)
+    assert dev.projects.find(:all, :readonly => true).all?(&:readonly?)
   end
 
   def test_has_many_find_readonly
     post = Post.find(1)
     assert !post.comments.empty?
-    post.comments.each { |r| assert !r.readonly? }
-    post.comments.find(:all) { |r| assert !r.readonly? }
-    post.comments.find(:all, :readonly => true) { |r| assert r.readonly? }
+    assert !post.comments.any?(&:readonly?)
+    assert !post.comments.find(:all).any?(&:readonly?)
+    assert post.comments.find(:all, :readonly => true).all?(&:readonly?)
   end
 
+  def test_has_many_with_through_is_not_implicitly_marked_readonly
+    assert people = Post.find(1).people
+    assert !people.any?(&:readonly?)
+  end
 
   def test_readonly_scoping
     Post.with_scope(:find => { :conditions => '1=1' }) do