Rename expectedResult to requiredResponse in the invokeCommands API. (#37416)

bzbarsky-apple · web-flow · commit deff77a2bb49 · 2025-02-06T03:41:49.000Z
The old naming was too confusing, and this more accurately represents what is
actually going on.
diff --git a/src/darwin/Framework/CHIP/MTRCommandWithRequiredResponse.h b/src/darwin/Framework/CHIP/MTRCommandWithRequiredResponse.h
@@ -20,11 +20,11 @@
 NS_ASSUME_NONNULL_BEGIN
 
 /**
- * An object representing a single command to be invoked and the expected
- * result of invoking it.
+ * An object representing a single command to be invoked and the response
+ * required for the invoke to be considered successful.
  */
 MTR_AVAILABLE(ios(18.4), macos(15.4), watchos(11.4), tvos(18.4))
-@interface MTRCommandWithExpectedResult : NSObject <NSCopying, NSSecureCoding>
+@interface MTRCommandWithRequiredResponse : NSObject <NSCopying, NSSecureCoding>
 
 /**
  * The path of the command being invoked.
@@ -39,22 +39,22 @@ MTR_AVAILABLE(ios(18.4), macos(15.4), watchos(11.4), tvos(18.4))
 @property (nonatomic, retain, nullable) NSDictionary<NSString *, id> * commandFields;
 
 /**
- * The expected result of invoking the command.
+ * The response that represents this command succeeding.
  *
  * If this is nil, that indicates that the invoke is considered successful if it
  * does not result in an error status response.
  *
- * If this is is not nil, then invoke is considered successful if
- * it results in a data response and for each entry in the provided
- * expectedResult the field whose field ID matches the key of the entry has a
+ * If this is is not nil, then the invoke is considered successful if
+ * the response is a data response and for each entry in the provided
+ * requiredResponse the field whose field ID matches the key of the entry has a
  * value that equals the value of the entry.  Values of entries are data-value
  * dictionaries.
  */
-@property (nonatomic, copy, nullable) NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> * expectedResult;
+@property (nonatomic, copy, nullable) NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> * requiredResponse;
 
 - (instancetype)initWithPath:(MTRCommandPath *)path
                commandFields:(nullable NSDictionary<NSString *, id> *)commandFields
-              expectedResult:(nullable NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> *)expectedResult;
+            requiredResponse:(nullable NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> *)requiredResponse;
 
 @end
 
diff --git a/src/darwin/Framework/CHIP/MTRCommandWithRequiredResponse.mm b/src/darwin/Framework/CHIP/MTRCommandWithRequiredResponse.mm
@@ -18,35 +18,35 @@
 #import "MTRLogging_Internal.h"
 #import <Matter/Matter.h>
 
-@implementation MTRCommandWithExpectedResult
+@implementation MTRCommandWithRequiredResponse
 - (instancetype)initWithPath:(MTRCommandPath *)path
                commandFields:(nullable NSDictionary<NSString *, id> *)commandFields
-              expectedResult:(nullable NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> *)expectedResult
+            requiredResponse:(nullable NSDictionary<NSNumber *, NSDictionary<NSString *, id> *> *)requiredResponse
 {
     if (self = [super init]) {
         self.path = path;
         self.commandFields = commandFields;
-        self.expectedResult = expectedResult;
+        self.requiredResponse = requiredResponse;
     }
 
     return self;
 }
 
 - (id)copyWithZone:(NSZone *)zone
 {
-    return [[MTRCommandWithExpectedResult alloc] initWithPath:self.path commandFields:self.commandFields expectedResult:self.expectedResult];
+    return [[MTRCommandWithRequiredResponse alloc] initWithPath:self.path commandFields:self.commandFields requiredResponse:self.requiredResponse];
 }
 
 - (NSString *)description
 {
-    return [NSString stringWithFormat:@"<%@: %p, path: %@, fields: %@, expectedResult: %@", NSStringFromClass(self.class), self, self.path, self.commandFields, self.expectedResult];
+    return [NSString stringWithFormat:@"<%@: %p, path: %@, fields: %@, requiredResponse: %@", NSStringFromClass(self.class), self, self.path, self.commandFields, self.requiredResponse];
 }
 
-#pragma mark - MTRCommandWithExpectedResult NSSecureCoding implementation
+#pragma mark - MTRCommandWithRequiredResponse NSSecureCoding implementation
 
 static NSString * const sPathKey = @"pathKey";
 static NSString * const sFieldsKey = @"fieldsKey";
-static NSString * const sExpectedResultKey = @"expectedResultKey";
+static NSString * const sExpectedResultKey = @"requiredResponseKey";
 
 + (BOOL)supportsSecureCoding
 {
@@ -62,38 +62,38 @@ - (nullable instancetype)initWithCoder:(NSCoder *)decoder
 
     _path = [decoder decodeObjectOfClass:MTRCommandPath.class forKey:sPathKey];
     if (!_path || ![_path isKindOfClass:MTRCommandPath.class]) {
-        MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for endpoint, not MTRCommandPath.", _path);
+        MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for endpoint, not MTRCommandPath.", _path);
         return nil;
     }
 
     _commandFields = [decoder decodeObjectOfClass:NSDictionary.class forKey:sFieldsKey];
     if (_commandFields) {
         if (![_commandFields isKindOfClass:NSDictionary.class]) {
-            MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for commandFields, not NSDictionary.", _commandFields);
+            MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for commandFields, not NSDictionary.", _commandFields);
             return nil;
         }
 
         if (!MTRDataValueDictionaryIsWellFormed(_commandFields) || ![MTRStructureValueType isEqual:_commandFields[MTRTypeKey]]) {
-            MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for commandFields, not a structure-typed data-value dictionary.", _commandFields);
+            MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for commandFields, not a structure-typed data-value dictionary.", _commandFields);
             return nil;
         }
     }
 
-    _expectedResult = [decoder decodeObjectOfClass:NSDictionary.class forKey:sExpectedResultKey];
-    if (_expectedResult) {
-        if (![_expectedResult isKindOfClass:NSDictionary.class]) {
-            MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for expectedResult, not NSDictionary.", _expectedResult);
+    _requiredResponse = [decoder decodeObjectOfClass:NSDictionary.class forKey:sExpectedResultKey];
+    if (_requiredResponse) {
+        if (![_requiredResponse isKindOfClass:NSDictionary.class]) {
+            MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for requiredResponse, not NSDictionary.", _requiredResponse);
             return nil;
         }
 
-        for (id key in _expectedResult) {
+        for (id key in _requiredResponse) {
             if (![key isKindOfClass:NSNumber.class]) {
-                MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded key %@ in expectedResult", key);
+                MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded key %@ in requiredResponse", key);
                 return nil;
             }
 
-            if (![_expectedResult[key] isKindOfClass:NSDictionary.class] || !MTRDataValueDictionaryIsWellFormed(_expectedResult[key])) {
-                MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded value %@ for key %@ in expectedResult", _expectedResult[key], key);
+            if (![_requiredResponse[key] isKindOfClass:NSDictionary.class] || !MTRDataValueDictionaryIsWellFormed(_requiredResponse[key])) {
+                MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded value %@ for key %@ in requiredResponse", _requiredResponse[key], key);
                 return nil;
             }
         }
@@ -111,8 +111,8 @@ - (void)encodeWithCoder:(NSCoder *)coder
     if (self.commandFields) {
         [coder encodeObject:self.commandFields forKey:sFieldsKey];
     }
-    if (self.expectedResult) {
-        [coder encodeObject:self.expectedResult forKey:sExpectedResultKey];
+    if (self.requiredResponse) {
+        [coder encodeObject:self.requiredResponse forKey:sExpectedResultKey];
     }
 }
 
diff --git a/src/darwin/Framework/CHIP/MTRDevice.h b/src/darwin/Framework/CHIP/MTRDevice.h
@@ -19,7 +19,7 @@
 #import <Matter/MTRAttributeValueWaiter.h>
 #import <Matter/MTRBaseClusters.h>
 #import <Matter/MTRBaseDevice.h>
-#import <Matter/MTRCommandWithExpectedResult.h>
+#import <Matter/MTRCommandWithRequiredResponse.h>
 #import <Matter/MTRDefines.h>
 
 NS_ASSUME_NONNULL_BEGIN
@@ -316,7 +316,7 @@ MTR_AVAILABLE(ios(16.1), macos(13.0), watchos(9.1), tvos(16.1))
  *    response.  In this case the data-value representing the response will be
  *    the value of this field.
  */
-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)commands
+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse *> *> *)commands
                  queue:(dispatch_queue_t)queue
             completion:(MTRDeviceResponseHandler)completion MTR_AVAILABLE(ios(18.4), macos(15.4), watchos(11.4), tvos(18.4));
 
diff --git a/src/darwin/Framework/CHIP/MTRDevice.mm b/src/darwin/Framework/CHIP/MTRDevice.mm
@@ -523,7 +523,7 @@ - (void)_invokeKnownCommandWithEndpointID:(NSNumber *)endpointID
                             completion:responseHandler];
 }
 
-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)commands
+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse *> *> *)commands
                  queue:(dispatch_queue_t)queue
             completion:(MTRDeviceResponseHandler)completion
 {
diff --git a/src/darwin/Framework/CHIP/MTRDeviceController_XPC.mm b/src/darwin/Framework/CHIP/MTRDeviceController_XPC.mm
@@ -150,10 +150,10 @@ - (NSXPCInterface *)_interfaceForServerProtocol
             argumentIndex:0
                   ofReply:YES];
 
-    // invokeCommands gets handed MTRCommandWithExpectedResult (which includes
+    // invokeCommands gets handed MTRCommandWithRequiredResponse (which includes
     // MTRCommandPath, which is already in allowedClasses).
     [allowedClasses addObjectsFromArray:@[
-        [MTRCommandWithExpectedResult class],
+        [MTRCommandWithRequiredResponse class],
     ]];
     [interface setClasses:allowedClasses
               forSelector:@selector(deviceController:nodeID:invokeCommands:completion:)
diff --git a/src/darwin/Framework/CHIP/MTRDevice_Concrete.mm b/src/darwin/Framework/CHIP/MTRDevice_Concrete.mm
@@ -3306,7 +3306,7 @@ - (void)_invokeCommandWithEndpointID:(NSNumber *)endpointID
                                               commandFields];
 }
 
