From f1584be3ebd239000b9376f25489d077bebccb84 Mon Sep 17 00:00:00 2001 From: normal Date: Sun, 13 Dec 2015 03:17:28 +0000 Subject: lib/shellwords.rb: do not change API with frozen-string-literal This fixes a bug introduced in r53066 when attempting to shellescape an empty string. * lib/shellwords.rb (shellescape): duplicate frozen literal * test/test_shellwords.rb (test_stringification): new test git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@53068 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- test/test_shellwords.rb | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'test') diff --git a/test/test_shellwords.rb b/test/test_shellwords.rb index 59cdbe96dc..e70e0d72bf 100644 --- a/test/test_shellwords.rb +++ b/test/test_shellwords.rb @@ -48,8 +48,17 @@ class TestShellwords < Test::Unit::TestCase end def test_stringification - assert_equal "3", shellescape(3) - assert_equal "ps -p #{$$}", ['ps', '-p', $$].shelljoin + three = shellescape(3) + assert_equal '3', three + assert_not_predicate three, :frozen? + + empty = shellescape('') + assert_equal "''", empty + assert_not_predicate empty, :frozen? + + joined = ['ps', '-p', $$].shelljoin + assert_equal "ps -p #{$$}", joined + assert_not_predicate joined, :frozen? end def test_multibyte_characters -- cgit v1.2.3