fix(bh1750): fix missing RTOS headers and update test_apps

close #521

Author: YanKE01

.gitlab/ci/build.yml

@@ -1522,6 +1522,14 @@ build_components_sensors_imu_lis2dh12_test_apps:
   variables:
     EXAMPLE_DIR: components/sensors/imu/lis2dh12/test_apps
 
+build_components_sensors_light_sensor_bh1750_test_apps:
+  extends:
+    - .build_examples_template
+    - .rules:build:components_sensors_light_sensor_bh1750_test_apps
+    - .build_idf_active_release_version
+  variables:
+    EXAMPLE_DIR: components/sensors/light_sensor/bh1750/test_apps
+
 build_components_sensors_light_sensor_veml6040_test_apps:
   extends:
     - .build_examples_template

.gitlab/ci/rules.yml

@@ -2404,7 +2404,7 @@
     - <<: *if-dev-push
       changes: *patterns-components_sensors_imu_mpu6050
 
-.rules:build:components_sensors_light_sensor_bh1750_test:
+.rules:build:components_sensors_light_sensor_bh1750_test_apps:
   rules:
     - <<: *if-protected
     - <<: *if-label-build
@@ -2414,6 +2414,8 @@
       changes: *patterns-build_system
     - <<: *if-dev-push
       changes: *patterns-components_sensors_light_sensor_bh1750
+    - <<: *if-dev-push
+      changes: *patterns-components_i2c_bus
 
 .rules:build:components_sensors_light_sensor_veml6040_test_apps:
   rules:

components/sensors/light_sensor/bh1750/bh1750.c

@@ -5,6 +5,8 @@
  */
 
 #include <stdio.h>
+#include "freertos/FreeRTOS.h"
+#include "freertos/task.h"
 #include "bh1750.h"
 
 #define WRITE_BIT  I2C_MASTER_WRITE  /*!< I2C master write */

components/sensors/light_sensor/bh1750/test/CMakeLists.txt

@@ -0,0 +1,7 @@
+# The following lines of boilerplate have to be in your project's CMakeLists
+# in this exact order for cmake to work correctly
+cmake_minimum_required(VERSION 3.5)
+
+set(EXTRA_COMPONENT_DIRS "$ENV{IDF_PATH}/tools/unit-test-app/components")
+include($ENV{IDF_PATH}/tools/cmake/project.cmake)
+project(bh1750_test)

components/sensors/light_sensor/bh1750/test_apps/CMakeLists.txt

@@ -0,0 +1,2 @@
+idf_component_register(SRC_DIRS "."
+                       PRIV_REQUIRES unity test_utils i2c_bus)

components/sensors/light_sensor/bh1750/test_apps/main/CMakeLists.txt

@@ -8,10 +8,12 @@
 #include "unity.h"
 #include "bh1750.h"
 #include "esp_log.h"
+#include "freertos/FreeRTOS.h"
+#include "freertos/task.h"
 
-#define I2C_MASTER_SCL_IO           22          /*!< gpio number for I2C master clock */
-#define I2C_MASTER_SDA_IO           21          /*!< gpio number for I2C master data  */
-#define I2C_MASTER_NUM              I2C_NUM_1   /*!< I2C port number for master dev */
+#define I2C_MASTER_SCL_IO           2           /*!< gpio number for I2C master clock */
+#define I2C_MASTER_SDA_IO           1           /*!< gpio number for I2C master data  */
+#define I2C_MASTER_NUM              I2C_NUM_0   /*!< I2C port number for master dev */
 #define I2C_MASTER_TX_BUF_DISABLE   0           /*!< I2C master do not need buffer */
 #define I2C_MASTER_RX_BUF_DISABLE   0           /*!< I2C master do not need buffer */
 #define I2C_MASTER_FREQ_HZ          100000      /*!< I2C master clock frequency */
@@ -44,7 +46,7 @@ static void bh1750_test_get_data()
     int ret;
     bh1750_cmd_measure_t cmd_measure;
     float bh1750_data;
-    int cnt = 2;
+    int cnt = 10;
 
     while (cnt--) {
         bh1750_power_on(bh1750);
@@ -80,3 +82,15 @@ TEST_CASE("Sensor BH1750 test", "[bh1750][iot][sensor]")
     bh1750_test_get_data();
     bh1750_test_deinit();
 }
+
+void app_main(void)
+{
+    printf(" ____  _   _ _ _____ ____   ___    _____ _____ ____ _____ \n");
+    printf("| __ )| | | / |___  | ___| / _ \\  |_   _| ____/ ___|_   _|\n");
+    printf("|  _ \\| |_| | |  / /|___ \\| | | |   | | |  _| \\___ \\ | |  \n");
+    printf("| |_) |  _  | | / /  ___) | |_| |   | | | |___ ___) || |  \n");
+    printf("|____/|_| |_|_|/_/  |____/ \\___/    |_| |_____|____/ |_|  \n");
+    printf("                                                          \n");
+
+    unity_run_menu();
+}

components/sensors/light_sensor/bh1750/test/bh1750_test.c renamed to components/sensors/light_sensor/bh1750/test_apps/main/bh1750_test.c

@@ -0,0 +1,8 @@
+dependencies:
+  idf: ">=4.4"
+  bh1750:
+    version: "*"
+    override_path: "../../../bh1750"
+  i2c_bus:
+    version: "*"
+    override_path: "../../../../../i2c_bus"