nebulous_stomp 3.0.1 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 53e2585b6b1424042306d69a871a38632257e79a
-  data.tar.gz: 0f4c7affb0acf4b97838866a0a16420daf234814
+  metadata.gz: dd218829ddf19457d5aab68ef9a6539a2e68f2a8
+  data.tar.gz: ef55678e7921f2ed9d750b6a6aac5b2b9417e0cb
 SHA512:
-  metadata.gz: 0ebec48ed70a194f814af1fc38382a9fd6b72b30b66212675ed75b6d8c70a321fd1c462660d60ca4ece99691778859fa4fcd24d6cc110e9776ea101ada50ae7f
-  data.tar.gz: 1bf072025eb60e1e7da496ca5d399fd77f989140ba0d567c0e30bdb35ee861e6fd2096faa53586a92a7c4f22a784012b279253e0461b38b2e4b72ddace52ee44
+  metadata.gz: dff8f6526477cf458da01e58ffcac4b59f6ebfeddc7374887ae230f1c2ff7ea7cd25c3278889a6bfd813959450554c1fa00323cb610e2ee2239b6cb9a01a1798
+  data.tar.gz: 46520f833f856a807a135ff0035211ee1e12edf0ae2ae057c28bcac033525ba321cdb2174f0e7b949b4d5a904ad0f9e9b29b0b08e7378dcf6021f739fb4f387d

data/feature/feature_test_spec.rb

@@ -96,12 +96,19 @@ describe 'stomp use cases:' do
     end
 
     it "can respond to a message with a non-Protocol message" do
-      message  = NebulousStomp::Message.new(verb: 'gimmemessage', contentType: 'text')
+      message  = NebulousStomp::Message.new(verb: 'gimmemessage', contentType: 'application/json')
       response = NebulousStomp::Request.new("featuretest", message).send_no_cache
 
       expect( response      ).to be_kind_of(NebulousStomp::Message)
       expect( response.verb ).to be_nil
-      expect( response.body ).to eq "weird message body"
+      expect( response.body ).to eq(["weird message body", 12])
+    end
+
+    it "can respond with an empty array as the message body" do
+      message  = NebulousStomp::Message.new(verb: 'gimmeempty', contentType: 'application/json')
+      response = NebulousStomp::Request.new("featuretest", message).send_no_cache
+      expect( response      ).to be_kind_of(NebulousStomp::Message)
+      expect( response.body ).to eq([])
     end
 
     it "can send a message >32k" do

data/feature/gimme.rb

@@ -48,8 +48,11 @@ class Gimme
         when "gimmeprotocol" 
           msg.respond_with_protocol("foo", "bar", "baz")
 
+        when "gimmeempty"
+          msg.respond([])
+
         when "gimmemessage" 
-          msg.respond("weird message body")
+          msg.respond(["weird message body", 12])
 
         when "gimmebigmessage"
           body = big_body msg.params

data/lib/nebulous_stomp/message.rb

@@ -135,6 +135,9 @@ module NebulousStomp
     #     * :stompHeaders         -- for a message from Stomp, the raw Stomp Headers string
     #     * :verb                 -- part of The Protocol
     #
+    # Note that body is taken to be a string if the content type is text, and is taken to be JSON
+    # if the content type is JSON. 
+    #
     def initialize(hash)
       @header = Msg::Header.new(hash)
       @body = Msg::Body.new(content_is_json?, hash)
@@ -185,7 +188,8 @@ module NebulousStomp
     # :call-seq: 
     # message.respond_with_protocol(body) -> queue, Message
     #
-    # Repond with a message body (presumably a custom one that's non-Protocol).
+    # Repond with a message body (presumably a custom one that's non-Protocol). If the content type
+    # is JSON, this can be an array or a Hash. If the content type is text, it must be a String.
     # 
     def respond(body)
       fail NebulousError, "Don't know which queue to reply to" unless reply_to

data/lib/nebulous_stomp/msg/body.rb

@@ -111,7 +111,8 @@ module NebulousStomp
         if @verb
           @body = protocol_hash
         elsif @body.nil? || @body.respond_to?(:empty?) && @body.empty?
-          @body = parse_stomp_body 
+          sb = parse_stomp_body 
+          @body = sb if sb
         end
 
         parse_body

data/lib/nebulous_stomp/version.rb

@@ -1,5 +1,5 @@
 module NebulousStomp
 
   # Nebulous version number
-  VERSION = "3.0.1"
+  VERSION = "3.0.2"
 end

data/spec/message_spec.rb

@@ -120,6 +120,12 @@ describe Message do
       expect( m.body   ).to eq "wigi wigi"
     end
 
+    it "leaves body alone if it\'s an empty array and there is no protocol and no stompbody" do
+      # rather than processing a nil stompbody and ending up with nil/NULL
+      m = Message.new(body: [])
+      expect( m.body ).to eq([])
+    end
+
     it 'takes the content type from the input arguments' do
       msg = Message.new( new_hash.merge(contentType: 'foo') )
       expect( msg.content_type ).to eq "foo"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nebulous_stomp
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.0.2
 platform: ruby
 authors:
 - Andy Jones
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-28 00:00:00.000000000 Z
+date: 2017-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler