Merge branch 'fix/ci_i2c_sleep_retention' into 'master'

fix(i2c): Modify the test memory leak check threshold and add more file pattern

See merge request espressif/esp-idf!29290

components/driver/test_apps/.build-test-rules.yml

@@ -31,6 +31,11 @@ components/driver/test_apps/legacy_i2c_driver:
       reason: lack of runner
   depends_filepatterns:
     - components/driver/i2c/**
+    # Following dependency is needed because they might increase lazy installed memory
+    # that can cause sleep retention memory leak check failed.
+    - components/ieee802154/**
+    - components/esp_coex/**
+    - components/esp_phy/**
 
 components/driver/test_apps/legacy_mcpwm_driver:
   disable:

components/driver/test_apps/legacy_i2c_driver/main/test_app_main.c

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -8,10 +8,15 @@
 #include "unity_test_runner.h"
 #include "unity_test_utils_memory.h"
 #include "esp_heap_caps.h"
+#include "sdkconfig.h"
 
 // Some resources are lazy allocated in I2C driver, so we reserved this threshold when checking memory leak
 // A better way to check a potential memory leak is running a same case by twice, for the second time, the memory usage delta should be zero
-#define LEAKS (900)
+#if CONFIG_PM_POWER_DOWN_CPU_IN_LIGHT_SLEEP
+#define LEAKS (1200) // For 802154 usage
+#else
+#define LEAKS (400)
+#endif
 
 void setUp(void)
 {

components/esp_driver_i2c/test_apps/.build-test-rules.yml

@@ -9,3 +9,8 @@ components/esp_driver_i2c/test_apps/i2c_test_apps:
       reason: lack of runners
   depends_components:
     - esp_driver_i2c
+    # Following dependency is needed because they might increase lazy installed memory
+    # that can cause sleep retention memory leak check failed.
+    - components/ieee802154/**
+    - components/esp_coex/**
+    - components/esp_phy/**

components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_app_main.c

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -8,10 +8,15 @@
 #include "unity_test_runner.h"
 #include "unity_test_utils_memory.h"
 #include "esp_heap_caps.h"
+#include "sdkconfig.h"
 
 // Some resources are lazy allocated in I2C driver, so we reserved this threshold when checking memory leak
 // A better way to check a potential memory leak is running a same case by twice, for the second time, the memory usage delta should be zero
-#define LEAKS (1000)
+#if CONFIG_PM_POWER_DOWN_CPU_IN_LIGHT_SLEEP
+#define LEAKS (1200) // For 802154 usage
+#else
+#define LEAKS (400)
+#endif
 
 void setUp(void)
 {