fix comments

src/util/newconfig/ConfigDescription.hpp

@@ -82,27 +82,23 @@ struct ClioConfigDescription {
     {
         namespace fs = std::filesystem;
 
-        // Construct the full file path
-        fs::path const fullFilePath = path.string();
-
         // Validate the directory exists
-        auto const dir = fs::path(fullFilePath).parent_path();
-        if (!dir.empty() && !fs::exists(dir))
+        auto const dir = path.parent_path();
+        if (!dir.empty() && !fs::exists(dir)) {
             return std::unexpected<Error>{
-                fmt::format("Error: Directory {} does not exist or provided path is invalid", dir.string())
+                fmt::format("Error: Directory '{}' does not exist or provided path is invalid", dir.string())
             };
+        }
 
-        std::ofstream file(fullFilePath);
+        std::ofstream file(path.string());
         if (!file.is_open()) {
-            return std::unexpected{
-                fmt::format("Failed to create file '{}': {}", fullFilePath.string(), std::strerror(errno))
-            };
+            return std::unexpected{fmt::format("Failed to create file '{}': {}", path.string(), std::strerror(errno))};
         }
 
         writeConfigDescriptionToFile(file);
         file.close();
 
-        std::cout << "Markdown file generated successfully: " << fullFilePath << "\n";
+        std::cout << "Markdown file generated successfully: " << path << "\n";
         return {};
     }
 

tests/unit/app/CliArgsTests.cpp

@@ -183,7 +183,6 @@ TEST_F(CliArgsTestsWithTmpFile, Parse_ConfigDescription)
 
     // user provide config markdown file name as well
     ASSERT_TRUE(std::filesystem::exists(tmpFile.path));
-    std::filesystem::remove(tmpFile.path);
 
     EXPECT_EQ(
         action.apply(
@@ -220,6 +219,4 @@ TEST_F(CliArgsTestsWithTmpFile, Parse_ConfigDescriptionFileContent)
     // all keys that exist in clio config should be listed in config description file
     for (auto const& key : gClioConfig)
         EXPECT_TRUE(fileContent.find(key.first));
-
-    std::filesystem::remove(tmpFile.path);
 }