Receive use_docker flag from the cli (#121)

* Receive use_docker flag from the cli

* Subparser from entry point proof of concept

* Add plugin subparser

Author: chuchocd85

python/rpdk/python/codegen.py

@@ -49,23 +49,25 @@ def __init__(self):
         self.namespace = None
         self.package_name = None
         self.package_root = None
-        self._use_docker = True
+        self._use_docker = None
         self._protocol_version = "2.0.0"
 
     def _init_from_project(self, project):
         self.namespace = tuple(s.lower() for s in project.type_info)
         self.package_name = "_".join(self.namespace)
-        self._use_docker = project.settings.get("use_docker", True)
+        self._use_docker = project.settings.get("use_docker")
         self.package_root = project.root / "src"
 
     def _init_settings(self, project):
         LOG.debug("Writing settings")
-        self._use_docker = input_with_validation(
+
+        self._use_docker = self._use_docker or input_with_validation(
             "Use docker for platform-independent packaging (Y/n)?\n",
             validate_no,
             "This is highly recommended unless you are experienced \n"
             "with cross-platform Python packaging.",
         )
+
         project.settings["use_docker"] = self._use_docker
         project.settings["protocolVersion"] = self._protocol_version
 

python/rpdk/python/parser.py

@@ -0,0 +1,29 @@
+def setup_subparser(subparsers, parents, python_version):
+    parser = subparsers.add_parser(
+        python_version,
+        description="""This sub command generates IDE and build files for Python {}
+            """.format(
+            "3.6" if python_version == "python36" else "3.7"
+        ),
+        parents=parents,
+    )
+    parser.set_defaults(language=python_version)
+
+    parser.add_argument(
+        "-d",
+        "--use-docker",
+        action="store_true",
+        help="""Use docker for python platform-independent packaging.
+            This is highly recommended unless you are experienced
+            with cross-platform Python packaging.""",
+    )
+
+    return parser
+
+
+def setup_subparser_python36(subparsers, parents):
+    return setup_subparser(subparsers, parents, "python36")
+
+
+def setup_subparser_python37(subparsers, parents):
+    return setup_subparser(subparsers, parents, "python37")

setup.py

@@ -43,7 +43,11 @@ def find_version(*file_paths):
         "rpdk.v1.languages": [
             "python37 = rpdk.python.codegen:Python37LanguagePlugin",
             "python36 = rpdk.python.codegen:Python36LanguagePlugin",
-        ]
+        ],
+        "rpdk.v1.parsers": [
+            "python37 = rpdk.python.parser:setup_subparser_python37",
+            "python36 = rpdk.python.parser:setup_subparser_python36",
+        ],
     },
     license="Apache License 2.0",
     classifiers=[

tests/plugin/parser_test.py

@@ -0,0 +1,27 @@
+import argparse
+
+from rpdk.python.parser import setup_subparser_python36, setup_subparser_python37
+
+
+def test_setup_subparser_python36():
+    parser = argparse.ArgumentParser()
+    subparsers = parser.add_subparsers(dest="subparser_name")
+
+    sub_parser = setup_subparser_python36(subparsers, [])
+
+    args = sub_parser.parse_args(["-d"])
+
+    assert args.language == "python36"
+    assert args.use_docker is True
+
+
+def test_setup_subparser_python37():
+    parser = argparse.ArgumentParser()
+    subparsers = parser.add_subparsers(dest="subparser_name")
+
+    sub_parser = setup_subparser_python37(subparsers, [])
+
+    args = sub_parser.parse_args([])
+
+    assert args.language == "python37"
+    assert args.use_docker is False