[DarwinCompatibility] Minor fixes for DarwinCompatibility tests

spevans · spevans · commit ef0828e699c2 · 2018-03-24T12:38:21.000Z
- un-XFAIL some Darwin tests that run on 10.13.3

- Process.executableURL only available on 10.13

- Dont deallocate an autoreleasing pointer on Darwin.

- Add TestBundle.testBundleName() to handle 'TestFoundation' v
  'DarwinCompatibiltyTests' naming.
diff --git a/DarwinCompatibilityTests.xcodeproj/project.pbxproj b/DarwinCompatibilityTests.xcodeproj/project.pbxproj
@@ -702,7 +702,7 @@
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_STYLE = Automatic;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks";
-				MACOSX_DEPLOYMENT_TARGET = 10.12;
+				MACOSX_DEPLOYMENT_TARGET = 10.13;
 				OTHER_SWIFT_FLAGS = "";
 				"OTHER_SWIFT_FLAGS[arch=*]" = "";
 				PRODUCT_NAME = "$(TARGET_NAME)";
@@ -717,7 +717,7 @@
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_STYLE = Automatic;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks";
-				MACOSX_DEPLOYMENT_TARGET = 10.12;
+				MACOSX_DEPLOYMENT_TARGET = 10.13;
 				OTHER_SWIFT_FLAGS = "";
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SWIFT_VERSION = 4.0;
diff --git a/DarwinCompatibilityTests/DarwinShims.swift b/DarwinCompatibilityTests/DarwinShims.swift
@@ -50,7 +50,7 @@ extension Thread {
 extension Scanner {
     public func scanString(_ searchString: String) -> String? {
         var result: NSString? = nil
-        if scanString(string, into: &result), let str = result {
+        if scanString(searchString, into: &result), let str = result {
             return str as String
         }
         return nil
diff --git a/TestFoundation/TestBundle.swift b/TestFoundation/TestBundle.swift
@@ -31,6 +31,11 @@ internal func testBundle() -> Bundle {
     #endif
 }
 
+internal func testBundleName() -> String {
+    // Either 'TestFoundation' or 'DarwinCompatibilityTests'
+    return testBundle().infoDictionary!["CFBundleName"] as! String
+}
+
 class BundlePlayground {
     enum Layout {
         case flat
diff --git a/TestFoundation/TestCodable.swift b/TestFoundation/TestCodable.swift
@@ -561,15 +561,13 @@ class TestCodable : XCTestCase {
     ]
 
     func test_URLComponents_JSON() {
-#if !DARWIN_COMPATIBILITY_TESTS // crashes on native Darwin
         for (components) in urlComponentsValues {
             do {
                 try expectRoundTripEqualityThroughJSON(for: components)
             } catch let error {
                 XCTFail("\(error)")
             }
         }
-#endif
     }
 }
 
diff --git a/TestFoundation/TestFileManager.swift b/TestFoundation/TestFileManager.swift
@@ -155,7 +155,10 @@ class TestFileManager : XCTestCase {
         XCTAssertEqual(UInt8(bitPattern: result[0]), 0xE2)
         XCTAssertEqual(UInt8(bitPattern: result[1]), 0x98)
         XCTAssertEqual(UInt8(bitPattern: result[2]), 0x83)
+
+#if !DARWIN_COMPATIBILITY_TESTS // auto-released by Darwin's Foundation
         result.deallocate()
+#endif
     }
     
     func test_fileAttributes() {
diff --git a/TestFoundation/TestNSData.swift b/TestFoundation/TestNSData.swift
@@ -4110,15 +4110,13 @@ extension TestNSData {
     }
 
     func test_validateMutation_slice_mutableBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {
-#if !DARWIN_COMPATIBILITY_TESTS // Crashes on native Darwin
         var base = Data(referencing: NSData(bytes: "hello world", length: 11))
         base.append(contentsOf: [1, 2, 3, 4, 5, 6])
         var data = base[4..<6]
         data.withUnsafeMutableBytes { (ptr: UnsafeMutablePointer<UInt8>) in
             ptr.advanced(by: 1).pointee = 0xFF
         }
         XCTAssertEqual(data[data.startIndex.advanced(by: 1)], 0xFF)
-#endif
     }
 
     func test_validateMutation_slice_customBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {
@@ -4130,15 +4128,13 @@ extension TestNSData {
     }
 
     func test_validateMutation_slice_customMutableBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {
-#if !DARWIN_COMPATIBILITY_TESTS // Crashes on native Darwin
         var base = Data(referencing: AllOnesData(length: 1))
         base.count = 10
         var data = base[4..<6]
         data.withUnsafeMutableBytes { (ptr: UnsafeMutablePointer<UInt8>) in
             ptr.advanced(by: 1).pointee = 0xFF
         }
         XCTAssertEqual(data[data.startIndex.advanced(by: 1)], 0xFF)
-#endif
     }
 }
 
diff --git a/TestFoundation/TestNSError.swift b/TestFoundation/TestNSError.swift
@@ -50,7 +50,8 @@ class TestNSError : XCTestCase {
     }
 
     func test_CustomNSError_domain() {
-        XCTAssertEqual(SwiftCustomNSError.errorDomain, "TestFoundation.SwiftCustomNSError")
+        let name = testBundleName()
+        XCTAssertEqual(SwiftCustomNSError.errorDomain, "\(name).SwiftCustomNSError")
     }
 
     func test_CustomNSError_userInfo() {
diff --git a/TestFoundation/TestObjCRuntime.swift b/TestFoundation/TestObjCRuntime.swift
@@ -32,8 +32,9 @@ class TestObjCRuntime: XCTestCase {
     }
 
     func testStringFromClass() {
+        let name = testBundleName()
         XCTAssertEqual(NSStringFromClass(NSObject.self), "NSObject")
-        XCTAssertEqual(NSStringFromClass(SwiftClass.self), "TestFoundation.SwiftClass")
+        XCTAssertEqual(NSStringFromClass(SwiftClass.self), "\(name).SwiftClass")
 #if DEPLOYMENT_RUNTIME_OBJC || os(Linux)
         XCTAssertEqual(NSStringFromClass(XCTestCase.self), "XCTest.XCTestCase");
 #else
@@ -42,9 +43,10 @@ class TestObjCRuntime: XCTestCase {
     }
 
     func testClassFromString() {
+        let name = testBundleName()
         XCTAssertNotNil(NSClassFromString("NSObject"))
-        XCTAssertNotNil(NSClassFromString("TestFoundation.SwiftClass"))
-        XCTAssertNil(NSClassFromString("TestFoundation.SwiftClass.InnerClass"))
+        XCTAssertNotNil(NSClassFromString("\(name).SwiftClass"))
+        XCTAssertNil(NSClassFromString("\(name).SwiftClass.InnerClass"))
         XCTAssertNil(NSClassFromString("SwiftClass"))
         XCTAssertNil(NSClassFromString("MadeUpClassName"))
         XCTAssertNil(NSClassFromString("SwiftStruct"));
diff --git a/TestFoundation/TestProcess.swift b/TestFoundation/TestProcess.swift
@@ -46,10 +46,14 @@ class TestProcess : XCTestCase {
         let process = Process()
         
         let executablePath = "/bin/bash"
-        process.executableURL = URL(fileURLWithPath: executablePath)
-        
+        if #available(OSX 10.13, *) {
+            process.executableURL = URL(fileURLWithPath: executablePath)
+        } else {
+            // Fallback on earlier versions
+            process.launchPath = executablePath
+        }
         XCTAssertEqual(executablePath, process.launchPath)
-        
+
         process.arguments = ["-c", "exit 0"]
         process.launch()
         process.waitUntilExit()
diff --git a/TestFoundation/TestProcessInfo.swift b/TestFoundation/TestProcessInfo.swift
@@ -41,8 +41,12 @@ class TestProcessInfo : XCTestCase {
         // Assert that the original process name is "TestFoundation". This test
         // will fail if the test target ever gets renamed, so maybe it should
         // just test that the initial name is not empty or something?
-        let processInfo = ProcessInfo.processInfo
+#if DARWIN_COMPATIBILITY_TESTS
+        let targetName = "xctest"
+#else
         let targetName = "TestFoundation"
+#endif
+        let processInfo = ProcessInfo.processInfo
         let originalProcessName = processInfo.processName
         XCTAssertEqual(originalProcessName, targetName, "\"\(originalProcessName)\" not equal to \"TestFoundation\"")
         
diff --git a/TestFoundation/TestUserDefaults.swift b/TestFoundation/TestUserDefaults.swift
@@ -180,14 +180,12 @@ class TestUserDefaults : XCTestCase {
 	}
 	
 	func test_setValue_String() {
-#if !DARWIN_COMPATIBILITY_TESTS  // Works if run on its own, hangs if all tests in class are run
 		let defaults = UserDefaults.standard
 		
 		// Register a String value. UserDefaults.string(forKey:) is supposed to return the String
 		defaults.set("hello", forKey: "key1")
 		
 		XCTAssertEqual(defaults.string(forKey: "key1"), "hello")
-#endif
 	}
 
 	func test_setValue_NSURL() {
@@ -200,14 +198,12 @@ class TestUserDefaults : XCTestCase {
 	}
 
 	func test_setValue_URL() {
-#if !DARWIN_COMPATIBILITY_TESTS  // Works if run on its own, hangs if all tests in class are run
 		let defaults = UserDefaults.standard
 		
 		// Set a URL value. UserDefaults.url(forKey:) is supposed to return the URL
 		defaults.set(URL(fileURLWithPath: "/hello/world"), forKey: "key1")
 		
 		XCTAssertEqual(defaults.url(forKey: "key1"), URL(fileURLWithPath: "/hello/world"))
-#endif
 	}
 
 	func test_setValue_NSData() {

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ extension Thread {`
`50`	`50`	`extension Scanner {`
`51`	`51`	`public func scanString(_ searchString: String) -> String? {`
`52`	`52`	`var result: NSString? = nil`
`53`		`- if scanString(string, into: &result), let str = result {`
	`53`	`+ if scanString(searchString, into: &result), let str = result {`
`54`	`54`	`return str as String`
`55`	`55`	`}`
`56`	`56`	`return nil`
Original file line number	Diff line number	Diff line change
`@@ -561,15 +561,13 @@ class TestCodable : XCTestCase {`
`561`	`561`	`]`
`562`	`562`
`563`	`563`	`func test_URLComponents_JSON() {`
`564`		`-#if !DARWIN_COMPATIBILITY_TESTS // crashes on native Darwin`
`565`	`564`	`for (components) in urlComponentsValues {`
`566`	`565`	`do {`
`567`	`566`	`try expectRoundTripEqualityThroughJSON(for: components)`
`568`	`567`	`} catch let error {`
`569`	`568`	`XCTFail("\(error)")`
`570`	`569`	`}`
`571`	`570`	`}`
`572`		`-#endif`
`573`	`571`	`}`
`574`	`572`	`}`
`575`	`573`
Original file line number	Diff line number	Diff line change
`@@ -4110,15 +4110,13 @@ extension TestNSData {`
`4110`	`4110`	`}`
`4111`	`4111`
`4112`	`4112`	`func test_validateMutation_slice_mutableBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {`
`4113`		`-#if !DARWIN_COMPATIBILITY_TESTS // Crashes on native Darwin`
`4114`	`4113`	`var base = Data(referencing: NSData(bytes: "hello world", length: 11))`
`4115`	`4114`	`base.append(contentsOf: [1, 2, 3, 4, 5, 6])`
`4116`	`4115`	`var data = base[4..<6]`
`4117`	`4116`	`data.withUnsafeMutableBytes { (ptr: UnsafeMutablePointer<UInt8>) in`
`4118`	`4117`	`ptr.advanced(by: 1).pointee = 0xFF`
`4119`	`4118`	`}`
`4120`	`4119`	`XCTAssertEqual(data[data.startIndex.advanced(by: 1)], 0xFF)`
`4121`		`-#endif`
`4122`	`4120`	`}`
`4123`	`4121`
`4124`	`4122`	`func test_validateMutation_slice_customBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {`
`@@ -4130,15 +4128,13 @@ extension TestNSData {`
`4130`	`4128`	`}`
`4131`	`4129`
`4132`	`4130`	`func test_validateMutation_slice_customMutableBacking_withUnsafeMutableBytes_lengthLessThanLowerBound() {`
`4133`		`-#if !DARWIN_COMPATIBILITY_TESTS // Crashes on native Darwin`
`4134`	`4131`	`var base = Data(referencing: AllOnesData(length: 1))`
`4135`	`4132`	`base.count = 10`
`4136`	`4133`	`var data = base[4..<6]`
`4137`	`4134`	`data.withUnsafeMutableBytes { (ptr: UnsafeMutablePointer<UInt8>) in`
`4138`	`4135`	`ptr.advanced(by: 1).pointee = 0xFF`
`4139`	`4136`	`}`
`4140`	`4137`	`XCTAssertEqual(data[data.startIndex.advanced(by: 1)], 0xFF)`
`4141`		`-#endif`
`4142`	`4138`	`}`
`4143`	`4139`	`}`
`4144`	`4140`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,8 @@ class TestNSError : XCTestCase {`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`func test_CustomNSError_domain() {`
`53`		`- XCTAssertEqual(SwiftCustomNSError.errorDomain, "TestFoundation.SwiftCustomNSError")`
	`53`	`+ let name = testBundleName()`
	`54`	`+ XCTAssertEqual(SwiftCustomNSError.errorDomain, "\(name).SwiftCustomNSError")`
`54`	`55`	`}`
`55`	`56`
`56`	`57`	`func test_CustomNSError_userInfo() {`