remove unnecessary test code

aratak · Apr 27, 2012 · a57b784 · a57b784
1 parent 4ed6167
commit a57b784
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/activerecord/test/cases/named_scope_test.rb b/activerecord/test/cases/named_scope_test.rb
@@ -70,10 +70,6 @@ def test_scopes_with_string_name_can_be_composed
     assert_equal Topic.replied.approved, Topic.replied.approved_as_string
   end
 
-  def test_scopes_can_be_specified_with_deep_hash_conditions
-    assert_equal Topic.replied.approved, Topic.replied.approved_as_hash_condition
-  end
-
   def test_scopes_are_composable
     assert_equal((approved = Topic.find(:all, :conditions => {:approved => true})), Topic.approved)
     assert_equal((replied = Topic.find(:all, :conditions => 'replies_count > 0')), Topic.replied)

diff --git a/activerecord/test/models/topic.rb b/activerecord/test/models/topic.rb
@@ -11,11 +11,7 @@ class Topic < ActiveRecord::Base
   scope :scope_with_lambda, lambda { scoped }
 
   scope :by_lifo, -> { where(:author_name => 'lifo') }
-
-  ActiveSupport::Deprecation.silence do
-    scope :approved_as_hash_condition, :conditions => {:topics => {:approved => true}}
-    scope :replied, :conditions => ['replies_count > 0']
-  end
+  scope :replied, -> { where 'replies_count > 0' }
 
   scope 'approved_as_string', -> { where(:approved => true) }
   scope :anonymous_extension, -> { scoped } do