Added msgserial check back to the protocol_message

lib/ably/models/protocol_message.rb

@@ -249,6 +249,7 @@ def attributes
     # Return a JSON ready object from the underlying #attributes using Ably naming conventions for keys
     def as_json(*args)
       raise TypeError, ':action is missing, cannot generate a valid Hash for ProtocolMessage' unless action
+      raise TypeError, ':msg_serial is missing, cannot generate a valid Hash for ProtocolMessage' if ack_required? && !has_message_serial?
 
       attributes.dup.tap do |hash_object|
         hash_object['action']   = action.to_i

spec/unit/models/protocol_message_spec.rb

@@ -397,6 +397,14 @@ def new_protocol_message(options)
       end
     end
 
+    context 'with missing msg_serial for ack message' do
+      let(:model) { new_protocol_message({ :action => message_action }) }
+
+      it 'it raises an exception' do
+        expect { model.to_json }.to raise_error TypeError, /msg_serial.*missing/
+      end
+    end
+
     context 'is aliased by #to_s' do
       let(:model) { new_protocol_message({ :action => attached_action, :channelSerial => 'unique', messages: [message1, message2, message3], :timestamp => as_since_epoch(Time.now) }) }