From 137893afd40f16306526128040b16addedc86c3e Mon Sep 17 00:00:00 2001 From: Boris Buegling Date: Fri, 14 Nov 2025 09:42:40 -0800 Subject: [PATCH] Remove linux/windows guard from `DependencyValidationTests` These tests are actually macOS-only at this point, so the requirements should reflect that. --- Tests/SWBBuildSystemTests/DependencyValidationTests.swift | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Tests/SWBBuildSystemTests/DependencyValidationTests.swift b/Tests/SWBBuildSystemTests/DependencyValidationTests.swift index fca50223..836f76c0 100644 --- a/Tests/SWBBuildSystemTests/DependencyValidationTests.swift +++ b/Tests/SWBBuildSystemTests/DependencyValidationTests.swift @@ -494,12 +494,12 @@ fileprivate struct DependencyValidationTests: CoreBasedTests { } } - @Test(.requireSDKs(.host), .skipHostOS(.windows, "toolchain too old"), .skipHostOS(.linux, "toolchain too old")) + @Test(.requireSDKs(.macOS)) func validateModuleDependenciesSwift() async throws { try await validateModuleDependenciesSwift(explicitModules: true) } - @Test(.requireSDKs(.host), .skipHostOS(.windows, "toolchain too old"), .skipHostOS(.linux, "toolchain too old")) + @Test(.requireSDKs(.macOS)) func validateModuleDependenciesSwiftExplicitModulesOff() async throws { try await validateModuleDependenciesSwift(explicitModules: false) } @@ -574,7 +574,7 @@ fileprivate struct DependencyValidationTests: CoreBasedTests { } } - @Test(.requireSDKs(.host), .requireClangFeatures(.printHeadersDirectPerFile), .skipHostOS(.windows, "toolchain too old"), .skipHostOS(.linux, "toolchain too old"), arguments: [false, true]) + @Test(.requireSDKs(.macOS), .requireClangFeatures(.printHeadersDirectPerFile), arguments: [false, true]) func validateModuleDependenciesMixedSource(downgradeErrors: Bool) async throws { try await withTemporaryDirectory { tmpDir async throws -> Void in let testWorkspace = try await TestWorkspace( @@ -719,7 +719,7 @@ fileprivate struct DependencyValidationTests: CoreBasedTests { } } - @Test(.requireSDKs(.host), .requireClangFeatures(.printHeadersDirectPerFile), .skipHostOS(.windows, "toolchain too old"), .skipHostOS(.linux, "toolchain too old")) + @Test(.requireSDKs(.macOS), .requireClangFeatures(.printHeadersDirectPerFile)) func validateUnusedModuleDependencies() async throws { try await withTemporaryDirectory { tmpDir in let testWorkspace = try await TestWorkspace(