diff --git a/Logger.xcodeproj/project.pbxproj b/Logger.xcodeproj/project.pbxproj index a26581b..076fc31 100644 --- a/Logger.xcodeproj/project.pbxproj +++ b/Logger.xcodeproj/project.pbxproj @@ -21,6 +21,7 @@ 2ED077D721329CA30058EEFC /* LoggetTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 2ED077D621329CA30058EEFC /* LoggetTests.swift */; }; 2ED077D92132A4820058EEFC /* AgregateLoggerTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 2ED077D82132A4820058EEFC /* AgregateLoggerTests.swift */; }; 2ED077DB2132B0320058EEFC /* FileSystem.swift in Sources */ = {isa = PBXBuildFile; fileRef = 2ED077DA2132B0320058EEFC /* FileSystem.swift */; }; + 2ED103E12135C61100EB3683 /* FileRotateTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 2ED103E02135C61100EB3683 /* FileRotateTests.swift */; }; /* End PBXBuildFile section */ /* Begin PBXContainerItemProxy section */ @@ -64,6 +65,7 @@ 2ED077D621329CA30058EEFC /* LoggetTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = LoggetTests.swift; sourceTree = ""; }; 2ED077D82132A4820058EEFC /* AgregateLoggerTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AgregateLoggerTests.swift; sourceTree = ""; }; 2ED077DA2132B0320058EEFC /* FileSystem.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = FileSystem.swift; sourceTree = ""; }; + 2ED103E02135C61100EB3683 /* FileRotateTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = FileRotateTests.swift; sourceTree = ""; }; /* End PBXFileReference section */ /* Begin PBXFrameworksBuildPhase section */ @@ -91,6 +93,7 @@ 2E58D35C21316C3500BEF81A /* LogStringConvertibleTests.swift */, 2ED077D621329CA30058EEFC /* LoggetTests.swift */, 2ED077D82132A4820058EEFC /* AgregateLoggerTests.swift */, + 2ED103E02135C61100EB3683 /* FileRotateTests.swift */, 2E58D35E21316C3500BEF81A /* Info.plist */, ); path = UnitTests; @@ -241,6 +244,7 @@ files = ( 2E58D35D21316C3500BEF81A /* LogStringConvertibleTests.swift in Sources */, 2ED077D92132A4820058EEFC /* AgregateLoggerTests.swift in Sources */, + 2ED103E12135C61100EB3683 /* FileRotateTests.swift in Sources */, 2ED077D721329CA30058EEFC /* LoggetTests.swift in Sources */, ); runOnlyForDeploymentPostprocessing = 0; diff --git a/UnitTests/FileRotateTests.swift b/UnitTests/FileRotateTests.swift new file mode 100644 index 0000000..13ca7c0 --- /dev/null +++ b/UnitTests/FileRotateTests.swift @@ -0,0 +1,181 @@ +// +// MIT License +// +// Copyright (c) 2018 Wojciech Nagrodzki +// +// Permission is hereby granted, free of charge, to any person obtaining a copy +// of this software and associated documentation files (the "Software"), to deal +// in the Software without restriction, including without limitation the rights +// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell +// copies of the Software, and to permit persons to whom the Software is +// furnished to do so, subject to the following conditions: +// +// The above copyright notice and this permission notice shall be included in all +// copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR +// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, +// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE +// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER +// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, +// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE +// SOFTWARE. +// + +import XCTest +@testable import Logger + +class FileRotateTests: XCTestCase { + + let logURL = URL(fileURLWithPath: "/var/log/application.log") + let log1URL = URL(fileURLWithPath: "/var/log/application.log.1") + let log2URL = URL(fileURLWithPath: "/var/log/application.log.2") + let log3URL = URL(fileURLWithPath: "/var/log/application.log.3") + + func test_1rotation_0files() { + let fileSystem = FileSystemMock(files: []) + let logrotate = FileRotate(fileURL: logURL, rotations: 1, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set() + XCTAssertEqual(actual, expected) + } + + func test_1rotation_1file() { + let fileSystem = FileSystemMock(files: [logURL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 1, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL]) + XCTAssertEqual(actual, expected) + } + + func test_1rotation_2files() { + let fileSystem = FileSystemMock(files: [logURL, log1URL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 1, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL]) + XCTAssertEqual(actual, expected) + } + + func test_1rotation_3files() { + let fileSystem = FileSystemMock(files: [logURL, log1URL, log2URL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 1, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL, log2URL]) + XCTAssertEqual(actual, expected) + } + + func test_2rotations_0files() { + let fileSystem = FileSystemMock(files: []) + let logrotate = FileRotate(fileURL: logURL, rotations: 2, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set() + XCTAssertEqual(actual, expected) + } + + func test_2rotations_1file() { + let fileSystem = FileSystemMock(files: [logURL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 2, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL]) + XCTAssertEqual(actual, expected) + } + + func test_2rotations_2files() { + let fileSystem = FileSystemMock(files: [logURL, log1URL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 2, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL, log2URL]) + XCTAssertEqual(actual, expected) + } + + func test_2rotations_3files() { + let fileSystem = FileSystemMock(files: [logURL, log1URL, log2URL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 2, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL, log2URL]) + XCTAssertEqual(actual, expected) + } + + func test_2rotations_4files() { + let fileSystem = FileSystemMock(files: [logURL, log1URL, log2URL, log3URL]) + let logrotate = FileRotate(fileURL: logURL, rotations: 2, fileSystem: fileSystem) + try? logrotate.rotate() + + let actual = fileSystem.files + let expected = Set([log1URL, log2URL, log3URL]) + XCTAssertEqual(actual, expected) + } + + func testErrorPropagation() { + let fileSystem = BrokenFileSystem() + let logrotate = FileRotate(fileURL: logURL, rotations: 1, fileSystem: fileSystem) + + XCTAssertThrowsError(try logrotate.rotate(), "An error when removing or moving an item") { (error) in + XCTAssertTrue(error is BrokenFileSystem.IOError) + } + } +} + +private class FileSystemMock: FileSystem { + + private(set) var files = Set() + + init(files: Set) { + self.files = files + } + + func itemExists(at URL: URL) -> Bool { + return files.contains(URL) + } + + func removeItem(at URL: URL) throws { + files.remove(URL) + } + + func moveItem(at srcURL: URL, to dstURL: URL) throws { + files.remove(srcURL) + files.insert(dstURL) + } + + func createFile(at URL: URL) -> Bool { + files.insert(URL) + return true + } +} + +private class BrokenFileSystem: FileSystem { + + struct IOError: Error { } + + func itemExists(at URL: URL) -> Bool { + return true + } + + func removeItem(at URL: URL) throws { + throw IOError() + } + + func moveItem(at srcURL: URL, to dstURL: URL) throws { + throw IOError() + } + + func createFile(at URL: URL) -> Bool { + return false + } +}