summaryrefslogtreecommitdiffstats
path: root/test/ipmi_commit_unittest.cpp
diff options
context:
space:
mode:
authorPatrick Venture <venture@google.com>2018-09-12 08:53:29 -0700
committerPatrick Venture <venture@google.com>2018-09-15 13:11:30 -0700
commitef3aeadc9be37c47d0627e576e81a74a5bb9e94f (patch)
tree95c183977468b107bdd3faaaa988b5d853be2f00 /test/ipmi_commit_unittest.cpp
parentbaa73da1abaaf05ea1133319405fb2b891825618 (diff)
downloadphosphor-ipmi-blobs-ef3aeadc9be37c47d0627e576e81a74a5bb9e94f.tar.gz
phosphor-ipmi-blobs-ef3aeadc9be37c47d0627e576e81a74a5bb9e94f.zip
initial drop of phosphor-ipmi-blobs
This implements a majority of the OEM IPMI BLOBS protocol. The only piece missing from this is the timed expiration of sessions. Change-Id: I82c9d17b625c94fc3340edcfabbbf1ffeb5ad7ac Signed-off-by: Patrick Venture <venture@google.com>
Diffstat (limited to 'test/ipmi_commit_unittest.cpp')
-rw-r--r--test/ipmi_commit_unittest.cpp112
1 files changed, 112 insertions, 0 deletions
diff --git a/test/ipmi_commit_unittest.cpp b/test/ipmi_commit_unittest.cpp
new file mode 100644
index 0000000..1cc47a4
--- /dev/null
+++ b/test/ipmi_commit_unittest.cpp
@@ -0,0 +1,112 @@
+#include "ipmi.hpp"
+#include "manager_mock.hpp"
+
+#include <cstring>
+
+#include <gtest/gtest.h>
+
+namespace blobs
+{
+
+using ::testing::_;
+using ::testing::ElementsAreArray;
+using ::testing::Return;
+
+// ipmid.hpp isn't installed where we can grab it and this value is per BMC
+// SoC.
+#define MAX_IPMI_BUFFER 64
+
+TEST(BlobCommitTest, InvalidCommitDataLengthReturnsFailure)
+{
+ // The commit command supports an optional commit blob. This test verifies
+ // we sanity check the length of that blob.
+
+ ManagerMock mgr;
+ size_t dataLen;
+ uint8_t request[MAX_IPMI_BUFFER] = {0};
+ uint8_t reply[MAX_IPMI_BUFFER] = {0};
+ auto req = reinterpret_cast<struct BmcBlobCommitTx*>(request);
+
+ req->cmd = BlobOEMCommands::bmcBlobCommit;
+ req->crc = 0;
+ req->sessionId = 0x54;
+ req->commitDataLen =
+ 1; // It's one byte, but that's more than the packet size.
+
+ dataLen = sizeof(struct BmcBlobCommitTx);
+
+ EXPECT_EQ(IPMI_CC_INVALID, commitBlob(&mgr, request, reply, &dataLen));
+}
+
+TEST(BlobCommitTest, ValidCommitNoDataHandlerRejectsReturnsFailure)
+{
+ // The commit packet is valid and the manager's commit call returns failure.
+
+ ManagerMock mgr;
+ size_t dataLen;
+ uint8_t request[MAX_IPMI_BUFFER] = {0};
+ uint8_t reply[MAX_IPMI_BUFFER] = {0};
+ auto req = reinterpret_cast<struct BmcBlobCommitTx*>(request);
+
+ req->cmd = BlobOEMCommands::bmcBlobCommit;
+ req->crc = 0;
+ req->sessionId = 0x54;
+ req->commitDataLen = 0;
+
+ dataLen = sizeof(struct BmcBlobCommitTx);
+
+ EXPECT_CALL(mgr, commit(req->sessionId, _)).WillOnce(Return(false));
+
+ EXPECT_EQ(IPMI_CC_INVALID, commitBlob(&mgr, request, reply, &dataLen));
+}
+
+TEST(BlobCommitTest, ValidCommitNoDataHandlerAcceptsReturnsSuccess)
+{
+ // Commit called with no data and everything returns success.
+
+ ManagerMock mgr;
+ size_t dataLen;
+ uint8_t request[MAX_IPMI_BUFFER] = {0};
+ uint8_t reply[MAX_IPMI_BUFFER] = {0};
+ auto req = reinterpret_cast<struct BmcBlobCommitTx*>(request);
+
+ req->cmd = BlobOEMCommands::bmcBlobCommit;
+ req->crc = 0;
+ req->sessionId = 0x54;
+ req->commitDataLen = 0;
+
+ dataLen = sizeof(struct BmcBlobCommitTx);
+
+ EXPECT_CALL(mgr, commit(req->sessionId, _)).WillOnce(Return(true));
+
+ EXPECT_EQ(IPMI_CC_OK, commitBlob(&mgr, request, reply, &dataLen));
+}
+
+TEST(BlobCommitTest, ValidCommitWithDataHandlerAcceptsReturnsSuccess)
+{
+ // Commit called with extra data and everything returns success.
+
+ ManagerMock mgr;
+ size_t dataLen;
+ uint8_t request[MAX_IPMI_BUFFER] = {0};
+ uint8_t reply[MAX_IPMI_BUFFER] = {0};
+ auto req = reinterpret_cast<struct BmcBlobCommitTx*>(request);
+
+ uint8_t expectedBlob[4] = {0x25, 0x33, 0x45, 0x67};
+
+ req->cmd = BlobOEMCommands::bmcBlobCommit;
+ req->crc = 0;
+ req->sessionId = 0x54;
+ req->commitDataLen = sizeof(expectedBlob);
+ std::memcpy(req->commitData, &expectedBlob[0], sizeof(expectedBlob));
+
+ dataLen = sizeof(struct BmcBlobCommitTx) + sizeof(expectedBlob);
+
+ EXPECT_CALL(mgr,
+ commit(req->sessionId,
+ ElementsAreArray(expectedBlob, sizeof(expectedBlob))))
+ .WillOnce(Return(true));
+
+ EXPECT_EQ(IPMI_CC_OK, commitBlob(&mgr, request, reply, &dataLen));
+}
+} // namespace blobs
OpenPOWER on IntegriCloud