Refactor IfConfigDecl handling and add tests for OR conditions and nested conditionals

- Move ifConfig processing inline in mockMemberBlock to match structure of other declaration types
- Add unit tests for OR conditions (#if DEBUG || TESTFLIGHT)
- Add unit tests for nested #if conditionals

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: 2 people

Sources/MockingMacros/Macros/MockedMacro/MockedMacro.swift

@@ -328,14 +328,6 @@ extension MockedMacro {
             ofType: IfConfigDeclSyntax.self
         )
 
-        let mockedIfConfigDeclarations: [IfConfigDeclSyntax] = try ifConfigDeclarations.compactMap {
-            try self.mockIfConfigDeclaration(
-                from: $0,
-                with: accessLevel,
-                in: protocolDeclaration
-            )
-        }
-
         return try MemberBlockSyntax {
             for initializerDeclaration in initializerDeclarations {
                 try self.mockInitializerConformanceDeclaration(
@@ -362,8 +354,14 @@ extension MockedMacro {
                 )
             }
 
-            for mockedIfConfig in mockedIfConfigDeclarations {
-                mockedIfConfig
+            for ifConfigDeclaration in ifConfigDeclarations {
+                if let mockedIfConfig = try self.mockIfConfigDeclaration(
+                    from: ifConfigDeclaration,
+                    with: accessLevel,
+                    in: protocolDeclaration
+                ) {
+                    mockedIfConfig
+                }
             }
         }
     }
@@ -703,7 +701,7 @@ extension MockedMacro {
                         MemberBlockItemSyntax(decl: mockedIfConfig)
                     }
                 }
-                // AssociatedTypeDeclSyntax is intentionally skipped here since
+                // AssociatedTypeDeclSyntax is intentionally skipped since
                 // associated types become generic parameters on the mock class.
             }
         }

Tests/MockingMacrosTests/Macros/MockedMacro/Mocked_IfConfigDeclTests.swift

@@ -230,5 +230,69 @@ struct Mocked_IfConfigDeclTests {
             """
         )
     }
+
+    // MARK: OR Condition Tests
+
+    @Test(arguments: mockedTestConfigurations)
+    func orCondition(
+        interface: InterfaceConfiguration,
+        mock: MockConfiguration
+    ) {
+        assertMocked(
+            """
+            \(interface.accessLevel) protocol Dependency {
+                func commonMethod()
+                #if DEBUG || TESTFLIGHT
+                func debugOrTestFlightMethod()
+                #endif
+            }
+            """,
+            generates: """
+            #if SWIFT_MOCKING_ENABLED
+            @MockedMembers
+            \(mock.modifiers)class DependencyMock: Dependency {
+                \(mock.memberModifiers)func commonMethod()
+                #if DEBUG || TESTFLIGHT
+                \(mock.memberModifiers)func debugOrTestFlightMethod()
+                #endif
+            }
+            #endif
+            """
+        )
+    }
+
+    // MARK: Nested #if Conditional Tests
+
+    @Test(arguments: mockedTestConfigurations)
+    func nestedIfConditionals(
+        interface: InterfaceConfiguration,
+        mock: MockConfiguration
+    ) {
+        assertMocked(
+            """
+            \(interface.accessLevel) protocol Dependency {
+                #if DEBUG
+                func debugMethod()
+                #if os(iOS)
+                func debugiOSOnlyMethod()
+                #endif
+                #endif
+            }
+            """,
+            generates: """
+            #if SWIFT_MOCKING_ENABLED
+            @MockedMembers
+            \(mock.modifiers)class DependencyMock: Dependency {
+                #if DEBUG
+                \(mock.memberModifiers)func debugMethod()
+                #if os(iOS)
+                \(mock.memberModifiers)func debugiOSOnlyMethod()
+                #endif
+                #endif
+            }
+            #endif
+            """
+        )
+    }
 }
 #endif