Fix process handle leak on Windows

- Add a missing CloseHandle as part of the process termination handling
- Enable Process tests on Windows

Author: daveinglis

Package.swift

@@ -349,7 +349,7 @@ let package = Package(
                 "FoundationNetworking",
                 "XCTest",
                 "Testing",
-                .target(name: "xdgTestHelper", condition: .when(platforms: [.linux, .android]))
+                .target(name: "xdgTestHelper", condition: .when(platforms: [.linux, .android, .windows]))
             ],
             resources: [
                 .copy("Foundation/Resources")

Sources/Foundation/Process.swift

@@ -653,6 +653,7 @@ open class Process: NSObject, @unchecked Sendable {
                 process._terminationReason = .exit
             }
 
+            CloseHandle(process.processHandle)
             // Signal waitUntilExit() and optionally invoke termination handler.
             process.terminateRunLoop()
 

Tests/Foundation/TestBundle.swift

@@ -43,15 +43,7 @@ internal func testBundleName() -> String {
 }
 
 internal func xdgTestHelperURL() throws -> URL {
-    #if os(Windows)
-    // Adding the xdgTestHelper as a dependency of TestFoundation causes its object files (including the main function) to be linked into the test runner executable as well
-    // While this works on Linux due to special linker functionality, this doesn't work on Windows and results in a collision between the two main symbols
-    // SwiftPM also cannot support depending on this executable (to ensure it is built) without also linking its objects into the test runner
-    // For those reasons, using the xdgTestHelper on Windows is currently unsupported and tests that rely on it must be skipped
-    throw XCTSkip("xdgTestHelper is not supported during testing on Windows (test executables are not supported by SwiftPM on Windows)")
-    #else
     testBundle().bundleURL.deletingLastPathComponent().appendingPathComponent("xdgTestHelper")
-    #endif
 }
 
 

Tests/Foundation/TestFileManager.swift

@@ -1235,6 +1235,9 @@ class TestFileManager : XCTestCase {
     }
 
     func test_fetchXDGPathsFromHelper() throws {
+        #if os(Windows)
+        throw XCTSkip("This test is disabled on Windows, needs investigation.")
+        #else
         let prefix = NSHomeDirectory() + "/_Foundation_Test_"
 
         let configuration = """
@@ -1674,7 +1677,7 @@ class TestFileManager : XCTestCase {
     func test_windowsPaths() throws {
         #if !os(Windows)
         throw XCTSkip("This test is only enabled on Windows")
-        #else
+        #endif
         let fm = FileManager.default
         let tmpPath = writableTestDirectoryURL.withUnsafeFileSystemRepresentation { String(cString: $0!) }
         do {