[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

src/darwin/Framework/CHIPTests/MTRPairingTests.m

@@ -1357,7 +1357,7 @@ - (void)test021_CommissioningOperationStopAfterPASEDoesNotFireWatchdog
     // to fire and double-notify -- it must NOT.
     XCTestExpectation * watchdogShouldNotFire = [self expectationWithDescription:@"Watchdog must not fire after stop"];
     watchdogShouldNotFire.inverted = YES;
-    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.5 * NSEC_PER_SEC)), callbackQueue, ^{
+    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.5 * NSEC_PER_SEC)), callbackQueue, ^{
         // If the watchdog had fired, stopExpectation would already have
         // tripped its over-fulfill check.  Keep this inverted expectation
         // unsatisfied so we time it out cleanly.
@@ -2586,10 +2586,10 @@ - (void)test036_StopWithArmedWatchdogAndDeadControllerStillCancelsWatchdog
     // that the slot was indeed cleared by -stop and not still live.
     XCTestExpectation * waitForNoFire = [self expectationWithDescription:@"brief wait for spurious fire"];
     waitForNoFire.inverted = YES;
-    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.3 * NSEC_PER_SEC)), callbackQueue, ^{
-        // No-op block; the inverted expectation times out cleanly if
-        // nothing fires.
-    });
+    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.3 * NSEC_PER_SEC)), callbackQueue, ^ {
+                                                                                         // No-op block; the inverted expectation times out cleanly if
+                                                                                         // nothing fires.
+                                                                                     });
     [self waitForExpectations:@[ waitForNoFire ] timeout:0.5];
 
     XCTAssertEqual(delegate.failureCallCount, 0u,
@@ -2863,13 +2863,13 @@ - (void)test039_PostPASEWatchdogIsOneShotAndDoesNotAutoRearmAfterFire
     // cancel correct even if it raced with a pending fire.
     XCTestExpectation * noSecondFire = [self expectationWithDescription:@"watchdog must not auto-rearm"];
     noSecondFire.inverted = YES;
-    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.5 * NSEC_PER_SEC)), callbackQueue, ^{
-        // No-op; if the dispatch_source spuriously re-fires, the failure
-        // callback would land on the (already-nilled) delegate, but the
-        // failure count would still increment -- which we check below.
-        // The inverted expectation here just gives the dispatch system
-        // time to deliver any latent timer event before we read the count.
-    });
+    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.5 * NSEC_PER_SEC)), callbackQueue, ^ {
+                                                                                         // No-op; if the dispatch_source spuriously re-fires, the failure
+                                                                                         // callback would land on the (already-nilled) delegate, but the
+                                                                                         // failure count would still increment -- which we check below.
+                                                                                         // The inverted expectation here just gives the dispatch system
+                                                                                         // time to deliver any latent timer event before we read the count.
+                                                                                     });
     [self waitForExpectations:@[ noSecondFire ] timeout:0.7];
 
     XCTAssertEqual(delegate.failureCallCount, 1u,
@@ -3267,14 +3267,12 @@ - (void)test043_LateFireGuardSuppressesWatchdogAfterClientAdvancePastPASE
     commissioning.isWaitingAfterPASEEstablished = YES;
 
     // Flush the setter's self-bounce.
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     XCTAssertTrue(commissioning.isWaitingAfterPASEEstablished);
 
     // Client advances: setter -> NO triggers _cancelPostPASEWatchdog.
     commissioning.isWaitingAfterPASEEstablished = NO;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     XCTAssertFalse(commissioning.isWaitingAfterPASEEstablished);
 
     // Now an in-flight late timer block (had it been posted to the
@@ -3325,8 +3323,7 @@ - (void)test044_ArmAndIsWaitingAreSerializedOnDelegateQueue
 
     // Off-queue setter call must self-bounce; flush by syncing the queue.
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     XCTAssertTrue(commissioning.isWaitingAfterPASEEstablished,
         @"setter must take effect after self-bounce drained");
 
@@ -3336,11 +3333,9 @@ - (void)test044_ArmAndIsWaitingAreSerializedOnDelegateQueue
     // produces a single live source (no leaks) -- exercised structurally
     // by ensuring no exceptions/asserts trip.
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     commissioning.isWaitingAfterPASEEstablished = NO;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     XCTAssertFalse(commissioning.isWaitingAfterPASEEstablished);
 }
 
@@ -3378,17 +3373,15 @@ - (void)test045_FirePostPASEWatchdogStopsCommissioningOnNonInternalPath
         [commissioning _armPostPASEWatchdog];
     });
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     dispatch_sync(callbackQueue, ^{
         [commissioning _firePostPASEWatchdog];
     });
 
     // Drain the delegate-queue dispatch_async that _dispatchCommissioningError
     // posts to deliver the failure callback.
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     // Even with a nil _controller (no live MTRDeviceController_Concrete
     // attached), _firePostPASEWatchdog must still route the timeout
@@ -3449,8 +3442,7 @@ - (void)test_WatchdogCancelledOnCommissionNodeWithID
         XCTAssertTrue([commissioning _armPostPASEWatchdog]);
     });
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     // 500ms after PASE-complete, the delegate calls commissionNodeWithID:
     // (in production this is what the client app does to advance the
@@ -3510,8 +3502,7 @@ - (void)test_WatchdogCancelledOnCancelCommissioning
         XCTAssertTrue([commissioning _armPostPASEWatchdog]);
     });
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     // From paseSessionEstablishmentComplete: -- which on a real
     // controller would mean "client decided to bail" -- simulate the
@@ -3596,8 +3587,7 @@ - (void)test_WatchdogCancelledOnControllerInvalidate
             XCTAssertTrue([commissioning _armPostPASEWatchdog]);
         });
         commissioning.isWaitingAfterPASEEstablished = YES;
-        dispatch_sync(callbackQueue, ^{
-        });
+        dispatch_sync(callbackQueue, ^ {});
         XCTAssertTrue(commissioning.isWaitingAfterPASEEstablished);
 
         // "Invalidate" the controller -- structurally, drop the
@@ -3674,8 +3664,7 @@ - (void)test_WatchdogIdempotentArm
     // Enter the post-PASE waiting state so the late-fire guard inside
     // the timer's event_handler does not suppress the production fire.
     commissioning.isWaitingAfterPASEEstablished = YES;
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     // Let the single armed source fire naturally (~2s).  If the second
     // arm had silently leaked a second source, both would fire here
@@ -3887,8 +3876,7 @@ - (void)test_WatchdogDoesNotFire_OnNormalFlow
     // protects us from a late-fire after success.  If isWaitingAfterPASEEstablished
     // were left stuck at YES, the late-fire guard inside the dispatch
     // event_handler would not suppress a leaked timer's fire.
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
     XCTAssertFalse(commissioning.isWaitingAfterPASEEstablished,
         @"after successful commissionNodeWithID:, isWaitingAfterPASEEstablished must be NO and the watchdog must be torn down");
 
@@ -4036,8 +4024,7 @@ - (void)test_WatchdogArmFailure_FaultInjectionIsOneShot
     // by the time we observe the arm-call results below, the flag write
     // has already been seen.
     [commissioning setForceNextArmFailureForTesting:YES];
-    dispatch_sync(callbackQueue, ^{
-    });
+    dispatch_sync(callbackQueue, ^ {});
 
     // First arm: must observe the injected failure and clear the flag.
     __block BOOL firstArm = YES;