Update code to set _deviceConfigurationChanged correctly in _getAttributesToReportWithReportedValues

- Add more tests to verify that for attributes that don't affect device configuration changed, the
  delegate callback is not called. Also add tests for testing an attribute report having multiple
  attributes one of which causes device configuration changed.

Author: nivi-apple

src/darwin/Framework/CHIP/MTRDevice.h

@@ -428,7 +428,6 @@ MTR_EXTERN NSString * const MTRDataVersionKey MTR_NEWLY_AVAILABLE;
  *
  * Device configuration is the set of functionality implemented by the device.
  *
- * The intention is that this is called whenever any of the things that usually don't change about a device might have changed.
  */
 - (void)deviceConfigurationChanged:(MTRDevice *)device MTR_NEWLY_AVAILABLE;
 

src/darwin/Framework/CHIP/MTRDevice.mm

@@ -2545,6 +2545,13 @@ - (NSArray *)_getAttributesToReportWithReportedValues:(NSArray<NSDictionary<NSSt
             // Now that we have grabbed previousValue, update our cache with the attribute value.
             if (readCacheValueChanged) {
                 [self _setCachedAttributeValue:attributeDataValue forPath:attributePath];
+                if (!_deviceConfigurationChanged)
+                {
+                    _deviceConfigurationChanged = [self _attributeAffectsDeviceConfiguration:attributePath];
+                    if (_deviceConfigurationChanged) {
+                        MTR_LOG_INFO("Device configuration changed due to changes in attribute %@", attributePath);
+                    }
+                }
             }
 
 #ifdef DEBUG
@@ -2565,11 +2572,6 @@ - (NSArray *)_getAttributesToReportWithReportedValues:(NSArray<NSDictionary<NSSt
             // when our cached value changes and no expected value exists.
             if (readCacheValueChanged && !expectedValue) {
                 shouldReportAttribute = YES;
-
-                _deviceConfigurationChanged = [self _isAttributeAffectingDeviceConfigurationChanged:attributePath];
-                if (_deviceConfigurationChanged) {
-                    MTR_LOG_INFO("Device configuration changed due to changes in attribute %@", attributePath);
-                }
             }
 
             if (!shouldReportAttribute) {