Skip to content

Commit

Permalink
Push a failing test for issues [#4994] and [#5003].
Browse files Browse the repository at this point in the history
  • Loading branch information
josevalim committed Jun 29, 2010
1 parent 7ea85ff commit 67582f0
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 1 deletion.
6 changes: 6 additions & 0 deletions activerecord/test/cases/named_scope_test.rb
Expand Up @@ -450,6 +450,12 @@ def test_named_scopes_are_reset_on_association_reload
assert before.object_id != post.comments.containing_the_letter_e.object_id, "AssociationCollection##{method} should reset the named scopes cache"
end
end

def test_named_scoped_are_lazy_loaded_if_table_still_does_not_exist
assert_nothing_raised do
require "models/without_table"
end
end
end

class DynamicScopeMatchTest < ActiveRecord::TestCase
Expand Down
3 changes: 3 additions & 0 deletions activerecord/test/models/without_table.rb
@@ -0,0 +1,3 @@
class WithoutTable < ActiveRecord::Base
default_scope where(:published => true)
end
15 changes: 14 additions & 1 deletion railties/test/application/loading_test.rb
Expand Up @@ -12,7 +12,7 @@ def app
@app ||= Rails.application
end

def test_load_should_load_constants
def test_constants_in_app_are_autoloaded
app_file "app/models/post.rb", <<-MODEL
class Post < ActiveRecord::Base
validates_acceptance_of :title, :accept => "omg"
Expand All @@ -29,6 +29,19 @@ class Post < ActiveRecord::Base
assert_equal 'omg', p.title
end

def test_models_without_table_do_not_panic_on_scope_definitions_when_loaded
app_file "app/models/user.rb", <<-MODEL
class User < ActiveRecord::Base
default_scope where(:published => true)
end
MODEL

require "#{rails_root}/config/environment"
setup_ar!

User
end

def test_descendants_are_cleaned_on_each_request_without_cache_classes
add_to_config <<-RUBY
config.cache_classes = false
Expand Down

0 comments on commit 67582f0

Please sign in to comment.