fix: logs are incomplete (#3)

- flush before zip logs

README.md

@@ -41,6 +41,22 @@ let url = try fileLogHandler.zipLogs()
 
 ![zip logs](docs/images/image-20220730230255924.png)
 
+For iOS side, you can share logs through `UIActivityViewController`, e.g.
+
+```swift
+guard let url = try? hadesLogger.zipLogs() else {
+    return
+}
+var filesToShare = [Any]()
+filesToShare.append(url)
+let activityViewController = UIActivityViewController(
+    activityItems: filesToShare,
+    applicationActivities: nil
+)
+```
+
+<img src="docs/images/image-20220731005753835.png" alt="share logs at iOS device" style="zoom: 25%;" />
+
 ## Class Diagram
 
 ![class diagram](./docs/01.class_diagram.png) 

Sources/HadesLogger/HadesLogger.swift

@@ -6,7 +6,7 @@ public struct HadesLogger {
     private let oslogHandler: OSLogHandler
     private let fileLogHandler: FileLogHandler
 
-    init(label: String) {
+    public init(label: String) {
         self.label = label
         oslogHandler = OSLogHandler(label: label)
         fileLogHandler = FileLogHandler()

Sources/HadesLogger/Handler/FileLogHandler.swift

@@ -128,7 +128,17 @@ class FileLogHandler: LogHandler {
     /// zip all logs into a zip file
     /// - Returns: a url of log file
     func zipLogs() throws -> URL? {
+        try loggerQueue?.sync {
+            try? self._flush(false)
+            return try self._zipLogs()
+        }
+    }
+
+    private func _zipLogs() throws -> URL? {
         let archiveURL = logFileManager.logDirectory.appendingPathComponent("logs.zip")
+        if logFileManager.isExistsDirectory(at: archiveURL) {
+            try logFileManager.remove(at: archiveURL)
+        }
         guard let archive = Archive(url: archiveURL, accessMode: .create) else  {
             return nil
         }

Tests/HadesLoggerTests/FileLogHandlerTests.swift

@@ -50,47 +50,41 @@ class FileLogHandlerTests: XCTestCase {
         for i in 0..<40 {
             log.debug("test \(i)")
         }
-        try fileLogHandler.flush()
-        try queue.sync {
-            let baseURL = try! FileManager.default.url(
-                for: .cachesDirectory,
-                in: .userDomainMask,
-                appropriateFor: nil,
-                create: true
-            )
-            let logDirectory = baseURL.appendingPathComponent("hades-log", isDirectory: true)
-            let logFileManager = LogFileManager(logDirectory: logDirectory)
-            let logs = try logFileManager.queryLogFiles()
-            if let logFileURL = logs.first?.url {
-                let content = try String(contentsOf: logFileURL)
-                XCTAssert(content.hasSuffix("test 39"), "This file should end with 'test 39' according to the code at line 50.")
-            }
-            for log in logs {
-                XCTAssert(log.size <= fileLogHandler.maximumFileSize)
-            }
-            if logs.count >= 2 {
-                XCTAssert((logs[0].size + logs[1].size) > fileLogHandler.maximumFileSize)
-            }
+        let baseURL = try! FileManager.default.url(
+            for: .cachesDirectory,
+            in: .userDomainMask,
+            appropriateFor: nil,
+            create: true
+        )
+        let logDirectory = baseURL.appendingPathComponent("hades-log", isDirectory: true)
+        let logFileManager = LogFileManager(logDirectory: logDirectory)
+        let logs = try logFileManager.queryLogFiles()
+        if let logFileURL = logs.first?.url {
+            let content = try String(contentsOf: logFileURL)
+            XCTAssert(content.hasSuffix("test 39"), "This file should end with 'test 39' according to the code at line 50.")
+        }
+        for log in logs {
+            XCTAssert(log.size <= fileLogHandler.maximumFileSize)
+        }
+        if logs.count >= 2 {
+            XCTAssert((logs[0].size + logs[1].size) > fileLogHandler.maximumFileSize)
         }
     }
 
     func testZip() throws {
         for i in 0..<40 {
             log.debug("test \(i)")
         }
-        try fileLogHandler.flush()
 
-        try queue.sync {
-            guard let url = try fileLogHandler.zipLogs() else {
-                XCTAssert(false)
-                return
-            }
-            let attributes = try FileManager.default.attributesOfItem(atPath: url.path)
-            guard let fileSize = attributes[.size] as? Int else {
-                XCTAssert(false)
-                return
-            }
-            XCTAssert(fileSize > 2_000 )
+        guard let url = try fileLogHandler.zipLogs() else {
+            XCTAssert(false)
+            return
+        }
+        let attributes = try FileManager.default.attributesOfItem(atPath: url.path)
+        guard let fileSize = attributes[.size] as? Int else {
+            XCTAssert(false)
+            return
         }
+        XCTAssert(fileSize > 2_000 )
     }
 }