Add unit tests for service and characteristics classes

Issue #5

ble/gatt_attribute.cc

@@ -33,6 +33,8 @@ ble::att::length_t attribute::write(ble::att::op_code write_type,
  ble::att::length_t length,
  void const* data)
 {
+ (void) write_type;
+
  uint8_t *data_dst = reinterpret_cast<uint8_t*>(this->data_pointer());
  ASSERT(data_dst);
 
@@ -45,15 +47,15 @@ ble::att::length_t attribute::write(ble::att::op_code write_type,
 
  data_end = std::min(data_end, data_max);
 
- std::ptrdiff_t const length = data_end - data_dst;
- if (length > 0)
+ std::ptrdiff_t const dest_length = data_end - data_dst;
+ if (dest_length > 0)
  {
- memcpy(data_dst, data, length);
- return static_cast<att::length_t>(length);
+ memcpy(data_dst, data, dest_length);
+ return static_cast<att::length_t>(dest_length);
  }
  else
  {
- logger::instance().warn("attribute::write: %d", length);
+ logger::instance().warn("attribute::write: %d", dest_length);
  return 0u;
  }
  }

unit_tests/Makefile

@@ -23,6 +23,7 @@ INCLUDE_PATH += -I $(GTEST_DIR)/include/gtest/internal
 vpath %.cc .
 vpath %.cc ../utilities
 vpath %.cc ../ble
+vpath %.cc ../ble/service
 vpath %.cc $(GTEST_DIR)/src/
 vpath %.c $(GTEST_DIR)/src/
 
@@ -51,6 +52,11 @@ CPPFLAGS += -isystem $(GTEST_DIR)/include
 TARGET_NAME = all_tests
 
 SRC =
+SRC += battery_service.cc
+SRC += gatt_attribute.cc
+SRC += gatt_characteristic.cc
+SRC += gatt_service.cc
+SRC += gatt_service_container.cc
 
 SRC += uuid.cc
 SRC += gregorian.cc
@@ -64,6 +70,7 @@ SRC += test_gregorian.cc
 SRC += test_int_to_string.cc
 SRC += test_make_array.cc
 SRC += test_observer.cc
+SRC += test_service_container.cc
 SRC += test_uuid.cc
 
 # Helpers, stubs, fakes.

unit_tests/test_service_container.cc

@@ -0,0 +1,53 @@
+/**
+ * @file test_service_container.cc
+ * @copyright (c) 2018, natersoz. Distributed under the Apache 2.0 license.
+ */
+
+#include "gtest/gtest.h"
+
+#include "ble/gatt_service_container.h"
+#include "ble/service/gap_service.h"
+#include "ble/service/battery_service.h"
+#include "ble/service/current_time_service.h"
+#include "ble/gap_connection_parameters.h"
+
+TEST(ServiceContainerTest, GAP_Battery_Time)
+{
+ // --- GAP service
+ constexpr char const device_name[] = "periph";
+ constexpr size_t const device_name_length = std::size(device_name) - 1u;
+
+ ble::gap::connection_parameters const gap_connection_parameters(
+ ble::gap::connection_interval_msec(100),
+ ble::gap::connection_interval_msec(200),
+ 0u,
+ ble::gap::supervision_timeout_msec(4000u));
+
+ ble::service::device_name device_name_characteristic(device_name, device_name_length);
+ ble::service::appearance appearance_characteristic(ble::gatt::appearance::unknown);
+ ble::service::ppcp ppcp_characteristic(gap_connection_parameters);
+ ble::service::gap_service gap_service;
+
+ gap_service.characteristic_add(device_name_characteristic);
+ gap_service.characteristic_add(appearance_characteristic);
+ gap_service.characteristic_add(ppcp_characteristic);
+
+ // --- Battery Service
+ ble::service::battery_level battery_level_characteristic;
+ ble::service::battery_power_state battery_power_characteristic;
+ ble::service::battery_service battery_service;
+
+ battery_service.characteristic_add(battery_level_characteristic);
+ battery_service.characteristic_add(battery_power_characteristic);
+
+ // --- Current Time Service
+ ble::service::current_time_service current_time_service;
+
+ ble::gatt::service_container service_container;
+ service_container.push_back(gap_service);
+ service_container.push_back(battery_service);
+ service_container.push_back(current_time_service);
+
+
+}
+