Revert "don't reuse same instance of TestFileManager across tests"

This reverts commit c1118aa.
realm · May 16, 2017 · 7918da7 · 7918da7
1 parent d836b9f
commit 7918da7
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/Tests/SwiftLintFrameworkTests/LinterCacheTests.swift b/Tests/SwiftLintFrameworkTests/LinterCacheTests.swift
@@ -13,17 +13,19 @@ import XCTest
 class LinterCacheTests: XCTestCase {
 
     private class TestFileManager: LintableFileManager {
-        fileprivate func filesToLint(inPath: String, rootDirectory: String? = nil) -> [String] {
+        func filesToLint(inPath: String, rootDirectory: String? = nil) -> [String] {
             return []
         }
 
-        fileprivate var stubbedModificationDateByPath: [String: Date] = [:]
+        internal var stubbedModificationDateByPath: [String: Date] = [:]
 
-        fileprivate func modificationDate(forFileAtPath path: String) -> Date? {
+        public func modificationDate(forFileAtPath path: String) -> Date? {
             return stubbedModificationDateByPath[path]
         }
     }
 
+    private let fileManager = TestFileManager()
+
     func testInitThrowsWhenUsingDifferentVersion() {
         let cache = ["version": "0.1.0"]
         checkError(LinterCacheError.differentVersion) {
@@ -61,7 +63,6 @@ class LinterCacheTests: XCTestCase {
 
     func testParsesViolations() {
         let cache = LinterCache(currentVersion: Version(value: "0.2.0"))
-        let fileManager = TestFileManager()
         cache.fileManager = fileManager
         let file = "foo.swift"
         let ruleDescription = RuleDescription(identifier: "rule", name: "Some rule",
@@ -87,7 +88,6 @@ class LinterCacheTests: XCTestCase {
 
     func testParsesViolationsWithEmptyViolations() {
         let cache = LinterCache(currentVersion: Version(value: "0.2.0"))
-        let fileManager = TestFileManager()
         cache.fileManager = fileManager
         let file = "foo.swift"
         fileManager.stubbedModificationDateByPath[file] = Date()
@@ -99,7 +99,6 @@ class LinterCacheTests: XCTestCase {
 
     func testParsesViolationsWithNoDate() {
         let cache = LinterCache(currentVersion: Version(value: "0.2.0"))
-        let fileManager = TestFileManager()
         cache.fileManager = fileManager
         let file = "foo.swift"
         let ruleDescription = RuleDescription(identifier: "rule", name: "Some rule",
@@ -125,7 +124,6 @@ class LinterCacheTests: XCTestCase {
 
     func testParsesViolationsWithDifferentDate() {
         let cache = LinterCache(currentVersion: Version(value: "0.2.0"))
-        let fileManager = TestFileManager()
         cache.fileManager = fileManager
         let file = "foo.swift"
         let ruleDescription = RuleDescription(identifier: "rule", name: "Some rule",