[test] fix unused returned value warning in `test_hdlc` (#9256)
This commit updates unit test `test_hdlc` to check the returned
error from `SaveFrame()` (which was recently changed in #9244).
diff --git a/tests/unit/test_hdlc.cpp b/tests/unit/test_hdlc.cpp
index f9e395c..9a2fc0f 100644
--- a/tests/unit/test_hdlc.cpp
+++ b/tests/unit/test_hdlc.cpp
@@ -150,7 +150,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sMottoText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sMottoText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(!frameBuffer.HasFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
@@ -160,7 +160,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sHelloText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sHelloText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(!frameBuffer.HasFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
@@ -189,7 +189,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sHexText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sHexText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(!frameBuffer.HasFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
@@ -225,7 +225,7 @@
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sOpenThreadText, frameBuffer.GetLength()) == 0,
"GetFrame() content is incorrect");
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
// Read the fourth saved frame and check the content
SuccessOrQuit(frameBuffer.GetNextSavedFrame(frame, length));
@@ -263,10 +263,10 @@
VerifyOrQuit(frameBuffer.GetLength() == 0, "after Clear()");
SuccessOrQuit(WriteToBuffer(sOpenThreadText, frameBuffer));
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
SuccessOrQuit(WriteToBuffer(sHelloText, frameBuffer));
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
frame = nullptr;
@@ -287,11 +287,11 @@
// Verify behavior of `ClearSavedFrames()`
SuccessOrQuit(WriteToBuffer(sHelloText, frameBuffer));
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
SuccessOrQuit(WriteToBuffer(sOpenThreadText, frameBuffer));
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
SuccessOrQuit(WriteToBuffer(sMottoText, frameBuffer));
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
SuccessOrQuit(WriteToBuffer(sHexText, frameBuffer));
frame = nullptr;
@@ -307,7 +307,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sHexText) - 1, "after ClearSavedFrames()");
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sHexText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
SuccessOrQuit(WriteToBuffer(sHelloText, frameBuffer));
@@ -334,7 +334,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sHelloText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sHelloText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
frame = nullptr;
SuccessOrQuit(frameBuffer.GetNextSavedFrame(frame, length));
VerifyOrQuit(length == sizeof(sHelloText) - 1, "GetNextSavedFrame() length is incorrect");
@@ -354,7 +354,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sMottoText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sMottoText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(!frameBuffer.HasFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.GetSkipLength() == 0, "after SaveFrame()");
@@ -367,7 +367,7 @@
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sOpenThreadText) - 1);
VerifyOrQuit(memcmp(frameBuffer.GetFrame(), sOpenThreadText, frameBuffer.GetLength()) == 0);
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit(!frameBuffer.HasFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.HasSavedFrame(), "after SaveFrame()");
VerifyOrQuit(frameBuffer.GetSkipLength() == 0, "after SaveFrame()");
@@ -398,14 +398,14 @@
VerifyOrQuit(frameBuffer.SetLength(sizeof(sHelloText)) == OT_ERROR_NONE);
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sHelloText));
VerifyOrQuit(frameBuffer.HasFrame());
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit((frame = frameBuffer.GetFrame()) != nullptr);
memcpy(frame, sMottoText, sizeof(sMottoText));
VerifyOrQuit(frameBuffer.SetLength(sizeof(sMottoText)) == OT_ERROR_NONE);
VerifyOrQuit(frameBuffer.GetLength() == sizeof(sMottoText));
VerifyOrQuit(frameBuffer.HasFrame());
- frameBuffer.SaveFrame();
+ SuccessOrQuit(frameBuffer.SaveFrame());
VerifyOrQuit((frame = frameBuffer.GetFrame()) != nullptr);
memcpy(frame, sHexText, sizeof(sHexText));
@@ -464,33 +464,33 @@
SuccessOrQuit(encoder.BeginFrame());
SuccessOrQuit(encoder.Encode(sOpenThreadText, sizeof(sOpenThreadText) - 1));
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
SuccessOrQuit(encoder.BeginFrame());
SuccessOrQuit(encoder.Encode(sMottoText, sizeof(sMottoText) - 1));
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
SuccessOrQuit(encoder.BeginFrame());
SuccessOrQuit(encoder.Encode(sHdlcSpecials, sizeof(sHdlcSpecials)));
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
SuccessOrQuit(encoder.BeginFrame());
SuccessOrQuit(encoder.Encode(sHelloText, sizeof(sHelloText) - 1));
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
SuccessOrQuit(encoder.BeginFrame());
// Empty frame
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
byte = kFlagSequence;
SuccessOrQuit(encoder.BeginFrame());
SuccessOrQuit(encoder.Encode(&byte, sizeof(uint8_t)));
SuccessOrQuit(encoder.EndFrame());
- encoderBuffer.SaveFrame();
+ SuccessOrQuit(encoderBuffer.SaveFrame());
// Feed the encoded frames to decoder and save the content
for (frame = nullptr; encoderBuffer.GetNextSavedFrame(frame, length) == OT_ERROR_NONE;)
@@ -502,7 +502,7 @@
VerifyOrQuit(decoderContext.mWasCalled);
VerifyOrQuit(decoderContext.mError == OT_ERROR_NONE, "Decoder::Decode() returned incorrect error code");
- decoderBuffer.SaveFrame();
+ SuccessOrQuit(decoderBuffer.SaveFrame());
}
// Verify the decoded frames match the original frames