CHANGELOG.md

@@ -7,12 +7,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [4.0.0](https://git.kemitix.net/kemitix/kxio/compare/v3.2.0...v4.0.0) - 2024-12-15
-
-### Added
-
-- *(fs)* [**breaking**] Reader now supports non-utf8 files
-
 ## [3.2.0](https://git.kemitix.net/kemitix/kxio/compare/v3.1.0...v3.2.0) - 2024-12-12
 
 ### Added

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "kxio"
-version = "4.0.0"
+version = "3.2.0"
 edition = "2021"
 authors = ["Paul Campbell <pcampbell@kemitix.net>"]
 description = "Provides injectable Filesystem and Network resources to make code more testable"

examples/get.rs

@@ -89,7 +89,7 @@ fn read_file(file: &FileHandle) -> kxio::Result<()> {
 
     // Creates a `Reader` which loaded the file into memory.
     let reader: kxio::fs::Reader = file.reader()?;
-    let contents: &str = reader.as_str()?;
+    let contents: &str = reader.as_str();
     println!("{contents}");
 
     Ok(())
@@ -106,7 +106,6 @@ fn delete_file(file: FileHandle) -> kxio::Result<()> {
 
 #[cfg(test)]
 mod tests {
-    use assert2::let_assert;
     use http::StatusCode;
 
     use super::*;
@@ -147,7 +146,7 @@ mod tests {
         // Read the file
         let file = fs.file(&file_path);
         let reader = file.reader().expect("reader");
-        let_assert!(Ok(contents) = reader.as_str());
+        let contents = reader.as_str();
 
         assert_eq!(contents, "contents");
 

src/fs/reader.rs

@@ -8,13 +8,13 @@ use super::Error;
 /// A reader for a file.
 #[derive(Clone, Debug, Default, PartialEq, Eq)]
 pub struct Reader {
-    contents: Vec<u8>,
+    contents: String,
 }
 impl Reader {
     #[tracing::instrument]
     pub(super) fn new(path: &Path) -> Result<Self> {
         tracing::debug!("std::fs::read_to_string");
-        let contents = std::fs::read(path).map_err(Error::Io)?;
+        let contents = std::fs::read_to_string(path).map_err(Error::Io)?;
         tracing::debug!(len = contents.len(), "contents");
         Ok(Self { contents })
     }
@@ -32,8 +32,8 @@ impl Reader {
     /// #    Ok(())
     /// # }
     /// ```
-    pub fn as_str(&self) -> Result<&str> {
+    pub fn as_str(&self) -> &str {
-        std::str::from_utf8(&self.contents).map_err(Error::Utf8Error)
+        &self.contents
     }
 
     /// Returns an iterator over the lines in the file.
@@ -49,8 +49,8 @@ impl Reader {
     /// #    Ok(())
     /// # }
     /// ```
-    pub fn lines(&self) -> Result<Lines<'_>> {
+    pub fn lines(&self) -> Lines<'_> {
-        self.as_str().map(|s| s.lines())
+        self.contents.lines()
     }
 
     /// Returns the contents of the file as bytes.
@@ -67,11 +67,11 @@ impl Reader {
     /// # }
     /// ```
     pub fn bytes(&self) -> &[u8] {
-        &self.contents
+        self.contents.as_bytes()
     }
 }
 impl Display for Reader {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        write!(f, "{}", self.as_str().map_err(|_| std::fmt::Error)?)
+        write!(f, "{}", self.contents)
     }
 }

src/fs/result.rs

@@ -12,8 +12,6 @@ pub enum Error {
 
     IoString(String),
 
-    Utf8Error(std::str::Utf8Error),
-
     #[display("Path access attempted outside of base ({base:?}): {path:?}")]
     PathTraversal {
         base: PathBuf,
@@ -36,7 +34,6 @@ impl Clone for Error {
         match self {
             Error::Io(err) => Error::IoString(err.to_string()),
             Error::IoString(err) => Error::IoString(err.clone()),
-            Error::Utf8Error(err) => Error::Utf8Error(*err),
             Error::PathTraversal { base, path } => Error::PathTraversal {
                 base: base.clone(),
                 path: path.clone(),

tests/fs.rs

@@ -723,7 +723,7 @@ mod file {
                 file.write(&contents).expect("write");
 
                 let reader = file.reader().expect("reader");
-                let string = reader.as_str().expect("as_str");
+                let string = reader.as_str();
 
                 assert_eq!(string, contents);
             }
@@ -743,7 +743,7 @@ mod file {
                 file.write(&contents).expect("write");
 
                 let reader = file.reader().expect("reader");
-                let lines = reader.lines().expect("lines").collect::<Vec<_>>();
+                let lines = reader.lines().collect::<Vec<_>>();
 
                 assert_eq!(lines, vec!["line 1", "line 2", &line3]);
             }
@@ -759,7 +759,7 @@ mod file {
                 file.write("line 1\nline 2").expect("write");
 
                 let reader = file.reader().expect("reader");
-                let lines = reader.lines().expect("lines").collect::<Vec<_>>();
+                let lines = reader.lines().collect::<Vec<_>>();
 
                 assert_eq!(lines, vec!["line 1", "line 2"]);