Remove redundant test function

Author: JamyGolden

tests/cli_apply_subcommand_tests.rs

@@ -3,9 +3,7 @@ mod utils;
 use std::fs;
 use std::path::Path;
 
-use crate::utils::{
-    read_file_to_string, setup, write_to_file, CURRENT_SCHEME_FILE_NAME, REPO_NAME,
-};
+use crate::utils::{setup, write_to_file, CURRENT_SCHEME_FILE_NAME, REPO_NAME};
 use anyhow::Result;
 
 #[test]
@@ -38,7 +36,7 @@ fn test_cli_apply_subcommand_with_setup() -> Result<()> {
         data_path.join(shell_theme_filename).exists(),
         "Path does not exist"
     );
-    assert_eq!(read_file_to_string(&current_scheme_path)?, scheme_name);
+    assert_eq!(fs::read_to_string(&current_scheme_path)?, scheme_name);
 
     cleanup()?;
     Ok(())

tests/cli_general_tests.rs

@@ -1,8 +1,7 @@
 mod utils;
 
 use crate::utils::{
-    cleanup, read_file_to_string, setup, write_to_file, COMMAND_NAME, CURRENT_SCHEME_FILE_NAME,
-    ORG_NAME, REPO_NAME,
+    cleanup, setup, write_to_file, COMMAND_NAME, CURRENT_SCHEME_FILE_NAME, ORG_NAME, REPO_NAME,
 };
 use anyhow::Result;
 use std::{fs, path::PathBuf};
@@ -121,7 +120,7 @@ fn test_cli_default_data_path() -> Result<()> {
 
     // This test is important to determine the config.toml is being read correctly
     assert_eq!(
-        read_file_to_string(&current_scheme_file_path)?,
+        fs::read_to_string(&current_scheme_file_path)?,
         init_scheme_name
     );
 
@@ -138,7 +137,7 @@ fn test_cli_default_data_path() -> Result<()> {
         "stdout does not contain the expected output"
     );
     assert_eq!(
-        read_file_to_string(&data_path.join(CURRENT_SCHEME_FILE_NAME))?,
+        fs::read_to_string(data_path.join(CURRENT_SCHEME_FILE_NAME))?,
         scheme_name
     );
     assert!(

tests/cli_init_subcommand_tests.rs

@@ -1,8 +1,8 @@
 mod utils;
 
-use crate::utils::{
-    read_file_to_string, setup, write_to_file, CURRENT_SCHEME_FILE_NAME, REPO_NAME,
-};
+use std::fs;
+
+use crate::utils::{setup, write_to_file, CURRENT_SCHEME_FILE_NAME, REPO_NAME};
 use anyhow::Result;
 
 #[test]
@@ -79,7 +79,7 @@ fn test_cli_init_subcommand_with_config_default_scheme() -> Result<()> {
     // ------
     // Assert
     // ------
-    let expected_scheme_name = read_file_to_string(&data_path.join(CURRENT_SCHEME_FILE_NAME))?;
+    let expected_scheme_name = fs::read_to_string(data_path.join(CURRENT_SCHEME_FILE_NAME))?;
     assert!(
         stdout.is_empty(),
         "stdout does not contain the expected output"

tests/utils.rs

@@ -1,7 +1,7 @@
 use anyhow::{anyhow, Result};
 use std::error::Error;
 use std::fs::{self, File};
-use std::io::{Read, Write};
+use std::io::Write;
 use std::path::{Path, PathBuf};
 use std::process::Command;
 use std::str;
@@ -79,20 +79,6 @@ pub fn write_to_file(path: &Path, contents: &str) -> Result<()> {
     Ok(())
 }
 
-#[allow(dead_code)]
-pub fn read_file_to_string(path: &Path) -> Result<String> {
-    if !path.exists() {
-        return Err(anyhow!("File does not exist: {}", path.display()));
-    }
-
-    let mut file = File::open(path)?;
-    let mut contents = String::new();
-
-    file.read_to_string(&mut contents)?;
-
-    Ok(contents)
-}
-
 #[allow(clippy::type_complexity)]
 pub fn setup(
     name: &str,