From ec966090702ff611692b08eb951388f7b1ff8c3b Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Mon, 18 Jul 2022 14:10:54 -0700 Subject: [PATCH] [test] fix "unused result warning" in newly updated unit tests (#7905) --- tests/unit/test_checksum.cpp | 4 ++-- tests/unit/test_ip_address.cpp | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/unit/test_checksum.cpp b/tests/unit/test_checksum.cpp index ebf3209d3..101b88582 100644 --- a/tests/unit/test_checksum.cpp +++ b/tests/unit/test_checksum.cpp @@ -432,7 +432,7 @@ void TestIcmp4MessageChecksum(void) uint8_t mPayload[sizeof(kExampleIcmpMessage)]; Ip4::Icmp::Header icmpHeader; - message->AppendBytes(kExampleIcmpMessage, sizeof(kExampleIcmpMessage)); + SuccessOrQuit(message->AppendBytes(kExampleIcmpMessage, sizeof(kExampleIcmpMessage))); // Random IPv4 address, ICMP message checksum does not include a presudo header like TCP and UDP. source.mFields.m32 = 0x12345678; @@ -440,7 +440,7 @@ void TestIcmp4MessageChecksum(void) Checksum::UpdateMessageChecksum(*message, source, dest, Ip4::kProtoIcmp); - message->Read(0, icmpHeader); + SuccessOrQuit(message->Read(0, icmpHeader)); VerifyOrQuit(icmpHeader.GetChecksum() == kChecksumForExampleMessage); SuccessOrQuit(message->Read(message->GetOffset(), mPayload, sizeof(mPayload))); diff --git a/tests/unit/test_ip_address.cpp b/tests/unit/test_ip_address.cpp index cea514742..b25cef9b6 100644 --- a/tests/unit/test_ip_address.cpp +++ b/tests/unit/test_ip_address.cpp @@ -500,7 +500,7 @@ void TestIp4Cidr(void) ot::Ip4::Cidr cidr; ot::Ip4::Address generated; - network.FromString(testCase.mNetwork); + SuccessOrQuit(network.FromString(testCase.mNetwork)); cidr.mAddress = network; cidr.mLength = testCase.mLength;