From 667beb662f68471bd9b10ad345ff480c79b7f869 Mon Sep 17 00:00:00 2001 From: Holger Just Date: Fri, 21 Jul 2017 22:48:39 +0200 Subject: [PATCH] Freeze Messages directly after initialization This ensures that they are actually immutable. --- lib/rackstash/message.rb | 2 ++ spec/rackstash/message_spec.rb | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/lib/rackstash/message.rb b/lib/rackstash/message.rb index 8aa3503..b1e3864 100644 --- a/lib/rackstash/message.rb +++ b/lib/rackstash/message.rb @@ -53,6 +53,8 @@ module Rackstash @progname = dup_freeze(progname) @message = cleanup(message) + + freeze end # @return [String] the guman readable label for the {#severity}. diff --git a/spec/rackstash/message_spec.rb b/spec/rackstash/message_spec.rb index fc40278..4675c19 100644 --- a/spec/rackstash/message_spec.rb +++ b/spec/rackstash/message_spec.rb @@ -50,6 +50,10 @@ describe Rackstash::Message do expect(message.message).not_to equal str expect(message.message).to eql str end + + it 'freezes the Message' do + expect(described_class.new('message')).to be_frozen + end end describe '#message' do