Merge pull request #80 from FundingCircle/fix_issue_in_json_serializa…

…tion

Update JSONSerializer when encoding to return value if already a JSON…

lib/vault/rails/serializers/json_serializer.rb

@@ -11,9 +11,11 @@ module JSONSerializer
 
         def self.encode(raw)
           return if raw.nil?
+          return raw if raw.is_a?(String)
           JSON.fast_generate(raw)
         end
 
+
         def self.decode(raw)
           return if raw.nil?
           JSON.parse(raw, DECODE_OPTIONS)

spec/unit/rails/serializers/json_serializer_spec.rb

@@ -1,11 +1,19 @@
 require 'spec_helper'
 
 describe Vault::Rails::Serializers::JSONSerializer do
-  it 'encodes values to strings' do
-    expect(subject.encode({"foo" => "bar", "baz" => 1})).to eq '{"foo":"bar","baz":1}'
+  context '.encode' do
+    it 'encodes values to strings' do
+      expect(subject.encode({"foo" => "bar", "baz" => 1})).to eq '{"foo":"bar","baz":1}'
+    end
+
+    it 'returns values already encoded as a JSON string' do
+      expect(subject.encode('{"anonymised":true}')).to eq('{"anonymised":true}')
+    end
   end
 
-  it 'decodes values from strings' do
-    expect(subject.decode('{"foo":"bar","baz":1}')).to eq({"foo" => "bar", "baz" => 1})
+  context '.decode' do
+    it 'decodes values from strings' do
+      expect(subject.decode('{"foo":"bar","baz":1}')).to eq({"foo" => "bar", "baz" => 1})
+    end
   end
 end