Unit test fix for TSAN

jtung-apple · Sep 9, 2024 · ba1a330 · ba1a330
1 parent 54071bb
commit ba1a330
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/darwin/Framework/CHIPTests/MTRPairingTests.m b/src/darwin/Framework/CHIPTests/MTRPairingTests.m
@@ -134,10 +134,10 @@ - (void)controller:(MTRDeviceController *)controller commissioningComplete:(NSEr
 @end
 
 @interface MTRPairingTestMonitoringControllerDelegate : NSObject <MTRDeviceControllerDelegate>
-@property (nonatomic, readonly) BOOL statusUpdateCalled;
-@property (nonatomic, readonly) BOOL commissioningSessionEstablishmentDoneCalled;
-@property (nonatomic, readonly) BOOL commissioningCompleteCalled;
-@property (nonatomic, readonly) BOOL readCommissioningInfoCalled;
+@property (atomic, readwrite) BOOL statusUpdateCalled;
+@property (atomic, readwrite) BOOL commissioningSessionEstablishmentDoneCalled;
+@property (atomic, readwrite) BOOL commissioningCompleteCalled;
+@property (atomic, readwrite) BOOL readCommissioningInfoCalled;
 @end
 
 @implementation MTRPairingTestMonitoringControllerDelegate
@@ -147,25 +147,25 @@ - (NSString *)description
 }
 - (void)controller:(MTRDeviceController *)controller statusUpdate:(MTRCommissioningStatus)status
 {
-    _statusUpdateCalled = YES;
+    self.statusUpdateCalled = YES;
 }
 
 - (void)controller:(MTRDeviceController *)controller commissioningSessionEstablishmentDone:(NSError * _Nullable)error
 {
-    _commissioningSessionEstablishmentDoneCalled = YES;
+    self.commissioningSessionEstablishmentDoneCalled = YES;
 }
 
 - (void)controller:(MTRDeviceController *)controller
     commissioningComplete:(NSError * _Nullable)error
                    nodeID:(NSNumber * _Nullable)nodeID
                   metrics:(MTRMetrics *)metrics
 {
-    _commissioningCompleteCalled = YES;
+    self.commissioningCompleteCalled = YES;
 }
 
 - (void)controller:(MTRDeviceController *)controller readCommissioningInfo:(MTRProductIdentity *)info
 {
-    _readCommissioningInfoCalled = YES;
+    self.readCommissioningInfoCalled = YES;
 }
 @end