Add clang-tidy comments

project-chip · Oct 1, 2024 · c3d686c · c3d686c
1 parent 1f995bf
commit c3d686c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/app/codegen-data-model-provider/tests/TestCodegenModelViaMocks.cpp b/src/app/codegen-data-model-provider/tests/TestCodegenModelViaMocks.cpp
@@ -2606,19 +2606,23 @@ TEST(TestCodegenModelViaMocks, DeviceTypeIteration)
     std::optional<DeviceTypeEntry> entry = model.FirstDeviceType(kMockEndpoint1);
     ASSERT_EQ(entry,
               std::make_optional(DeviceTypeEntry{ .deviceTypeId = kDeviceTypeId1, .deviceTypeVersion = kDeviceTypeId1Version }));
+    // NOLINTNEXTLINE(bugprone-unchecked-optional-access): Assert above that this is not none
     entry = model.NextDeviceType(kMockEndpoint1, *entry);
     ASSERT_EQ(entry,
               std::make_optional(DeviceTypeEntry{ .deviceTypeId = kDeviceTypeId2, .deviceTypeVersion = kDeviceTypeId2Version }));
+    // NOLINTNEXTLINE(bugprone-unchecked-optional-access): Assert above that this is not none
     entry = model.NextDeviceType(kMockEndpoint1, *entry);
     ASSERT_EQ(entry,
               std::make_optional(DeviceTypeEntry{ .deviceTypeId = kDeviceTypeId3, .deviceTypeVersion = kDeviceTypeId3Version }));
+    // NOLINTNEXTLINE(bugprone-unchecked-optional-access): Assert above that this is not none
     entry = model.NextDeviceType(kMockEndpoint1, *entry);
     ASSERT_FALSE(entry.has_value());
 
     // Mock endpoint 2 has 1 device types
     entry = model.FirstDeviceType(kMockEndpoint2);
     ASSERT_EQ(entry,
               std::make_optional(DeviceTypeEntry{ .deviceTypeId = kDeviceTypeId2, .deviceTypeVersion = kDeviceTypeId2Version }));
+    // NOLINTNEXTLINE(bugprone-unchecked-optional-access): Assert above that this is not none
     entry = model.NextDeviceType(kMockEndpoint2, *entry);
     ASSERT_FALSE(entry.has_value());