Add check_mode support to HipChat module

The HipChat module declares to support check_mode,
but the message is sent in any case.

With this, if executed in check mode, the module will exit
before actually sending the message to HipChat.

It will return changed=False, as per the convention
for notifications modules.
reviewable/pr18780/r1
Jean-Fred Berthelot 10 years ago committed by Jean-Fred
parent 19b345932a
commit f0508fa30e

@ -96,6 +96,11 @@ def send_msg(module, token, room, msg_from, msg, msg_format='text',
url = api + "?auth_token=%s" % (token) url = api + "?auth_token=%s" % (token)
data = urllib.urlencode(params) data = urllib.urlencode(params)
if module.check_mode:
# In check mode, exit before actually sending the message
module.exit_json(changed=False)
response, info = fetch_url(module, url, data=data) response, info = fetch_url(module, url, data=data)
if info['status'] == 200: if info['status'] == 200:
return response.read() return response.read()
@ -119,8 +124,8 @@ def main():
"purple", "gray", "random"]), "purple", "gray", "random"]),
msg_format=dict(default="text", choices=["text", "html"]), msg_format=dict(default="text", choices=["text", "html"]),
notify=dict(default=True, type='bool'), notify=dict(default=True, type='bool'),
validate_certs = dict(default='yes', type='bool'), validate_certs=dict(default='yes', type='bool'),
api = dict(default=MSG_URI), api=dict(default=MSG_URI),
), ),
supports_check_mode=True supports_check_mode=True
) )

Loading…
Cancel
Save