Skip to content

Commit 7a8a038

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent d86cd38 commit 7a8a038

1 file changed

Lines changed: 19 additions & 26 deletions

File tree

src/darwin/Framework/CHIPTests/MTRPairingTests.m

Lines changed: 19 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1335,7 +1335,7 @@ - (void)test021_CommissioningOperationStopAfterPASEDoesNotFireWatchdog
13351335
// to fire and double-notify -- it must NOT.
13361336
XCTestExpectation * watchdogShouldNotFire = [self expectationWithDescription:@"Watchdog must not fire after stop"];
13371337
watchdogShouldNotFire.inverted = YES;
1338-
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.5 * NSEC_PER_SEC)), callbackQueue, ^{
1338+
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.5 * NSEC_PER_SEC)), callbackQueue, ^{
13391339
// If the watchdog had fired, stopExpectation would already have
13401340
// tripped its over-fulfill check. Keep this inverted expectation
13411341
// unsatisfied so we time it out cleanly.
@@ -2564,10 +2564,10 @@ - (void)test036_StopWithArmedWatchdogAndDeadControllerStillCancelsWatchdog
25642564
// that the slot was indeed cleared by -stop and not still live.
25652565
XCTestExpectation * waitForNoFire = [self expectationWithDescription:@"brief wait for spurious fire"];
25662566
waitForNoFire.inverted = YES;
2567-
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.3 * NSEC_PER_SEC)), callbackQueue, ^{
2568-
// No-op block; the inverted expectation times out cleanly if
2569-
// nothing fires.
2570-
});
2567+
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.3 * NSEC_PER_SEC)), callbackQueue, ^ {
2568+
// No-op block; the inverted expectation times out cleanly if
2569+
// nothing fires.
2570+
});
25712571
[self waitForExpectations:@[ waitForNoFire ] timeout:0.5];
25722572

25732573
XCTAssertEqual(delegate.failureCallCount, 0u,
@@ -2841,13 +2841,13 @@ - (void)test039_PostPASEWatchdogIsOneShotAndDoesNotAutoRearmAfterFire
28412841
// cancel correct even if it raced with a pending fire.
28422842
XCTestExpectation * noSecondFire = [self expectationWithDescription:@"watchdog must not auto-rearm"];
28432843
noSecondFire.inverted = YES;
2844-
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.5 * NSEC_PER_SEC)), callbackQueue, ^{
2845-
// No-op; if the dispatch_source spuriously re-fires, the failure
2846-
// callback would land on the (already-nilled) delegate, but the
2847-
// failure count would still increment -- which we check below.
2848-
// The inverted expectation here just gives the dispatch system
2849-
// time to deliver any latent timer event before we read the count.
2850-
});
2844+
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.5 * NSEC_PER_SEC)), callbackQueue, ^ {
2845+
// No-op; if the dispatch_source spuriously re-fires, the failure
2846+
// callback would land on the (already-nilled) delegate, but the
2847+
// failure count would still increment -- which we check below.
2848+
// The inverted expectation here just gives the dispatch system
2849+
// time to deliver any latent timer event before we read the count.
2850+
});
28512851
[self waitForExpectations:@[ noSecondFire ] timeout:0.7];
28522852

28532853
XCTAssertEqual(delegate.failureCallCount, 1u,
@@ -3245,14 +3245,12 @@ - (void)test043_LateFireGuardSuppressesWatchdogAfterClientAdvancePastPASE
32453245
commissioning.isWaitingAfterPASEEstablished = YES;
32463246

32473247
// Flush the setter's self-bounce.
3248-
dispatch_sync(callbackQueue, ^{
3249-
});
3248+
dispatch_sync(callbackQueue, ^ {});
32503249
XCTAssertTrue(commissioning.isWaitingAfterPASEEstablished);
32513250

32523251
// Client advances: setter -> NO triggers _cancelPostPASEWatchdog.
32533252
commissioning.isWaitingAfterPASEEstablished = NO;
3254-
dispatch_sync(callbackQueue, ^{
3255-
});
3253+
dispatch_sync(callbackQueue, ^ {});
32563254
XCTAssertFalse(commissioning.isWaitingAfterPASEEstablished);
32573255

32583256
// Now an in-flight late timer block (had it been posted to the
@@ -3303,8 +3301,7 @@ - (void)test044_ArmAndIsWaitingAreSerializedOnDelegateQueue
33033301

33043302
// Off-queue setter call must self-bounce; flush by syncing the queue.
33053303
commissioning.isWaitingAfterPASEEstablished = YES;
3306-
dispatch_sync(callbackQueue, ^{
3307-
});
3304+
dispatch_sync(callbackQueue, ^ {});
33083305
XCTAssertTrue(commissioning.isWaitingAfterPASEEstablished,
33093306
@"setter must take effect after self-bounce drained");
33103307

@@ -3314,11 +3311,9 @@ - (void)test044_ArmAndIsWaitingAreSerializedOnDelegateQueue
33143311
// produces a single live source (no leaks) -- exercised structurally
33153312
// by ensuring no exceptions/asserts trip.
33163313
commissioning.isWaitingAfterPASEEstablished = YES;
3317-
dispatch_sync(callbackQueue, ^{
3318-
});
3314+
dispatch_sync(callbackQueue, ^ {});
33193315
commissioning.isWaitingAfterPASEEstablished = NO;
3320-
dispatch_sync(callbackQueue, ^{
3321-
});
3316+
dispatch_sync(callbackQueue, ^ {});
33223317
XCTAssertFalse(commissioning.isWaitingAfterPASEEstablished);
33233318
}
33243319

@@ -3356,17 +3351,15 @@ - (void)test045_FirePostPASEWatchdogStopsCommissioningOnNonInternalPath
33563351
[commissioning _armPostPASEWatchdog];
33573352
});
33583353
commissioning.isWaitingAfterPASEEstablished = YES;
3359-
dispatch_sync(callbackQueue, ^{
3360-
});
3354+
dispatch_sync(callbackQueue, ^ {});
33613355

33623356
dispatch_sync(callbackQueue, ^{
33633357
[commissioning _firePostPASEWatchdog];
33643358
});
33653359

33663360
// Drain the delegate-queue dispatch_async that _dispatchCommissioningError
33673361
// posts to deliver the failure callback.
3368-
dispatch_sync(callbackQueue, ^{
3369-
});
3362+
dispatch_sync(callbackQueue, ^ {});
33703363

33713364
// Even with a nil _controller (no live MTRDeviceController_Concrete
33723365
// attached), _firePostPASEWatchdog must still route the timeout

0 commit comments

Comments
 (0)