xenofem/python-zulip-api
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

zulip_bot_output.py: Rename options to args.

Browse source
This commit is contained in:
derAnfaenger 2017-09-01 12:02:31 +02:00 committed by Tim Abbott
parent c5ab72a50a
commit 2aa6201d47
1 changed files with 17 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
zulip_bots/zulip_bots/zulip_bot_output.py
View file

@ -51,9 +51,9 @@ def parse_args():
action='store_true', action='store_true',
help='Install dependencies for the bot.') help='Install dependencies for the bot.')
options = parser.parse_args() args = parser.parse_args()
if not options.name and not options.path_to_bot: if not args.name and not args.path_to_bot:
error_message = """ error_message = """
You must either specify the name of an existing bot or You must either specify the name of an existing bot or
specify a path to the file (--path-to-bot) that contains specify a path to the file (--path-to-bot) that contains
@ -64,33 +64,33 @@ the bot handler class.
# checks if both of these are in sync, otherwise we'll # checks if both of these are in sync, otherwise we'll
# have to be bias towards one and the user may get incorrect # have to be bias towards one and the user may get incorrect
# result. # result.
elif not name_and_path_match(options.name, options.path_to_bot): elif not name_and_path_match(args.name, args.path_to_bot):
error_message = """ error_message = """
Please make sure that the given name of the bot and the Please make sure that the given name of the bot and the
given path to the bot are same and valid. given path to the bot are same and valid.
""" """
parser.error(error_message) parser.error(error_message)
return options return args
def main(): def main():
# type: () -> None # type: () -> None
options = parse_args() args = parse_args()
bot_name = options.name bot_name = args.name
if options.path_to_bot: if args.path_to_bot:
if options.provision: if args.provision:
bot_dir = os.path.dirname(os.path.abspath(options.path_to_bot)) bot_dir = os.path.dirname(os.path.abspath(args.path_to_bot))
provision_bot(bot_dir, options.force) provision_bot(bot_dir, args.force)
lib_module = import_module_from_source(options.path_to_bot, name=bot_name) lib_module = import_module_from_source(args.path_to_bot, name=bot_name)
elif options.name: elif args.name:
if options.provision: if args.provision:
current_dir = os.path.dirname(os.path.abspath(__file__)) current_dir = os.path.dirname(os.path.abspath(__file__))
bots_parent_dir = os.path.join(current_dir, "bots") bots_parent_dir = os.path.join(current_dir, "bots")
bot_dir = os.path.join(bots_parent_dir, options.name) bot_dir = os.path.join(bots_parent_dir, args.name)
provision_bot(bot_dir, options.force) provision_bot(bot_dir, args.force)
lib_module = import_module('zulip_bots.bots.{bot}.{bot}'.format(bot=bot_name)) lib_module = import_module('zulip_bots.bots.{bot}.{bot}'.format(bot=bot_name))
message = {'content': options.message, 'sender_email': 'foo_sender@zulip.com'} message = {'content': args.message, 'sender_email': 'foo_sender@zulip.com'}
message_handler = lib_module.handler_class() message_handler = lib_module.handler_class()
with patch('zulip_bots.lib.ExternalBotHandler') as mock_bot_handler: with patch('zulip_bots.lib.ExternalBotHandler') as mock_bot_handler:
@ -120,7 +120,7 @@ def main():
bot_handler=mock_bot_handler, bot_handler=mock_bot_handler,
state_handler=StateHandler() state_handler=StateHandler()
) )
print("On sending ", options.name, " bot the following message:\n\"", options.message, "\"") print("On sending ", args.name, " bot the following message:\n\"", args.message, "\"")
# send_reply and send_message have slightly arguments; the # send_reply and send_message have slightly arguments; the
# following takes that into account. # following takes that into account.