diff --git a/Sources/XCLinting/Rules/EmbeddedBuildSettingsRule.swift b/Sources/XCLinting/Rules/EmbeddedBuildSettingsRule.swift index 7a01aef..5df45dc 100644 --- a/Sources/XCLinting/Rules/EmbeddedBuildSettingsRule.swift +++ b/Sources/XCLinting/Rules/EmbeddedBuildSettingsRule.swift @@ -1,23 +1,29 @@ import Foundation -func embeddedBuildSettingsRule(_ environment: XCLinter.Environment) -> [Violation] { - var violations = [Violation]() +import XcodeProj - for target in environment.project.pbxproj.legacyTargets { - for config in target.buildConfigurationList?.buildConfigurations ?? [] { - if config.buildSettings.isEmpty == false { - violations.append(.init("found some settings for \(target.name), \(config.name)")) +struct EmbeddedBuildSettingsRule { + func run(_ environment: XCLinter.Environment) throws -> [Violation] { + var violations = [Violation]() + + // check top-level + for project in environment.project.pbxproj.projects { + for config in project.buildConfigurationList?.buildConfigurations ?? [] { + if config.buildSettings.isEmpty == false { + violations.append(.init("found settings for project \(project.name), \(config.name)")) + } } } - } - for target in environment.project.pbxproj.nativeTargets { - for config in target.buildConfigurationList?.buildConfigurations ?? [] { - if config.buildSettings.isEmpty == false { - violations.append(.init("found some settings for \(target.name), \(config.name)")) + // check targets + environment.project.pbxproj.enumerateBuildConfigurations { name, configList in + for config in configList.buildConfigurations { + if config.buildSettings.isEmpty == false { + violations.append(.init("found settings for target \(name), \(config.name)")) + } } } - } - return violations + return violations + } } diff --git a/Sources/XCLinting/XCLinter.swift b/Sources/XCLinting/XCLinter.swift index 55d2c66..cff0899 100644 --- a/Sources/XCLinting/XCLinter.swift +++ b/Sources/XCLinting/XCLinter.swift @@ -74,7 +74,7 @@ extension XCLinter { public static let ruleIdentifiers: Set = Set(ruleMap.keys) public static let ruleMap: [String: Rule] = [ - "embedded_build_setting": embeddedBuildSettingsRule, + "embedded_build_setting": { try EmbeddedBuildSettingsRule().run($0) }, "build_files_ordered": { try BuildFilesAreOrderedRule().run($0) }, "groups_sorted": groupsAreSortedRule, "validate_build_settings": { try ValidateBuildSettingsRule().run($0) }, diff --git a/Tests/XCLintTests/EmbeddedBuildSettingsRuleTests.swift b/Tests/XCLintTests/EmbeddedBuildSettingsRuleTests.swift index 5ce0f2c..4660be1 100644 --- a/Tests/XCLintTests/EmbeddedBuildSettingsRuleTests.swift +++ b/Tests/XCLintTests/EmbeddedBuildSettingsRuleTests.swift @@ -15,7 +15,23 @@ final class EmbeddedBuildSettingsRuleTests: XCTestCase { configuration: Configuration() ) - let violations = embeddedBuildSettingsRule(env) + let violations = try EmbeddedBuildSettingsRule().run(env) + + XCTAssertFalse(violations.isEmpty) + } + + func testProjectWithProjectLevelBuildSettingsOnly() throws { + let url = try Bundle.module.testDataURL(named: "ProjectOnlyBuildSettings.xcodeproj") + + let project = try XcodeProj(pathString: url.path) + + let env = XCLinter.Environment( + project: project, + projectRootURL: url, + configuration: Configuration() + ) + + let violations = try EmbeddedBuildSettingsRule().run(env) XCTAssertFalse(violations.isEmpty) } @@ -31,7 +47,7 @@ final class EmbeddedBuildSettingsRuleTests: XCTestCase { configuration: Configuration() ) - let violations = embeddedBuildSettingsRule(env) + let violations = try EmbeddedBuildSettingsRule().run(env) XCTAssertTrue(violations.isEmpty) } diff --git a/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.pbxproj b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.pbxproj new file mode 100644 index 0000000..5c94a9e --- /dev/null +++ b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.pbxproj @@ -0,0 +1,200 @@ +// !$*UTF8*$! +{ + archiveVersion = 1; + classes = { + }; + objectVersion = 56; + objects = { + +/* Begin PBXBuildFile section */ + C965BD2C2AE6E5D700E5836A /* StockMacOSAppApp.swift in Sources */ = {isa = PBXBuildFile; fileRef = C965BD2B2AE6E5D700E5836A /* StockMacOSAppApp.swift */; }; + C965BD2E2AE6E5D700E5836A /* ContentView.swift in Sources */ = {isa = PBXBuildFile; fileRef = C965BD2D2AE6E5D700E5836A /* ContentView.swift */; }; + C965BD302AE6E5D800E5836A /* Assets.xcassets in Resources */ = {isa = PBXBuildFile; fileRef = C965BD2F2AE6E5D800E5836A /* Assets.xcassets */; }; + C965BD332AE6E5D800E5836A /* Preview Assets.xcassets in Resources */ = {isa = PBXBuildFile; fileRef = C965BD322AE6E5D800E5836A /* Preview Assets.xcassets */; }; +/* End PBXBuildFile section */ + +/* Begin PBXFileReference section */ + C965BD282AE6E5D700E5836A /* .app */ = {isa = PBXFileReference; explicitFileType = wrapper.application; includeInIndex = 0; path = .app; sourceTree = BUILT_PRODUCTS_DIR; }; + C965BD2B2AE6E5D700E5836A /* StockMacOSAppApp.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = StockMacOSAppApp.swift; sourceTree = ""; }; + C965BD2D2AE6E5D700E5836A /* ContentView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ContentView.swift; sourceTree = ""; }; + C965BD2F2AE6E5D800E5836A /* Assets.xcassets */ = {isa = PBXFileReference; lastKnownFileType = folder.assetcatalog; path = Assets.xcassets; sourceTree = ""; }; + C965BD322AE6E5D800E5836A /* Preview Assets.xcassets */ = {isa = PBXFileReference; lastKnownFileType = folder.assetcatalog; path = "Preview Assets.xcassets"; sourceTree = ""; }; + C965BD342AE6E5D800E5836A /* StockMacOSApp.entitlements */ = {isa = PBXFileReference; lastKnownFileType = text.plist.entitlements; path = StockMacOSApp.entitlements; sourceTree = ""; }; +/* End PBXFileReference section */ + +/* Begin PBXFrameworksBuildPhase section */ + C965BD252AE6E5D700E5836A /* Frameworks */ = { + isa = PBXFrameworksBuildPhase; + buildActionMask = 2147483647; + files = ( + ); + runOnlyForDeploymentPostprocessing = 0; + }; +/* End PBXFrameworksBuildPhase section */ + +/* Begin PBXGroup section */ + C965BD1F2AE6E5D700E5836A = { + isa = PBXGroup; + children = ( + C965BD2A2AE6E5D700E5836A /* StockMacOSApp */, + C965BD292AE6E5D700E5836A /* Products */, + ); + sourceTree = ""; + }; + C965BD292AE6E5D700E5836A /* Products */ = { + isa = PBXGroup; + children = ( + C965BD282AE6E5D700E5836A /* .app */, + ); + name = Products; + sourceTree = ""; + }; + C965BD2A2AE6E5D700E5836A /* StockMacOSApp */ = { + isa = PBXGroup; + children = ( + C965BD2B2AE6E5D700E5836A /* StockMacOSAppApp.swift */, + C965BD2D2AE6E5D700E5836A /* ContentView.swift */, + C965BD2F2AE6E5D800E5836A /* Assets.xcassets */, + C965BD342AE6E5D800E5836A /* StockMacOSApp.entitlements */, + C965BD312AE6E5D800E5836A /* Preview Content */, + ); + path = StockMacOSApp; + sourceTree = ""; + }; + C965BD312AE6E5D800E5836A /* Preview Content */ = { + isa = PBXGroup; + children = ( + C965BD322AE6E5D800E5836A /* Preview Assets.xcassets */, + ); + path = "Preview Content"; + sourceTree = ""; + }; +/* End PBXGroup section */ + +/* Begin PBXNativeTarget section */ + C965BD272AE6E5D700E5836A /* StockMacOSApp */ = { + isa = PBXNativeTarget; + buildConfigurationList = C965BD372AE6E5D800E5836A /* Build configuration list for PBXNativeTarget "StockMacOSApp" */; + buildPhases = ( + C965BD242AE6E5D700E5836A /* Sources */, + C965BD252AE6E5D700E5836A /* Frameworks */, + C965BD262AE6E5D700E5836A /* Resources */, + ); + buildRules = ( + ); + dependencies = ( + ); + name = StockMacOSApp; + productName = StockMacOSApp; + productReference = C965BD282AE6E5D700E5836A /* .app */; + productType = "com.apple.product-type.application"; + }; +/* End PBXNativeTarget section */ + +/* Begin PBXProject section */ + C965BD202AE6E5D700E5836A /* Project object */ = { + isa = PBXProject; + attributes = { + BuildIndependentTargetsInParallel = 1; + LastSwiftUpdateCheck = 1510; + LastUpgradeCheck = 1510; + TargetAttributes = { + C965BD272AE6E5D700E5836A = { + CreatedOnToolsVersion = 15.1; + }; + }; + }; + buildConfigurationList = C965BD232AE6E5D700E5836A /* Build configuration list for PBXProject "ProjectOnlyBuildSettings" */; + compatibilityVersion = "Xcode 14.0"; + developmentRegion = en; + hasScannedForEncodings = 0; + knownRegions = ( + en, + Base, + ); + mainGroup = C965BD1F2AE6E5D700E5836A; + productRefGroup = C965BD292AE6E5D700E5836A /* Products */; + projectDirPath = ""; + projectRoot = ""; + targets = ( + C965BD272AE6E5D700E5836A /* StockMacOSApp */, + ); + }; +/* End PBXProject section */ + +/* Begin PBXResourcesBuildPhase section */ + C965BD262AE6E5D700E5836A /* Resources */ = { + isa = PBXResourcesBuildPhase; + buildActionMask = 2147483647; + files = ( + C965BD332AE6E5D800E5836A /* Preview Assets.xcassets in Resources */, + C965BD302AE6E5D800E5836A /* Assets.xcassets in Resources */, + ); + runOnlyForDeploymentPostprocessing = 0; + }; +/* End PBXResourcesBuildPhase section */ + +/* Begin PBXSourcesBuildPhase section */ + C965BD242AE6E5D700E5836A /* Sources */ = { + isa = PBXSourcesBuildPhase; + buildActionMask = 2147483647; + files = ( + C965BD2E2AE6E5D700E5836A /* ContentView.swift in Sources */, + C965BD2C2AE6E5D700E5836A /* StockMacOSAppApp.swift in Sources */, + ); + runOnlyForDeploymentPostprocessing = 0; + }; +/* End PBXSourcesBuildPhase section */ + +/* Begin XCBuildConfiguration section */ + C965BD352AE6E5D800E5836A /* Debug */ = { + isa = XCBuildConfiguration; + buildSettings = { + ONLY_ACTIVE_ARCH = YES; + }; + name = Debug; + }; + C965BD362AE6E5D800E5836A /* Release */ = { + isa = XCBuildConfiguration; + buildSettings = { + ONLY_ACTIVE_ARCH = YES; + }; + name = Release; + }; + C965BD382AE6E5D800E5836A /* Debug */ = { + isa = XCBuildConfiguration; + buildSettings = { + }; + name = Debug; + }; + C965BD392AE6E5D800E5836A /* Release */ = { + isa = XCBuildConfiguration; + buildSettings = { + }; + name = Release; + }; +/* End XCBuildConfiguration section */ + +/* Begin XCConfigurationList section */ + C965BD232AE6E5D700E5836A /* Build configuration list for PBXProject "ProjectOnlyBuildSettings" */ = { + isa = XCConfigurationList; + buildConfigurations = ( + C965BD352AE6E5D800E5836A /* Debug */, + C965BD362AE6E5D800E5836A /* Release */, + ); + defaultConfigurationIsVisible = 0; + defaultConfigurationName = Release; + }; + C965BD372AE6E5D800E5836A /* Build configuration list for PBXNativeTarget "StockMacOSApp" */ = { + isa = XCConfigurationList; + buildConfigurations = ( + C965BD382AE6E5D800E5836A /* Debug */, + C965BD392AE6E5D800E5836A /* Release */, + ); + defaultConfigurationIsVisible = 0; + defaultConfigurationName = Release; + }; +/* End XCConfigurationList section */ + }; + rootObject = C965BD202AE6E5D700E5836A /* Project object */; +} diff --git a/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/contents.xcworkspacedata b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/contents.xcworkspacedata new file mode 100644 index 0000000..919434a --- /dev/null +++ b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/contents.xcworkspacedata @@ -0,0 +1,7 @@ + + + + + diff --git a/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist new file mode 100644 index 0000000..18d9810 --- /dev/null +++ b/Tests/XCLintTests/TestData/ProjectOnlyBuildSettings.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist @@ -0,0 +1,8 @@ + + + + + IDEDidComputeMac32BitWarning + + +