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 <venture@google.com>

1 files changed, 3 insertions, 3 deletions

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<CrcMock> crc;
     StrictMock<ManagerMock> 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;
     };