Rename functions (#207)

kyu08 · Jan 22, 2024 · 2388552 · 2388552
1 parent f0972b3
commit 2388552
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/file/toml.rs b/src/file/toml.rs
@@ -31,7 +31,7 @@ struct History {
     executed_targets: Vec<String>,
 }
 
-pub fn read_history(content: String) -> Result<Vec<(PathBuf, Vec<String>)>> {
+pub fn parse_history(content: String) -> Result<Vec<(PathBuf, Vec<String>)>> {
     let histories: Histories = toml::from_str(&content)?;
 
     let mut result: Vec<(PathBuf, Vec<String>)> = Vec::new();
@@ -42,7 +42,7 @@ pub fn read_history(content: String) -> Result<Vec<(PathBuf, Vec<String>)>> {
     Ok(result)
 }
 
-pub fn write_history(
+pub fn store_history(
     history_directory_path: PathBuf,
     history_file_name: String,
     histories_tuple: Vec<(PathBuf, Vec<String>)>,
@@ -113,13 +113,13 @@ executed-targets = ["run", "echo1"]
             match case.expect {
                 Ok(v) => assert_eq!(
                     v,
-                    read_history(case.content).unwrap(),
+                    parse_history(case.content).unwrap(),
                     "\nFailed: 🚨{:?}🚨\n",
                     case.title,
                 ),
                 Err(e) => assert_eq!(
                     e.to_string(),
-                    read_history(case.content).unwrap_err().to_string(),
+                    parse_history(case.content).unwrap_err().to_string(),
                     "\nFailed: 🚨{:?}🚨\n",
                     case.title,
                 ),

diff --git a/src/usecase/fzf_make/app.rs b/src/usecase/fzf_make/app.rs
@@ -148,7 +148,7 @@ impl Model<'_> {
                     Err(_) => return Histories::new(makefile_path, vec![]), // NOTE: Show error message on message pane https://github.com/kyu08/fzf-make/issues/152
                     Ok(c) => c,
                 };
-            let histories = match toml::read_history(content.to_string()) {
+            let histories = match toml::parse_history(content.to_string()) {
                 Err(_) => vec![], // NOTE: Show error message on message pane https://github.com/kyu08/fzf-make/issues/152
                 Ok(h) => h,
             };
@@ -431,7 +431,7 @@ fn update(model: &mut Model, message: Option<Message>) {
 
             if let (Some((dir, file_name)), Some(h)) = (history_file_path(), &model.histories) {
                 // TODO: handle error
-                let _ = toml::write_history(dir, file_name, h.to_tuple());
+                let _ = toml::store_history(dir, file_name, h.to_tuple());
             }
         }