github/esphome
2
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-10-21 21:36:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

[packet_interface] Fix clang-tidy findings in tests

Browse Source 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
clydebarrow
2026-10-19 22:21:49 +11:00
co-authored by Claude Opus 5.5
parent 659219027e
commit 63b8d6a556
2 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/components/packet_interface/basic_session.cpp
+3 -1
View File
@@ -84,7 +84,9 @@ class BasicSessionTest : public ::testing::Test {
protected:
void SetUp() override {
new (&App) Application;
App.pre_setup("session-tests", 10, "", 0);
static char name[] = "session-tests";
static char friendly_name[] = "";
App.pre_setup(name, sizeof(name) - 1, friendly_name, 0);
}
void TearDown() override { App.~Application(); }
tests/components/packet_interface/xxtea_encryption.cpp
+6 -4
View File
@@ -79,7 +79,9 @@ class XxteaEncryptionTest : public ::testing::Test {
protected:
void SetUp() override {
new (&App) Application;
App.pre_setup("xxtea-tests", 10, "", 0);
static char name[] = "xxtea-tests";
static char friendly_name[] = "";
App.pre_setup(name, sizeof(name) - 1, friendly_name, 0);
}
void TearDown() override { App.~Application(); }
};
@@ -100,10 +102,10 @@ static void round_trip_all_sizes(bool rolling_code, size_t trailer_size) {
Peer a(0x12345678, 64, rolling_code), b(0x12345678, 64, rolling_code);
a.introduce(b);
std::vector<Packet> expected;
for (uint8_t size = 0; size <= a.encryption->get_max_packet_size(); size++) {
for (size_t size = 0; size <= a.encryption->get_max_packet_size(); size++) {
Packet packet(size);
for (uint8_t i = 0; i != size; i++)
packet[i] = size + i;
for (size_t i = 0; i != size; i++)
packet[i] = static_cast<uint8_t>(size + i);
expected.push_back(packet);
ASSERT_TRUE(a.send(packet));
ASSERT_EQ(a.link.sent.back().size() % 4, 0);