Fix tests compilation for FileHandle missing functions in old Darwin OSes

Author: Frizlab

Tests/JSONLoggerTests/Helpers/FileHandleHelper.swift

@@ -0,0 +1,50 @@
+import Foundation
+
+
+
+private let system_read = read
+private let system_close = close
+
+extension FileHandle {
+	
+	func jl_close() throws {
+		if #available(macOS 10.15, *) {
+			try close()
+		} else {
+			/* closeFile exists but I’m not sure it never throws an ObjC exception, so I call the C function. */
+			let ret = system_close(fileDescriptor)
+			guard ret == 0 else {
+				throw Errno()
+			}
+		}
+	}
+	
+	func jl_readToEnd() throws -> Data? {
+		if #available(macOS 10.15, tvOS 13.0, iOS 13.0, watchOS 6.0, *) {
+			return try readToEnd()
+		} else {
+			let buffer = UnsafeMutableRawBufferPointer.allocate(byteCount: 512, alignment: MemoryLayout<UInt8>.alignment)
+			defer {buffer.deallocate()}
+			
+			var nread = 0
+			var ret = Data()
+			repeat {
+				nread = system_read(fileDescriptor, buffer.baseAddress, buffer.count)
+				ret += buffer
+			} while nread > 0
+			guard nread >= 0 else {
+				throw Errno()
+			}
+			return ret
+		}
+	}
+	
+}
+
+
+struct Errno : Error {
+	var err: errno_t
+	init() {
+		self.err = errno
+	}
+}

Tests/JSONLoggerTests/PipeHelper.swift renamed to Tests/JSONLoggerTests/Helpers/PipeHelper.swift

@@ -66,17 +66,17 @@ final class JSONLoggerTests : XCTestCase {
 		let pipe = Pipe()
 		let jsonLogger = JSONLogger(label: "best-logger", fileHandle: pipe.fileHandleForWriting, lineSeparator: Data([0x0a]), prefix: Data(), suffix: Data())
 		jsonLogger.log(level: .info, message: "First log message", metadata: nil, source: "dummy-source", file: "dummy-file", function: "dummy-function", line: 42)
-		try pipe.fileHandleForWriting.close()
-		let data = try pipe.fileHandleForReading.readToEnd() ?? Data()
+		try pipe.fileHandleForWriting.jl_close()
+		let data = try pipe.fileHandleForReading.jl_readToEnd() ?? Data()
 		XCTAssertEqual(data.first, 0x7b)
 	}
 
 	func testSeparatorForNotFirstLog() throws {
 		let pipe = Pipe()
 		let jsonLogger = JSONLogger(label: "best-logger", fileHandle: pipe.fileHandleForWriting, lineSeparator: Data([0x0a]), prefix: Data(), suffix: Data())
 		jsonLogger.log(level: .info, message: "Not first log message", metadata: nil, source: "dummy-source", file: "dummy-file", function: "dummy-function", line: 42)
-		try pipe.fileHandleForWriting.close()
-		let data = try pipe.fileHandleForReading.readToEnd() ?? Data()
+		try pipe.fileHandleForWriting.jl_close()
+		let data = try pipe.fileHandleForReading.jl_readToEnd() ?? Data()
 		XCTAssertEqual(data.first, 0x0a)
 	}
 
@@ -91,8 +91,8 @@ final class JSONLoggerTests : XCTestCase {
 		let pipe = Pipe()
 		let jsonLogger = JSONLogger(label: "best-logger", fileHandle: pipe.fileHandleForWriting)
 		jsonLogger.log(level: ref.level, message: "\(ref.message)", metadata: ["yolo": .stringConvertible(BestStruct(val: 21))], source: ref.source, file: ref.file, function: ref.function, line: ref.line)
-		try pipe.fileHandleForWriting.close()
-		let data = try pipe.fileHandleForReading.readToEnd() ?? Data()
+		try pipe.fileHandleForWriting.jl_close()
+		let data = try pipe.fileHandleForReading.jl_readToEnd() ?? Data()
 		//print(data.reduce("", { $0 + String(format: "%02x", $1) }))
 		var line = try Self.defaultJSONDecoder.decode(LogLine.self, from: data)
 		XCTAssertLessThanOrEqual(line.date.timeIntervalSince(ref.date), 0.1)
@@ -122,8 +122,8 @@ final class JSONLoggerTests : XCTestCase {
 		let pipe = Pipe()
 		let jsonLogger = JSONLogger(label: "best-logger", fileHandle: pipe.fileHandleForWriting, jsonEncoder: failEncoder)
 		jsonLogger.log(level: ref.level, message: "\(ref.message)", metadata: ["yolo": .stringConvertible(BestStruct(val: 21))], source: ref.source, file: ref.file, function: ref.function, line: ref.line)
-		try pipe.fileHandleForWriting.close()
-		let data = try pipe.fileHandleForReading.readToEnd() ?? Data()
+		try pipe.fileHandleForWriting.jl_close()
+		let data = try pipe.fileHandleForReading.jl_readToEnd() ?? Data()
 		var line = try Self.defaultJSONDecoder.decode(LogLine.self, from: data)
 		XCTAssertLessThanOrEqual(line.date.timeIntervalSince(ref.date), 0.1)
 		line.date = mangled.date