Strutils join const

Fabio3rs · Aug 29, 2024 · 389df32 · 389df32
1 parent 1bb5f89
commit 389df32
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/utils/Strutils.hpp b/src/utils/Strutils.hpp
@@ -64,7 +64,7 @@ class Strutils {
                                   std::string("$1"));
     }
 
-    static inline auto join(const std::span<std::string> &vec,
+    static inline auto join(const std::span<const std::string> &vec,
                             const std::string &term) -> std::string {
         std::string result;
         result.reserve(vec.size() * term.size());

diff --git a/tests/test_strutils.cpp b/tests/test_strutils.cpp
@@ -50,4 +50,8 @@ TEST(TestStrutils, JoinTest) {
     std::vector<std::string> vec5 = {};
     std::string result5 = Strutils::join(vec5, " ");
     EXPECT_EQ(result5, "");
+
+    const std::vector<std::string> cvec2 = {"apple", "banana", "cherry"};
+    std::string cresult2 = Strutils::join(vec2, ", ");
+    EXPECT_EQ(cresult2, "apple, banana, cherry");
 }