Merge pull request #168 from Logicworks/feature/allow-notify-on-user-…

…send add notify as an option to direct messages
hipchat · Feb 23, 2016 · 6287393 · 6287393
2 parents b12358e + c2909f9
commit 6287393
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/hipchat/user.rb b/lib/hipchat/user.rb
@@ -19,12 +19,13 @@ def initialize(token, params)
     #
     # Send a private message to user.
     #
-    def send(message, message_format='text')
+    def send(message, message_format='text', notify=false)
       response = self.class.post(@api.send_config[:url],
                                  :query => { :auth_token => @token },
                                  :body => {
                                      :message => message,
-                                     :message_format => message_format
+                                     :message_format => message_format,
+                                     :notify => notify
                                  }.send(@api.send_config[:body_format]),
                                  :headers => @api.headers
       )

diff --git a/spec/support/shared_contexts_for_hipchat.rb b/spec/support/shared_contexts_for_hipchat.rb
@@ -304,7 +304,8 @@ def mock_successful_user_send(message)
     stub_request(:post, "https://api.hipchat.com/v2/user/12345678/message").with(
                                    :query   => {:auth_token => "blah"},
                                    :body    => {:message => "Equal bytes for everyone",
-                                                :message_format => "text"},
+                                                :message_format => "text",
+                                                :notify => false},
                                    :headers => {'Accept' => 'application/json',
                                                 'Content-Type' => 'application/json'}).to_return(:status => 200, :body => "", :headers => {})
   end