From 881bd1bf66e296a1790104f2d2c9dc790e905c78 Mon Sep 17 00:00:00 2001
From: Euripedes Rocha <rocha.euripedes@espressif.com>
Date: Wed, 16 Oct 2024 09:17:19 +0200
Subject: [PATCH] fix(mqtt): Corrects mqtt test apps dependencies

Test app build rules were missing the mqtt component as a dependency.
---
 examples/protocols/.build-test-rules.yml      | 37 ++++++++++++-------
 .../test_apps/protocols/.build-test-rules.yml |  2 +-
 2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/examples/protocols/.build-test-rules.yml b/examples/protocols/.build-test-rules.yml
index 35295523a4..1efc6a7495 100644
--- a/examples/protocols/.build-test-rules.yml
+++ b/examples/protocols/.build-test-rules.yml
@@ -10,6 +10,13 @@
     - mbedtls
     - protocol_examples_common
 
+.mqtt_dependencies: &mqtt_dependencies
+  <<: *default_dependencies
+  depends_filepatterns:
+    - components/mqtt/**/*
+  depends_components+:
+    - mqtt
+
 examples/protocols/esp_http_client:
   <<: *default_dependencies
   enable:
@@ -134,51 +141,55 @@ examples/protocols/modbus:
     - examples/common_components/protocol_examples_common/**/*
     - examples/protocols/modbus/mb_example_common/**/*
 
-
-examples/protocols/mqtt:
-  <<: *default_dependencies
-  depends_filepatterns:
-    - components/mqtt/**/*
-
 examples/protocols/mqtt/custom_outbox:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
 
 examples/protocols/mqtt/ssl:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable_test:
     - if: IDF_TARGET != "esp32"
       reason: only test on esp32
 
 examples/protocols/mqtt/ssl_ds:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable:
     - if: SOC_DIG_SIGN_SUPPORTED != 1
       temporary: false
       reason: DS not present
+  depends_filepatterns+:
+    - examples/protocols/mqtt/ssl_ds/**/*
 
 examples/protocols/mqtt/tcp:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable_test:
     - if: IDF_TARGET != "esp32"
       reason: only test on esp32
+  depends_filepatterns+:
+    - examples/protocols/mqtt/tcp/**/*
 
 examples/protocols/mqtt/ws:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable_test:
     - if: IDF_TARGET != "esp32"
       reason: only test on esp32
+  depends_filepatterns+:
+    - examples/protocols/mqtt/ws/**/*
 
 examples/protocols/mqtt/wss:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable_test:
     - if: IDF_TARGET != "esp32"
       reason: only test on esp32
+  depends_filepatterns+:
+    - examples/protocols/mqtt/wss/**/*
 
 examples/protocols/mqtt5:
-  <<: *default_dependencies
+  <<: *mqtt_dependencies
   disable_test:
     - if: IDF_TARGET != "esp32"
       reason: only test on esp32
+  depends_filepatterns+:
+    - examples/protocols/mqtt5/**/*
 
 examples/protocols/smtp_client:
   <<: *default_dependencies
diff --git a/tools/test_apps/protocols/.build-test-rules.yml b/tools/test_apps/protocols/.build-test-rules.yml
index 81198acd88..21d8dfa455 100644
--- a/tools/test_apps/protocols/.build-test-rules.yml
+++ b/tools/test_apps/protocols/.build-test-rules.yml
@@ -16,7 +16,7 @@ tools/test_apps/protocols/mqtt/publish_connect_test:
       temporary: true
       reason: lack of runners
   depends_components:
-    - esp_eth
+    - mqtt
   depends_filepatterns:
     - tools/ci/python_packages/common_test_methods.py
     - examples/common_components/**/*