Merge pull request #64 from jrbeck/do-not-mutate-strings

Prevent String mutation

Author: rmm5t

lib/strip_attributes.rb

@@ -56,7 +56,7 @@ def self.strip_record(record, options = {})
 
   def self.strip_string(value, options = {})
     return value unless value.is_a?(String)
-    return value if value.frozen?
+    value = value.dup
 
     allow_empty      = options[:allow_empty]
     collapse_spaces  = options[:collapse_spaces]

test/strip_attributes_test.rb

@@ -204,10 +204,12 @@ def test_should_strip_and_allow_empty
     assert_equal "",            record.bang
   end
 
-  def test_should_skip_frozen_values
-    record = StripAllMockRecord.new(frozen: " ice ".freeze)
+  def test_should_not_mutate_values
+    record = StripAllMockRecord.new(foo: " foo ")
+    old_value = record.foo
     record.valid?
-    assert_equal " ice ", record.frozen
+    assert_equal "foo",        record.foo
+    refute_equal old_value,    record.foo
   end
 
   def test_should_collapse_duplicate_spaces