add(fast-paced commit): implementation of fast commit to flag commit

Author: andre-filho

commit_helper/__main__.py

@@ -11,9 +11,13 @@ def main():
     parser = parser_cli()
     args = parser.parse_args()
     debug_mode = args.debug
+
     debug('args variable', args, debug_mode)
+
     file_path = Path('commiter.yml')
+
     debug('file_path', file_path, debug_mode)
+
     if file_path.is_file():
         handle_file_based_commit(file_path, debug_mode, args)
 

commit_helper/conventions/no_convention.py

@@ -2,7 +2,11 @@
 from commit_helper.utils.colors import RESET
 
 
-def just_message():
-    msg = str(input(INPUT_COLOR + "commit message: " + RESET))
-    composed = "%s\n" % msg.capitalize()
+def just_message(msg=''):
+    if msg is '':
+        message = str(input(INPUT_COLOR + "commit message: " + RESET))
+    else:
+        message = msg
+
+    composed = "%s\n" % message.capitalize()
     return composed

commit_helper/utils/flag_commit_handler.py

@@ -5,6 +5,8 @@
 from .text_utils import debug
 from .text_utils import get_text
 from .text_utils import get_context
+from .text_utils import handle_context_arg
+from .text_utils import handle_tag_message_args
 # conventions imports
 from commit_helper.conventions.karma_angular import angular_convention
 from commit_helper.conventions.changelog import changelog_convention
@@ -17,18 +19,25 @@ def convention_flag_handler(args, debug_mode):
     debug('convention flag', convention, debug_mode)
 
     if convention == 'message':
-        commit_message = just_message()
+        if args.message is not '':
+            commit_message = just_message(msg=args.message)
+        else:
+            commit_message = just_message()
+
         create_file('none', args.no_file)
 
     else:
-        tag, msg = get_text()
+        tag, msg = handle_tag_message_args(args.tag, args.message)
+
         if convention == 'angular' or convention == 'karma':
-            context = get_context()
+            context = handle_context_arg(args.context)
             commit_message = angular_convention(tag, msg, context)
             create_file(convention, args.no_file)
+
         elif convention == 'changelog':
             commit_message = changelog_convention(tag, msg)
             create_file(convention, args.no_file)
+
         elif convention == 'symphony':
             commit_message = symphony_convention(tag, msg)
             create_file(convention, args.no_file)

commit_helper/utils/text_utils.py

@@ -30,3 +30,15 @@ def debug(message, value, show=False):
     if show:
         mid = 'DEBUG: ' + str(message) + ' ~> ' + str(value)
         print(DEBUG_COLOR + mid + RESET)
+
+
+def handle_tag_message_args(tag='', message=''):
+    if tag + message is not '':
+        return tag, message
+    return get_text()
+
+
+def handle_context_arg(context=''):
+    if context is not '':
+        return context
+    return get_context()

commit_helper/utils/utils.py

@@ -24,7 +24,7 @@
 def gen_co_author(co_author):
     if co_author is '':
         return ''
-    return "\nCo-authored-by: %s" % co_author
+    return '\nCo-authored-by: %s' % co_author
 
 
 def create_file(convention_name, dont_create=False):    # pragma: no cover
@@ -38,23 +38,36 @@ def create_file(convention_name, dont_create=False):    # pragma: no cover
 
 
 def parser_cli():
-    desc = "A commit formatter tool to help you follow commit conventions."
+    desc = 'A commit formatter tool to help you follow commit conventions.'
     help_convention = \
         """
         Selects a convention to be used for the commit.
         Required if there's no commiter.yml file.
         """
     parser = argparse.ArgumentParser(description=desc)
-    parser.add_argument("-ca", "--co-author",
-                        help="Make your friend an co-author to the commit",
-                        dest="co_author", default='')
-    parser.add_argument("-nf", "--no-file", dest="no_file",
-                        help="Disables the creation of a commiter.yml file",
-                        action="store_true")
-    parser.add_argument("-c", "--convention", choices=supported_conventions,
-                        dest="convention", default='', help=help_convention)
-    parser.add_argument("-d", "--debug", action="store_true", dest="debug",
-                        help="Toggles debug option")
+    parser.add_argument('-t', '--tag', dest='tag', default='',
+                        help='Pass your commit tag directly')
+
+    parser.add_argument('-m', '--message', dest='message', default='',
+                        help='Pass your commit message directly')
+
+    parser.add_argument('-ct', '--context', dest='context', default='',
+                        help='Pass your commit context directly')
+
+    parser.add_argument('-ca', '--co-author',
+                        help='Make your friend an co-author to the commit',
+                        dest='co_author', default='')
+
+    parser.add_argument('-nf', '--no-file', dest='no_file',
+                        help='Disables the creation of a commiter.yml file',
+                        action='store_true')
+
+    parser.add_argument('-c', '--convention', choices=supported_conventions,
+                        dest='convention', default='', help=help_convention)
+
+    parser.add_argument('-d', '--debug', action='store_true', dest='debug',
+                        help='Toggles debug option')
+
     return parser
 
 

test/test_conventions.py

@@ -44,14 +44,21 @@ def test_custom_convention():
         raise AssertionError()
 
 
-def test_no_convention():
+def test_no_convention_without_args():
     inputs = [
         "message",
     ]
 
     def mock_input(s):
         return inputs.pop(0)
+
     no_convention.input = mock_input
     message = no_convention.just_message()
     if not message == 'Message\n':
         raise AssertionError()
+
+
+def test_no_convention_with_args():
+    message = no_convention.just_message('Message')
+    if not message == 'Message\n':
+        raise AssertionError()

test/test_utils.py

@@ -1,12 +1,10 @@
 import yaml
 import commit_helper.utils.utils as utils
 import commit_helper.utils.text_utils as text_utils
+
 from commit_helper.utils.colors import RESET
 from commit_helper.utils.colors import DEBUG_COLOR
 from commit_helper.utils.colors import NOTIFY_COLOR
-# from commit_helper.utils.utils import validate_commiter_file
-# import commit_helper.utils.file_handler as file_utils
-# import commit_helper.utils.flag_commit_handler as flag_utils
 
 
 def test_get_text():
@@ -91,3 +89,43 @@ def test_notify(capsys):
     captured = capsys.readouterr()
     if not captured.out == NOTIFY_COLOR + "msg" + RESET + "\n":
         raise AssertionError()
+
+
+def test_handle_tag_message_args_with_args():
+    tag, msg = text_utils.handle_tag_message_args('tag', 'msg')
+
+    if not (tag is 'tag' and msg is 'msg'):
+        raise AssertionError()
+
+
+def test_handle_tag_message_args_without_args():
+    inputs = ['tag', 'msg']
+
+    def mock_input(s):
+        return inputs.pop(0)
+
+    text_utils.input = mock_input
+    tag, msg = text_utils.handle_tag_message_args()
+
+    if not (tag == 'tag' and msg == 'msg'):
+        raise AssertionError()
+
+
+def test_handle_context_arg_with_args():
+    context = text_utils.handle_context_arg('context')
+
+    if not context is 'context':
+        raise AssertionError()
+
+
+def test_handle_context_arg_without_args():
+    inputs = ['context']
+
+    def mock_input(s):
+        return inputs.pop(0)
+
+    text_utils.input = mock_input
+    context = text_utils.handle_context_arg()
+
+    if not context == 'context':
+        raise AssertionError()