Move TemplateCommand struct into template mod

Author: sbernauer

src/cli_args.rs

@@ -1,8 +1,6 @@
-use std::path::PathBuf;
-
 use clap::{Parser, Subcommand};
 
-use config_utils::file_types::FileType;
+use config_utils::template::cli_args::TemplateCommand;
 
 /// Utility that helps you handling config files.
 #[derive(Debug, Parser)]
@@ -14,20 +12,5 @@ pub struct Args {
 
 #[derive(Debug, Subcommand)]
 pub enum Command {
-    /// Fill out variables in config files from either env variables or files directly.
-    Template {
-        /// The path to the file that should be templated
-        file: PathBuf,
-
-        /// The optional file type of the file to be templated. If this is not specified this utility will try to infer
-        /// the type based on the file name.
-        #[arg(value_enum)]
-        file_type: Option<FileType>,
-
-        /// By default inserted values are automatically escaped according to the deteced file format. You can disable
-        /// this, e.g. when you need to insert XML tags (as they otherwise would be escaped).
-        /// NOTE: Please make sure to correctly escape the inserted text on your own!
-        #[clap(long)]
-        dont_escape: bool,
-    },
+    Template(TemplateCommand),
 }

src/main.rs

@@ -1,5 +1,5 @@
 use clap::Parser;
-use config_utils::template::{self, template};
+use config_utils::template::{self, cli_args::TemplateCommand, template};
 use snafu::{ResultExt, Snafu};
 
 use cli_args::{Args, Command};
@@ -19,11 +19,11 @@ fn main() -> Result<()> {
     let args = Args::parse();
 
     match args.command {
-        Command::Template {
+        Command::Template(TemplateCommand {
             file,
             file_type,
             dont_escape,
-        } => {
+        }) => {
             template(&file, file_type.as_ref(), !dont_escape).context(TemplateFileSnafu)?;
         }
     }

src/template/cli_args.rs

@@ -0,0 +1,23 @@
+use std::path::PathBuf;
+
+use clap::Parser;
+
+use crate::file_types::FileType;
+
+/// Fill out variables in config files from either env variables or files directly.
+#[derive(Debug, Parser)]
+pub struct TemplateCommand {
+    /// The path to the file that should be templated
+    pub file: PathBuf,
+
+    /// The optional file type of the file to be templated. If this is not specified this utility will try to infer
+    /// the type based on the file name.
+    #[arg(value_enum)]
+    pub file_type: Option<FileType>,
+
+    /// By default inserted values are automatically escaped according to the deteced file format. You can disable
+    /// this, e.g. when you need to insert XML tags (as they otherwise would be escaped).
+    /// NOTE: Please make sure to correctly escape the inserted text on your own!
+    #[clap(long)]
+    pub dont_escape: bool,
+}

src/template/mod.rs

@@ -12,6 +12,8 @@ use crate::{
     ENV_VAR_PATTERN_END, ENV_VAR_PATTERN_START, FILE_PATTERN_END, FILE_PATTERN_START,
 };
 
+pub mod cli_args;
+
 #[derive(Debug, Snafu)]
 pub enum Error {
     #[snafu(display("Could not read file {file_name:?}"))]
@@ -125,7 +127,7 @@ pub fn template(file_name: &PathBuf, file_type: Option<&FileType>, escape: bool)
             })?;
     }
 
-    fs::rename(&tmp_file_name, &file_name).context(RenameTemporaryFileSnafu {
+    fs::rename(&tmp_file_name, file_name).context(RenameTemporaryFileSnafu {
         tmp_file_name,
         destination_file_name: file_name,
     })?;

tests/templating.rs

@@ -2,7 +2,7 @@ use std::{
     env,
     fs::{self, File},
     io::Write,
-    path::PathBuf,
+    path::{Path, PathBuf},
 };
 
 use rstest::rstest;
@@ -21,13 +21,13 @@ fn test_file_templating(#[files("tests/resources/**/*.in")] test_file_in: PathBu
     fs::copy(&test_file_in, &test_file).unwrap();
     template(&test_file, None, true).unwrap();
 
-    let actual = fs::read_to_string(&test_file).unwrap();
-    let expected = fs::read_to_string(&test_file_expected).unwrap();
+    let actual = fs::read_to_string(test_file).unwrap();
+    let expected = fs::read_to_string(test_file_expected).unwrap();
 
     similar_asserts::assert_eq!(actual, expected);
 }
 
-fn set_example_envs(example_dir: &PathBuf) {
+fn set_example_envs(example_dir: &Path) {
     // SAFETY: We only use a single thread to set this env vars
     env::set_var("ENV_TEST", "foo");
     env::set_var("ENV_TEST_USERNAME", "example user");