Parse create command options

The title and description passed to the create command have to be
forwarded to the application service.

This is achieved using docopt again. Error handling is currently not
done at that level as there are no tests requiring it.

Author: arnemertz

behave/issue_list_create.feature

@@ -6,7 +6,6 @@ Feature: Creating and listing issues
     Then it prints "total: 0 issues"
     And terminates with exit code OK
 
-  @wip
   Scenario: create an issue
     Given an empty issue repository
     When we create an issue titled "My first issue in Fix" with description

behave/steps/fix_cli.py

@@ -12,7 +12,7 @@ def _start_fix_with_args(context, args):
 
 @when(u'we call Fix with argument list "{args}"')
 def start_fix(context, args):
-    _start_fix_with_args(context, shlex.split(args.strip(), posix=False))
+    _start_fix_with_args(context, shlex.split(args.strip(), posix=True))
 
 
 @when(u'we call Fix without arguments')

src/fix_cli/app.cpp

@@ -21,7 +21,14 @@ Available commands:
    list          List all existing issues
    show          Show a specific issue
 )"sv;
-}
+
+constexpr auto CREATE_USAGE = R"(usage: fix create -t <title> -d <descr>
+
+Options:
+  -t <title> --title=<title> Title of the new issue
+  -d <descr> --descr=<descr> Description text
+)";
+} // namespace
 
 app::app(std::ostream& out) : out{out} {}
 
@@ -31,7 +38,7 @@ auto app::run(const std::vector<std::string_view>& args) -> int {
   try {
     auto const& parsed_args = docopt::docopt_parse(std::string(USAGE), argv, true, false, true);
     auto const& command = parsed_args.at("<command>").asString();
-    return run_command(command);
+    return run_command(command, argv);
 
   } catch (docopt::DocoptExitHelp const&) {
     out << USAGE;
@@ -43,15 +50,19 @@ auto app::run(const std::vector<std::string_view>& args) -> int {
   }
 }
 
-int app::run_command(std::string const& command) {
+int app::run_command(std::string const& command, const std::vector<std::string>& argv) {
   if (command == "list"sv) {
     out << "total: 0 issues\n";
     return EXIT_SUCCESS;
   }
 
   if (command == "create"sv) {
+    auto const& parsed_args = docopt::docopt_parse(std::string(CREATE_USAGE), argv, false, false, false);
+    auto const& title = parsed_args.at("--title").asString();
+    auto const& description = parsed_args.at("--descr").asString();
+
     domain::application_service application_service;
-    const auto issue_id = application_service.create("this is a new issue", "");
+    const auto issue_id = application_service.create(title, description);
     out << fmt::format("Issue created: {}\n", issue_id);
     return EXIT_SUCCESS;
   }

src/fix_cli/app.hpp

@@ -16,7 +16,7 @@ class app {
   auto run(std::vector<std::string_view> const& args) -> int;
 
 private:
-  int run_command(std::string const& command);
+  int run_command(std::string const& command, const std::vector<std::string>& argv);
 };
 
 } // namespace fix::cli

test/fix_cli/app_test.cpp

@@ -86,8 +86,12 @@ TEST_CASE("List command prints number of issues") {
 }
 
 TEST_CASE("Create issue command prints issue ID") {
-  auto const [output, exit_code] = run_app({"create", "-t", "this is a new issue", "-d", "some text"});
+  auto const& [title, description, id_prefix]
+  = GENERATE(std::tuple("this is a new issue", "some text", "thi-is-a-new"),
+             std::tuple("My first issue in Fix", "Dorem Fixum dolor sit amet", "my-fir-iss-in"));
 
-  CHECK_THAT(output, Catch::Matches("Issue created: thi-is-a-new-[0-9a-f]{7}\n"));
+  auto const [output, exit_code] = run_app({"create", "-t", title, "-d", description});
+
+  CHECK_THAT(output, Catch::Matches(fmt::format("Issue created: {}-[0-9a-f]{{7}}\n", id_prefix)));
   CHECK(exit_code == EXIT_SUCCESS);
 }