Add C++ tests (#132)

Author: syssi

.github/workflows/ci.yaml

@@ -48,10 +48,12 @@ jobs:
           ref: dev
           path: esphome
 
-      - name: Copy external component into the esphome project
+      - name: Copy external component and tests into the esphome project
         run: |
           cd esphome
           cp -r ../components/* esphome/components/
+          mkdir -p tests/components
+          cp -r ../tests/components/* tests/components/
           git config user.name "ci"
           git config user.email "ci@github.com"
           git add .
@@ -410,6 +412,55 @@ jobs:
           . venv/bin/activate
           pytest tests/ -v
 
+  cpp-unit-tests:
+    name: Run C++ unit tests
+    runs-on: ubuntu-24.04
+    needs:
+      - bundle
+      - common
+    defaults:
+      run:
+        working-directory: esphome
+    steps:
+      - name: Download prepared repository
+        uses: pyTooling/download-artifact@dc575e4e9df4b6e3580712285f1c90f579bb8712  # v8
+        with:
+          name: bundle
+          path: .
+
+      - name: Update index to make "git diff-index" happy
+        run: git update-index -q --really-refresh
+
+      - name: Restore Python
+        uses: ./.github/actions/restore-python
+        with:
+          python-version: ${{ env.DEFAULT_PYTHON }}
+          cache-key: ${{ needs.common.outputs.cache-key }}
+
+      - name: Cache platformio
+        if: github.ref == 'refs/heads/main'
+        uses: actions/cache@668228422ae6a00e4ad889ee87cd7109ec5666a7  # v5.0.4
+        with:
+          path: ~/.platformio
+          key: platformio-host-${{ hashFiles('esphome/platformio.ini') }}
+          restore-keys: platformio-host-
+
+      - name: Cache platformio
+        if: github.ref != 'refs/heads/main'
+        uses: actions/cache/restore@668228422ae6a00e4ad889ee87cd7109ec5666a7  # v5.0.4
+        with:
+          path: ~/.platformio
+          key: platformio-host-${{ hashFiles('esphome/platformio.ini') }}
+          restore-keys: platformio-host-
+
+      - name: Run C++ unit tests
+        run: |
+          . venv/bin/activate
+          script/cpp_unit_test.py atorch_dl24
+        env:
+          PLATFORMIO_LIBDEPS_DIR: ~/.platformio/libdeps
+          ASAN_OPTIONS: detect_leaks=0
+
   esphome-config:
     name: Validate example configurations
     runs-on: ubuntu-24.04

components/atorch_dl24/atorch_dl24.cpp

@@ -2,29 +2,16 @@
 #include "esphome/core/log.h"
 #include "esphome/core/version.h"
 
-#if ESPHOME_VERSION_CODE >= VERSION_CODE(2025, 12, 0)
-#define ADDR_STR(x) x
-#else
-#define ADDR_STR(x) (x).c_str()
-#endif
-
-#ifdef USE_ESP32
-
 namespace esphome {
 namespace atorch_dl24 {
 
 static const char *const TAG = "atorch_dl24";
 
-static const uint16_t DL24_SERVICE_UUID = 0xFFE0;
-static const uint16_t DL24_REPORT_CHARACTERISTIC_UUID = 0xFFE1;
-static const uint16_t DL24_COMMAND_CHARACTERISTIC_UUID = 0xFFE2;
-
 static const uint8_t START_OF_FRAME_BYTE1 = 0xFF;
 static const uint8_t START_OF_FRAME_BYTE2 = 0x55;
 
 static const uint8_t MESSAGE_TYPE_REPORT = 0x01;
 static const uint8_t MESSAGE_TYPE_REPLY = 0x02;
-static const uint8_t MESSAGE_TYPE_COMMAND = 0x11;
 
 static const uint8_t DEVICE_TYPE_AC_METER = 0x01;
 static const uint8_t DEVICE_TYPE_DC_METER = 0x02;
@@ -34,6 +21,19 @@ static const uint8_t COMMAND_SUCCESS = 0x01;
 static const uint8_t COMMAND_FAILED = 0x02;
 static const uint8_t COMMAND_UNSUPPORTED = 0x03;
 
+#ifdef USE_ESP32
+static const uint16_t DL24_SERVICE_UUID = 0xFFE0;
+static const uint16_t DL24_REPORT_CHARACTERISTIC_UUID = 0xFFE1;
+static const uint16_t DL24_COMMAND_CHARACTERISTIC_UUID = 0xFFE2;
+static const uint8_t MESSAGE_TYPE_COMMAND = 0x11;
+
+#if ESPHOME_VERSION_CODE >= VERSION_CODE(2025, 12, 0)
+#define ADDR_STR(x) x
+#else
+#define ADDR_STR(x) (x).c_str()
+#endif
+#endif
+
 uint8_t crc(const uint8_t data[], const uint16_t len) {
   uint8_t crc = 0;
 
@@ -45,6 +45,7 @@ uint8_t crc(const uint8_t data[], const uint16_t len) {
 }
 
 bool AtorchDL24::write_register(uint8_t device_type, uint8_t address, uint32_t value) {
+#ifdef USE_ESP32
   uint8_t frame[10];
   frame[0] = START_OF_FRAME_BYTE1;
   frame[1] = START_OF_FRAME_BYTE2;
@@ -66,6 +67,9 @@ bool AtorchDL24::write_register(uint8_t device_type, uint8_t address, uint32_t v
     ESP_LOGW(TAG, "[%s] esp_ble_gattc_write_char failed, status=%d", ADDR_STR(this->parent_->address_str()), status);
 
   return (status == 0);
+#else
+  return false;
+#endif
 }
 
 void AtorchDL24::dump_config() {
@@ -87,6 +91,7 @@ void AtorchDL24::dump_config() {
   LOG_TEXT_SENSOR(" ", "Runtime Formatted", this->runtime_formatted_text_sensor_);
 }
 
+#ifdef USE_ESP32
 void AtorchDL24::gattc_event_handler(esp_gattc_cb_event_t event, esp_gatt_if_t gattc_if,
                                      esp_ble_gattc_cb_param_t *param) {
   switch (event) {
@@ -169,6 +174,7 @@ void AtorchDL24::gattc_event_handler(esp_gattc_cb_event_t event, esp_gatt_if_t g
       break;
   }
 }
+#endif
 
 // AC Meter
 //
@@ -483,5 +489,3 @@ void AtorchDL24::publish_state_(text_sensor::TextSensor *text_sensor, const std:
 
 }  // namespace atorch_dl24
 }  // namespace esphome
-
-#endif

components/atorch_dl24/atorch_dl24.h

@@ -1,26 +1,31 @@
 #pragma once
 
 #include "esphome/core/component.h"
-#include "esphome/components/ble_client/ble_client.h"
-#include "esphome/components/esp32_ble_tracker/esp32_ble_tracker.h"
 #include "esphome/components/binary_sensor/binary_sensor.h"
 #include "esphome/components/sensor/sensor.h"
 #include "esphome/components/text_sensor/text_sensor.h"
 #include "esphome/core/hal.h"
 
 #ifdef USE_ESP32
-
+#include "esphome/components/ble_client/ble_client.h"
+#include "esphome/components/esp32_ble_tracker/esp32_ble_tracker.h"
 #include <esp_gattc_api.h>
+namespace espbt = esphome::esp32_ble_tracker;
+#endif
 
 namespace esphome {
 namespace atorch_dl24 {
 
-namespace espbt = esphome::esp32_ble_tracker;
-
-class AtorchDL24 : public esphome::ble_client::BLEClientNode, public Component {
+class AtorchDL24 :
+#ifdef USE_ESP32
+    public esphome::ble_client::BLEClientNode,
+#endif
+    public Component {
  public:
+#ifdef USE_ESP32
   void gattc_event_handler(esp_gattc_cb_event_t event, esp_gatt_if_t gattc_if,
                            esp_ble_gattc_cb_param_t *param) override;
+#endif
   void dump_config() override;
   float get_setup_priority() const override { return setup_priority::DATA; }
 
@@ -109,18 +114,18 @@ class AtorchDL24 : public esphome::ble_client::BLEClientNode, public Component {
 
   text_sensor::TextSensor *runtime_formatted_text_sensor_{nullptr};
 
-  bool check_crc_;
+  bool check_crc_{false};
 
   uint8_t previous_value_ = 61;
   std::vector<uint8_t> frame_buffer_;
   uint8_t device_type_ = 0x00;
+#ifdef USE_ESP32
   uint16_t char_notify_handle_{0};
   uint16_t char_command_handle_{0};
+#endif
   uint32_t last_publish_{0};
   uint32_t throttle_{0};
 };
 
 }  // namespace atorch_dl24
 }  // namespace esphome
-
-#endif