Implement file getter in archive

8 files changed, 71 insertions, 8 deletions

diff --git a/src/archive.rs b/src/archive.rs
index fe03a12..e635007 100644
--- a/src/archive.rs
+++ b/src/archive.rs
@@ -1,20 +1,37 @@
 use crate::driver::{ArchiveRead, ArchiveWrite, Driver};
 use crate::ArchiveResult;
+use std::fs::File;
 use std::io::{Read, Write};
+use std::path::Path;
 
 pub struct Archive<D: Driver> {
-    pub driver: D,
+    pub(crate) driver: D,
 }
 
 impl<D: ArchiveRead> Archive<D>
 where
     D::IO: std::io::Read,
 {
-    pub fn new(io: D::IO) -> ArchiveResult<Self, D::Error> {
+    pub fn read(io: D::IO) -> ArchiveResult<Self, D::Error> {
         Ok(Self {
             driver: D::read(io)?,
         })
     }
+
+    pub fn read_from_file(path: impl AsRef<Path>) -> ArchiveResult<Self, D::Error>
+    where
+        D: ArchiveRead<IO = File>,
+    {
+        Self::read(File::open(path)?)
+    }
+
+    pub fn files(&self) -> Vec<&D::File> {
+        self.driver.files()
+    }
+
+    pub fn get_file(&self, name: &str) -> Option<&D::File> {
+        self.driver.get_file(name)
+    }
 }
 
 impl<D: ArchiveWrite> Archive<D> where D::IO: Read + Write {}
diff --git a/src/driver/driver.rs b/src/driver/driver.rs
index 3a8ed16..9c18e1f 100644
--- a/src/driver/driver.rs
+++ b/src/driver/driver.rs
@@ -16,6 +16,12 @@ where
 {
     // Create driver instance
     fn read(io: Self::IO) -> ArchiveResult<Self, Self::Error>;
+
+    // Return vec of files (sorted by name)
+    fn files(&self) -> Vec<&Self::File>;
+
+    // Return file by name
+    fn get_file(&self, name: &str) -> Option<&Self::File>;
 }
 
 pub trait ArchiveWrite: ArchiveRead
diff --git a/src/zip/archive.rs b/src/zip/archive.rs
new file mode 100644
index 0000000..9a244fc
--- /dev/null
+++ b/src/zip/archive.rs
@@ -0,0 +1,10 @@
+use crate::{Archive, Zip};
+use std::io::{Read, Seek, Write};
+
+impl<IO: Read + Seek> Archive<Zip<IO>> {
+    pub fn comment(&self) -> &String {
+        self.driver.comment()
+    }
+}
+
+impl<IO: Read + Write + Seek> Archive<Zip<IO>> {}
diff --git a/src/zip/driver.rs b/src/zip/driver.rs
index 8662623..650344e 100644
--- a/src/zip/driver.rs
+++ b/src/zip/driver.rs
@@ -4,10 +4,9 @@ use crate::zip::structs::{deserialize, EOCDR64Locator, ExtraHeader, CDR, EOCDR,
 use crate::zip::{ZipError, ZipFile, ZipResult};
 use chrono::{Local, NaiveDate, NaiveDateTime, NaiveTime};
 use std::collections::HashMap as Map;
-use std::fs::File;
 use std::io::{Read, Seek, SeekFrom, Write};
 
-pub struct Zip<IO = File> {
+pub struct Zip<IO> {
     io: IO,
 
     files: Map<String, ZipFile>,
@@ -169,10 +168,24 @@ impl<IO: Read + Seek> ArchiveRead for Zip<IO> {
 
         Ok(Self { io, files, comment })
     }
+
+    fn files(&self) -> Vec<&Self::File> {
+        let mut files: Vec<&Self::File> = self.files.values().collect();
+        files.sort_by_key(|f| &f.name);
+        files
+    }
+
+    fn get_file(&self, name: &str) -> Option<&Self::File> {
+        self.files.get(name)
+    }
 }
 
-impl<IO: Read + Seek> Zip<IO> {}
+impl<IO: Read + Seek> Zip<IO> {
+    pub fn comment(&self) -> &String {
+        &self.comment
+    }
+}
 
 impl<IO: Read + Write + Seek> ArchiveWrite for Zip<IO> {}
 
-impl<IO: Read + Write> Zip<IO> {}
+impl<IO: Read + Write + Seek> Zip<IO> {}
diff --git a/src/zip/file.rs b/src/zip/file.rs
index b4f2d7c..5b0723f 100644
--- a/src/zip/file.rs
+++ b/src/zip/file.rs
@@ -29,7 +29,7 @@ impl CompressionMethod {
 }
 
 pub struct BitFlag {
-    pub flag: u16,
+    flag: u16,
 }
 
 pub mod bit {
diff --git a/src/zip/mod.rs b/src/zip/mod.rs
index 23cadb9..89d748b 100644
--- a/src/zip/mod.rs
+++ b/src/zip/mod.rs
@@ -1,3 +1,4 @@
+mod archive;
 mod driver;
 mod error;
 mod file;
@@ -5,7 +6,7 @@ mod structs;
 
 pub use driver::Zip;
 pub use error::{ZipError, ZipResult};
-pub use file::ZipFile;
+pub use file::{bit, BitFlag, CompressionMethod, ZipFile};
 
 #[cfg(test)]
 mod tests;
diff --git a/tests/files/zip.zip b/tests/files/zip.zip
new file mode 100644
index 0000000..e17bbb7
--- /dev/null
+++ b/tests/files/zip.zip
diff --git a/tests/zip.rs b/tests/zip.rs
new file mode 100644
index 0000000..65244c1
--- /dev/null
+++ b/tests/zip.rs
@@ -0,0 +1,16 @@
+use archivator::{Archive, Zip};
+
+#[test]
+fn test_zip() {
+    let archive = Archive::<Zip<_>>::read_from_file("tests/files/zip.zip").unwrap();
+
+    assert_eq!(archive.comment(), "archive comment");
+    assert_eq!(
+        archive
+            .files()
+            .iter()
+            .map(|f| &f.name)
+            .collect::<Vec<&String>>(),
+        vec!["a", "b", "c"]
+    );
+}