From 0e414175fdcc19550fe44cc1ad8cb4827d2e3da8 Mon Sep 17 00:00:00 2001
From: nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Thu, 5 Feb 2015 02:10:44 +0000
Subject: [PATCH] test_inadvertent_creation.rb: move tests

* test/-ext-/symbol/test_inadvertent_creation.rb: move symbol leak
  tests, for implementation details, from test/ruby/test_symbol.rb.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@49504 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 .../-ext-/symbol/test_inadvertent_creation.rb | 62 ++++++++++++++++++
 test/ruby/test_symbol.rb                      | 65 -------------------
 2 files changed, 62 insertions(+), 65 deletions(-)

diff --git a/test/-ext-/symbol/test_inadvertent_creation.rb b/test/-ext-/symbol/test_inadvertent_creation.rb
index b180d08b04..fdd1e252ce 100644
--- a/test/-ext-/symbol/test_inadvertent_creation.rb
+++ b/test/-ext-/symbol/test_inadvertent_creation.rb
@@ -358,5 +358,67 @@ module Test_Symbol
       }
       assert_not_pinneddown(name)
     end
+
+    def assert_no_immortal_symbol_created(name)
+      name = noninterned_name(name)
+      yield(name)
+      assert_not_pinneddown(name)
+    end
+
+    def test_send_leak_string
+      assert_no_immortal_symbol_created("send should not leak - str") do |name|
+        assert_raise(NoMethodError) {42.send(name)}
+      end
+    end
+
+    def test_send_leak_symbol
+      assert_no_immortal_symbol_created("send should not leak - sym") do |name|
+        assert_raise(NoMethodError) {42.send(name.to_sym)}
+      end
+    end
+
+    def test_send_leak_string_custom_method_missing
+      x = Object.new
+      def x.method_missing(*); end
+      assert_no_immortal_symbol_created("send should not leak - str mm") do |name|
+        assert_nothing_raised(NoMethodError) {x.send(name)}
+      end
+    end if false
+
+    def test_send_leak_symbol_custom_method_missing
+      x = Object.new
+      def x.method_missing(*); end
+      assert_no_immortal_symbol_created("send should not leak - sym mm") do |name|
+        assert_nothing_raised(NoMethodError) {x.send(name.to_sym)}
+      end
+    end if false
+
+    def test_send_leak_string_no_optimization
+      assert_no_immortal_symbol_created("send should not leak - str slow") do |name|
+        assert_raise(NoMethodError) {42.method(:send).call(name)}
+      end
+    end
+
+    def test_send_leak_symbol_no_optimization
+      assert_no_immortal_symbol_created("send should not leak - sym slow") do |name|
+        assert_raise(NoMethodError) {42.method(:send).call(name.to_sym)}
+      end
+    end
+
+    def test_send_leak_string_custom_method_missing_no_optimization
+      x = Object.new
+      def x.method_missing(*); end
+      assert_no_immortal_symbol_created("send should not leak - str mm slow") do |name|
+        assert_nothing_raised(NoMethodError) {x.method(:send).call(name)}
+      end
+    end
+
+    def test_send_leak_symbol_custom_method_missing_no_optimization
+      x = Object.new
+      def x.method_missing(*); end
+      assert_no_immortal_symbol_created("send should not leak - sym mm slow") do |name|
+        assert_nothing_raised(NoMethodError) {x.method(:send).call(name.to_sym)}
+      end
+    end
   end
 end
diff --git a/test/ruby/test_symbol.rb b/test/ruby/test_symbol.rb
index 6606b51112..596f2b09f6 100644
--- a/test/ruby/test_symbol.rb
+++ b/test/ruby/test_symbol.rb
@@ -238,69 +238,4 @@ class TestSymbol < Test::Unit::TestCase
       200_000.times { |i| i.to_s.to_sym }
     end;
   end
-
-  def assert_no_immortal_symbol_created
-    delta = -Symbol.all_symbols.size
-    yield
-    GC.start
-    delta += Symbol.all_symbols.size
-    assert_equal 0, delta, "#{delta} immortal symbols were created"
-  end
-
-  def test_symbol_send_leak_string
-    assert_no_immortal_symbol_created do
-      10.times { 42.send "send should not leak #{i} - str" rescue nil }
-    end
-  end
-
-  def test_symbol_send_leak_symbol
-    assert_no_immortal_symbol_created do
-      10.times { 42.send "send should not leak #{i} - sym".to_sym rescue nil }
-    end
-  end
-
-  def test_symbol_send_leak_string_custom_method_missing
-    x = Object.new
-    def x.method_missing(*); end
-    assert_no_immortal_symbol_created do
-      10.times { |i| x.send "send should not leak #{i} - str mm" }
-    end
-  end if false
-
-  def test_symbol_send_leak_symbol_custom_method_missing
-    x = Object.new
-    def x.method_missing(*); end
-    assert_no_immortal_symbol_created do
-      10.times { |i| x.send "send should not leak #{i} - sym mm".to_sym }
-    end
-  end if false
-
-  def test_symbol_send_leak_string_no_optimization
-    assert_no_immortal_symbol_created do
-      10.times { 42.method(:send).call "send should not leak #{i} - str slow" rescue nil }
-    end
-  end
-
-  def test_symbol_send_leak_symbol_no_optimization
-    assert_no_immortal_symbol_created do
-      10.times { 42.method(:send).call "send should not leak #{i} - sym slow".to_sym rescue nil }
-    end
-  end
-
-  def test_symbol_send_leak_string_custom_method_missing_no_optimization
-    x = Object.new
-    def x.method_missing(*); end
-    assert_no_immortal_symbol_created do
-      10.times { |i| x.method(:send).call "send should not leak #{i} - str mm slow" }
-    end
-  end
-
-  def test_symbol_send_leak_symbol_custom_method_missing_no_optimization
-    x = Object.new
-    def x.method_missing(*); end
-    assert_no_immortal_symbol_created do
-      10.times { |i| x.method(:send).call "send should not leak #{i} - sym mm slow".to_sym }
-    end
-  end
-
 end