-- (BOOL)_invokeResponse:(MTRDeviceResponseValueDictionary)response matchesExpectedResult:(NSDictionary<NSNumber *, MTRDeviceDataValueDictionary> *)expectedResult
+- (BOOL)_invokeResponse:(MTRDeviceResponseValueDictionary)response matchesRequiredResponse:(NSDictionary<NSNumber *, MTRDeviceDataValueDictionary> *)requiredResponse
 {
     if (response[MTRDataKey] == nil) {
         MTR_LOG_ERROR("%@ invokeCommands expects a data response for %@ but got no data", self, response[MTRCommandPathKey]);
@@ -3321,7 +3321,7 @@ - (BOOL)_invokeResponse:(MTRDeviceResponseValueDictionary)response matchesExpect
 
     NSArray<NSDictionary<NSString *, id> *> * fields = data[MTRValueKey];
 
-    for (NSNumber * fieldID in expectedResult) {
+    for (NSNumber * fieldID in requiredResponse) {
         // Check that this field is present in the response.
         MTRDeviceDataValueDictionary _Nullable fieldValue = nil;
         for (NSDictionary<NSString *, id> * field in fields) {
@@ -3336,7 +3336,7 @@ - (BOOL)_invokeResponse:(MTRDeviceResponseValueDictionary)response matchesExpect
             return NO;
         }
 
-        auto * expected = expectedResult[fieldID];
+        auto * expected = requiredResponse[fieldID];
         if (![expected isEqual:fieldValue]) {
             MTR_LOG_ERROR("%@ invokeCommands response for %@ field %@ got %@ but expected %@", self, response[MTRCommandPathKey], fieldID, fieldValue, expected);
             return NO;
@@ -3346,7 +3346,7 @@ - (BOOL)_invokeResponse:(MTRDeviceResponseValueDictionary)response matchesExpect
     return YES;
 }
 
-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)commands
+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse *> *> *)commands
                  queue:(dispatch_queue_t)queue
             completion:(MTRDeviceResponseHandler)completion
 {
@@ -3361,10 +3361,10 @@ - (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)com
     // We want to invoke the command groups in order, stopping after failures as needed.  Build up a
     // linked list of groups via chaining the completions, with calls out to the original
     // completion instead of going to the next list item when we want to stop.
-    for (NSArray<MTRCommandWithExpectedResult *> * commandGroup in [commands reverseObjectEnumerator]) {
+    for (NSArray<MTRCommandWithRequiredResponse *> * commandGroup in [commands reverseObjectEnumerator]) {
         // We want to invoke all the commands in the group in order, propagating along the list of
         // current responses.  Build up that linked list of command invokes via chaining the completions.
-        for (MTRCommandWithExpectedResult * command in [commandGroup reverseObjectEnumerator]) {
+        for (MTRCommandWithRequiredResponse * command in [commandGroup reverseObjectEnumerator]) {
             auto commandInvokeBlock = ^(BOOL allSucceededSoFar, NSArray<MTRDeviceResponseValueDictionary> * previousResponses) {
                 [self invokeCommandWithEndpointID:command.path.endpoint
                                         clusterID:command.path.cluster
@@ -3394,7 +3394,7 @@ - (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)com
 
                                            BOOL nextAllSucceeded = allSucceededSoFar;
                                            MTRDeviceResponseValueDictionary response = responses[0];
-                                           if (command.expectedResult != nil && ![self _invokeResponse:response matchesExpectedResult:command.expectedResult]) {
+                                           if (command.requiredResponse != nil && ![self _invokeResponse:response matchesRequiredResponse:command.requiredResponse]) {
                                                nextAllSucceeded = NO;
                                            }
 
diff --git a/src/darwin/Framework/CHIP/MTRDevice_XPC.mm b/src/darwin/Framework/CHIP/MTRDevice_XPC.mm
@@ -459,7 +459,7 @@ - (void)_invokeCommandWithEndpointID:(NSNumber *)endpointID
     }
 }
 
-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)commands
+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse *> *> *)commands
                  queue:(dispatch_queue_t)queue
             completion:(MTRDeviceResponseHandler)completion
 {
diff --git a/src/darwin/Framework/CHIP/Matter.h b/src/darwin/Framework/CHIP/Matter.h
@@ -34,7 +34,7 @@
 #import <Matter/MTRClusterStateCacheContainer.h>
 #import <Matter/MTRClusters.h>
 #import <Matter/MTRCommandPayloadsObjc.h>
-#import <Matter/MTRCommandWithExpectedResult.h>
+#import <Matter/MTRCommandWithRequiredResponse.h>
 #import <Matter/MTRCommissionableBrowserDelegate.h>
 #import <Matter/MTRCommissionableBrowserResult.h>
 #import <Matter/MTRCommissioneeInfo.h>
diff --git a/src/darwin/Framework/CHIP/XPC Protocol/MTRXPCServerProtocol.h b/src/darwin/Framework/CHIP/XPC Protocol/MTRXPCServerProtocol.h
@@ -52,7 +52,7 @@ MTR_AVAILABLE(ios(18.2), macos(15.2), watchos(11.2), tvos(18.2))
  */
 - (oneway void)deviceController:(NSUUID *)controller nodeID:(NSNumber *)nodeID downloadLogOfType:(MTRDiagnosticLogType)type timeout:(NSTimeInterval)timeout completion:(void (^)(NSURL * _Nullable url, NSError * _Nullable error))completion;
 
-- (oneway void)deviceController:(NSUUID *)controller nodeID:(NSNumber *)nodeID invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult *> *> *)commands completion:(MTRDeviceResponseHandler)completion MTR_AVAILABLE(ios(18.4), macos(15.4), watchos(11.4), tvos(18.4));
+- (oneway void)deviceController:(NSUUID *)controller nodeID:(NSNumber *)nodeID invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse *> *> *)commands completion:(MTRDeviceResponseHandler)completion MTR_AVAILABLE(ios(18.4), macos(15.4), watchos(11.4), tvos(18.4));
 
 @end
 
diff --git a/src/darwin/Framework/CHIPTests/MTRDeviceTests.m b/src/darwin/Framework/CHIPTests/MTRDeviceTests.m
diff --git a/src/darwin/Framework/Matter.xcodeproj/project.pbxproj b/src/darwin/Framework/Matter.xcodeproj/project.pbxproj

Original file line number	Diff line number	Diff line change
`@@ -18,35 +18,35 @@`
`18`	`18`	`#import "MTRLogging_Internal.h"`
`19`	`19`	`#import <Matter/Matter.h>`
`20`	`20`
`21`		`-@implementation MTRCommandWithExpectedResult`
	`21`	`+@implementation MTRCommandWithRequiredResponse`
`22`	`22`	`- (instancetype)initWithPath:(MTRCommandPath *)path`
`23`	`23`	`commandFields:(nullable NSDictionary<NSString , id> )commandFields`
`24`		`- expectedResult:(nullable NSDictionary<NSNumber , NSDictionary<NSString , id> > )expectedResult`
	`24`	`+ requiredResponse:(nullable NSDictionary<NSNumber , NSDictionary<NSString , id> > )requiredResponse`
`25`	`25`	`{`
`26`	`26`	`if (self = [super init]) {`
`27`	`27`	`self.path = path;`
`28`	`28`	`self.commandFields = commandFields;`
`29`		`- self.expectedResult = expectedResult;`
	`29`	`+ self.requiredResponse = requiredResponse;`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`return self;`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`- (id)copyWithZone:(NSZone *)zone`
`36`	`36`	`{`
`37`		`- return [[MTRCommandWithExpectedResult alloc] initWithPath:self.path commandFields:self.commandFields expectedResult:self.expectedResult];`
	`37`	`+ return [[MTRCommandWithRequiredResponse alloc] initWithPath:self.path commandFields:self.commandFields requiredResponse:self.requiredResponse];`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`- (NSString *)description`
`41`	`41`	`{`
`42`		`- return [NSString stringWithFormat:@"<%@: %p, path: %@, fields: %@, expectedResult: %@", NSStringFromClass(self.class), self, self.path, self.commandFields, self.expectedResult];`
	`42`	`+ return [NSString stringWithFormat:@"<%@: %p, path: %@, fields: %@, requiredResponse: %@", NSStringFromClass(self.class), self, self.path, self.commandFields, self.requiredResponse];`
`43`	`43`	`}`
`44`	`44`
`45`		`-#pragma mark - MTRCommandWithExpectedResult NSSecureCoding implementation`
	`45`	`+#pragma mark - MTRCommandWithRequiredResponse NSSecureCoding implementation`
`46`	`46`
`47`	`47`	`static NSString * const sPathKey = @"pathKey";`
`48`	`48`	`static NSString * const sFieldsKey = @"fieldsKey";`
`49`		`-static NSString * const sExpectedResultKey = @"expectedResultKey";`
	`49`	`+static NSString * const sExpectedResultKey = @"requiredResponseKey";`
`50`	`50`
`51`	`51`	`+ (BOOL)supportsSecureCoding`
`52`	`52`	`{`
`@@ -62,38 +62,38 @@ - (nullable instancetype)initWithCoder:(NSCoder *)decoder`
`62`	`62`
`63`	`63`	`_path = [decoder decodeObjectOfClass:MTRCommandPath.class forKey:sPathKey];`
`64`	`64`	`if (!_path \|\| ![_path isKindOfClass:MTRCommandPath.class]) {`
`65`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for endpoint, not MTRCommandPath.", _path);`
	`65`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for endpoint, not MTRCommandPath.", _path);`
`66`	`66`	`return nil;`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`_commandFields = [decoder decodeObjectOfClass:NSDictionary.class forKey:sFieldsKey];`
`70`	`70`	`if (_commandFields) {`
`71`	`71`	`if (![_commandFields isKindOfClass:NSDictionary.class]) {`
`72`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for commandFields, not NSDictionary.", _commandFields);`
	`72`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for commandFields, not NSDictionary.", _commandFields);`
`73`	`73`	`return nil;`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`if (!MTRDataValueDictionaryIsWellFormed(_commandFields) \|\| ![MTRStructureValueType isEqual:_commandFields[MTRTypeKey]]) {`
`77`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for commandFields, not a structure-typed data-value dictionary.", _commandFields);`
	`77`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for commandFields, not a structure-typed data-value dictionary.", _commandFields);`
`78`	`78`	`return nil;`
`79`	`79`	`}`
`80`	`80`	`}`
`81`	`81`
`82`		`- _expectedResult = [decoder decodeObjectOfClass:NSDictionary.class forKey:sExpectedResultKey];`
`83`		`- if (_expectedResult) {`
`84`		`- if (![_expectedResult isKindOfClass:NSDictionary.class]) {`
`85`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded %@ for expectedResult, not NSDictionary.", _expectedResult);`
	`82`	`+ _requiredResponse = [decoder decodeObjectOfClass:NSDictionary.class forKey:sExpectedResultKey];`
	`83`	`+ if (_requiredResponse) {`
	`84`	`+ if (![_requiredResponse isKindOfClass:NSDictionary.class]) {`
	`85`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded %@ for requiredResponse, not NSDictionary.", _requiredResponse);`
`86`	`86`	`return nil;`
`87`	`87`	`}`
`88`	`88`
`89`		`- for (id key in _expectedResult) {`
	`89`	`+ for (id key in _requiredResponse) {`
`90`	`90`	`if (![key isKindOfClass:NSNumber.class]) {`
`91`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded key %@ in expectedResult", key);`
	`91`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded key %@ in requiredResponse", key);`
`92`	`92`	`return nil;`
`93`	`93`	`}`
`94`	`94`
`95`		`- if (![_expectedResult[key] isKindOfClass:NSDictionary.class] \|\| !MTRDataValueDictionaryIsWellFormed(_expectedResult[key])) {`
`96`		`- MTR_LOG_ERROR("MTRCommandWithExpectedResult decoded value %@ for key %@ in expectedResult", _expectedResult[key], key);`
	`95`	`+ if (![_requiredResponse[key] isKindOfClass:NSDictionary.class] \|\| !MTRDataValueDictionaryIsWellFormed(_requiredResponse[key])) {`
	`96`	`+ MTR_LOG_ERROR("MTRCommandWithRequiredResponse decoded value %@ for key %@ in requiredResponse", _requiredResponse[key], key);`
`97`	`97`	`return nil;`
`98`	`98`	`}`
`99`	`99`	`}`
`@@ -111,8 +111,8 @@ - (void)encodeWithCoder:(NSCoder *)coder`
`111`	`111`	`if (self.commandFields) {`
`112`	`112`	`[coder encodeObject:self.commandFields forKey:sFieldsKey];`
`113`	`113`	`}`
`114`		`- if (self.expectedResult) {`
`115`		`- [coder encodeObject:self.expectedResult forKey:sExpectedResultKey];`
	`114`	`+ if (self.requiredResponse) {`
	`115`	`+ [coder encodeObject:self.requiredResponse forKey:sExpectedResultKey];`
`116`	`116`	`}`
`117`	`117`	`}`
`118`	`118`
Original file line number	Diff line number	Diff line change
`@@ -523,7 +523,7 @@ - (void)_invokeKnownCommandWithEndpointID:(NSNumber *)endpointID`
`523`	`523`	`completion:responseHandler];`
`524`	`524`	`}`
`525`	`525`
`526`		`-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult > > *)commands`
	`526`	`+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse > > *)commands`
`527`	`527`	`queue:(dispatch_queue_t)queue`
`528`	`528`	`completion:(MTRDeviceResponseHandler)completion`
`529`	`529`	`{`
Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,7 @@ - (void)_invokeCommandWithEndpointID:(NSNumber *)endpointID`
`459`	`459`	`}`
`460`	`460`	`}`
`461`	`461`
`462`		`-- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithExpectedResult > > *)commands`
	`462`	`+- (void)invokeCommands:(NSArray<NSArray<MTRCommandWithRequiredResponse > > *)commands`
`463`	`463`	`queue:(dispatch_queue_t)queue`
`464`	`464`	`completion:(MTRDeviceResponseHandler)completion`
`465`	`465`	`{`