From d1c3e86f2368ec69098a4e786a5c4e9d2455ae1d Mon Sep 17 00:00:00 2001 From: Patrick Venture Date: Thu, 10 Jan 2019 13:12:20 -0800 Subject: bugfix: process: can be only crc The read command, if returning 0 bytes of data, does return the CRC. Tested: Verified via a client implementation that it now behaves as expected on returning 0 bytes from a read (returns only the CRC). Change-Id: I84200c0fbe8400cc9234b55991fee750cb84faa2 Signed-off-by: Patrick Venture --- test/process_unittest.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'test') diff --git a/test/process_unittest.cpp b/test/process_unittest.cpp index a908ee6..0f683e5 100644 --- a/test/process_unittest.cpp +++ b/test/process_unittest.cpp @@ -250,8 +250,8 @@ TEST(ProcessBlobCommandTest, CommandReturnsOkWithNoPayload) TEST(ProcessBlobCommandTest, CommandReturnsOkWithInvalidPayloadLength) { - // There is a minimum payload length of 3 bytes, this command returns a - // payload of 2 bytes. + // There is a minimum payload length of 2 bytes (the CRC only, no data, for + // read), this returns 1. StrictMock crc; StrictMock manager; @@ -261,7 +261,7 @@ TEST(ProcessBlobCommandTest, CommandReturnsOkWithInvalidPayloadLength) IpmiBlobHandler h = [](ManagerInterface* mgr, const uint8_t* reqBuf, uint8_t* replyCmdBuf, size_t* dataLen) { - (*dataLen) = sizeof(uint16_t); + (*dataLen) = sizeof(uint8_t); return IPMI_CC_OK; }; -- cgit v1.2.3