2 files changed, 13 insertions, 12 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb
index 82788869e..cecb904ab 100644
--- a/Library/Homebrew/formula.rb
+++ b/Library/Homebrew/formula.rb
@@ -551,8 +551,8 @@ private
     instance_variable_set("@#{type}", class_value) if class_value
   end
 
-  def method_added method
-    raise 'You cannot override Formula.brew' if method == 'brew'
+  def self.method_added method
+    raise 'You cannot override Formula.brew' if method == :brew
   end
 
   class << self
diff --git a/Library/Homebrew/test/test_formula.rb b/Library/Homebrew/test/test_formula.rb
index 27761e289..ef3e0952b 100644
--- a/Library/Homebrew/test/test_formula.rb
+++ b/Library/Homebrew/test/test_formula.rb
@@ -19,15 +19,6 @@ class MostlyAbstractFormula <Formula
   @homepage = 'http://example.com/'
 end
 
-class TestBallOverrideBrew <Formula
-  def initialize
-    super "foo"
-  end
-  def brew
-  end
-end
-
-
 class FormulaTests < Test::Unit::TestCase
 
   def test_prefix
@@ -48,7 +39,17 @@ class FormulaTests < Test::Unit::TestCase
   end
 
   def test_cant_override_brew
-    assert_raises(RuntimeError) { TestBallOverrideBrew.new }
+    assert_raises(RuntimeError) do
+      eval <<-EOS
+      class TestBallOverrideBrew <Formula
+        def initialize
+          super "foo"
+        end
+        def brew
+        end
+      end
+      EOS
+    end
   end
   
   def test_abstract_formula