From de8025e0f90005295ea96cabe039534fc2393699 Mon Sep 17 00:00:00 2001 From: Ian Roddis Date: Fri, 20 Aug 2021 10:58:10 -0300 Subject: [PATCH] - Fixing bug that yielded empty expanded command sets. --- daggy/src/Utilities.cpp | 3 ++- tests/unit_utilities.cpp | 5 +---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/daggy/src/Utilities.cpp b/daggy/src/Utilities.cpp index 8bd00fe..0deff84 100644 --- a/daggy/src/Utilities.cpp +++ b/daggy/src/Utilities.cpp @@ -17,8 +17,9 @@ namespace daggy { std::vector> commands{{}}; for (const auto &part : command) { - std::vector expandedPart; + std::vector expandedPart{part}; + // Find all values of parameters, and expand them for (const auto &[param, paramValue] : parameters) { auto pos = part.find(param); if (pos == std::string::npos) continue; diff --git a/tests/unit_utilities.cpp b/tests/unit_utilities.cpp index e1fc3eb..8456681 100644 --- a/tests/unit_utilities.cpp +++ b/tests/unit_utilities.cpp @@ -49,10 +49,7 @@ TEST_CASE("Parameter Expansion", "[utilities_parameter_expansion]") { // TYPE isn't used, so it's just |DATE| * |SOURCE| REQUIRE(result.size() == 4); - for (const auto &command : result) { - std::copy(command.begin(), command.end(), std::ostream_iterator(std::cout, " ")); - std::cout << std::endl; - } + } }