Skip to content

Commit d3f5620

Browse files
committed
test: fix warn_unused_result from utils::Hex in commissioner_impl_test
1 parent 729c61f commit d3f5620

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

src/library/commissioner_impl_test.cpp

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -749,7 +749,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse)
749749
// TLV: Answer(32, 2, 0000), QueryID(33, 2, 007B)
750750
// Hex: 200200002102007B
751751
ByteArray payload1;
752-
utils::Hex(payload1, "200200002102007B");
752+
EXPECT_EQ(utils::Hex(payload1, "200200002102007B"), ErrorCode::kNone);
753753
request.Append(payload1);
754754

755755
mCommissioner.HandleDiagGetAnswer(request);
@@ -776,7 +776,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse)
776776
TestRequest request2(coap::Type::kConfirmable, coap::Code::kContent);
777777
request2.SetEndpointPublic(&endpoint);
778778
ByteArray payload2;
779-
utils::Hex(payload2, "200280012102007B1101FF");
779+
EXPECT_EQ(utils::Hex(payload2, "200280012102007B1101FF"), ErrorCode::kNone);
780780
request2.Append(payload2);
781781

782782
mCommissioner.HandleDiagGetAnswer(request2);
@@ -809,7 +809,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_SinglePacketWithQuer
809809
// ChannelPages(17, 1, FF)
810810
// Hex: 2102007B1101FF
811811
ByteArray payload;
812-
utils::Hex(payload, "2102007B1101FF");
812+
EXPECT_EQ(utils::Hex(payload, "2102007B1101FF"), ErrorCode::kNone);
813813
request.Append(payload);
814814

815815
// Pre-populate pending query to verify it gets cleaned up
@@ -844,7 +844,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_SinglePacketWithQuer
844844
// ChannelPages(17, 1, EE)
845845
// Hex: 210201C8200280001101EE
846846
ByteArray payload;
847-
utils::Hex(payload, "210201C8200280001101EE");
847+
EXPECT_EQ(utils::Hex(payload, "210201C8200280001101EE"), ErrorCode::kNone);
848848
request.Append(payload);
849849

850850
mCommissioner.HandleDiagGetAnswer(request);
@@ -876,7 +876,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_Legacy)
876876
// ChannelPages(17, 1, DD)
877877
// Hex: 1101DD
878878
ByteArray payload;
879-
utils::Hex(payload, "1101DD");
879+
EXPECT_EQ(utils::Hex(payload, "1101DD"), ErrorCode::kNone);
880880
request.Append(payload);
881881

882882
mCommissioner.HandleDiagGetAnswer(request);
@@ -905,7 +905,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_MultiPacket_ThreeFra
905905
request.SetEndpointPublic(&endpoint);
906906
ByteArray payload;
907907
// QueryID(33, 2, AAAA), Answer(32, 2, 0000), IPv6(8, 16, fd00::1)
908-
utils::Hex(payload, "2102AAAA200200000810fd000000000000000000000000000001");
908+
EXPECT_EQ(utils::Hex(payload, "2102AAAA200200000810fd000000000000000000000000000001"), ErrorCode::kNone);
909909
request.Append(payload);
910910
mCommissioner.HandleDiagGetAnswer(request);
911911

@@ -920,7 +920,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_MultiPacket_ThreeFra
920920
request.SetEndpointPublic(&endpoint);
921921
ByteArray payload;
922922
// QueryID(33, 2, AAAA), Answer(32, 2, 0001), IPv6(8, 16, fd00::2)
923-
utils::Hex(payload, "2102AAAA200200010810fd000000000000000000000000000002");
923+
EXPECT_EQ(utils::Hex(payload, "2102AAAA200200010810fd000000000000000000000000000002"), ErrorCode::kNone);
924924
request.Append(payload);
925925
mCommissioner.HandleDiagGetAnswer(request);
926926

@@ -935,7 +935,7 @@ TEST_F(CommissionerImplFragmentTest, FragmentedDiagResponse_MultiPacket_ThreeFra
935935
request.SetEndpointPublic(&endpoint);
936936
ByteArray payload;
937937
// QueryID(33, 2, AAAA), Answer(32, 2, 8002), IPv6(8, 16, fd00::3)
938-
utils::Hex(payload, "2102AAAA200280020810fd000000000000000000000000000003");
938+
EXPECT_EQ(utils::Hex(payload, "2102AAAA200280020810fd000000000000000000000000000003"), ErrorCode::kNone);
939939
request.Append(payload);
940940
mCommissioner.HandleDiagGetAnswer(request);
941941

0 commit comments

Comments
 (0)