| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 1 | #include "data_interface_mock.hpp" | 
| Patrick Venture | 84778b8 | 2019-06-26 20:11:09 -0700 | [diff] [blame] | 2 | #include "flags.hpp" | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 3 | #include "status.hpp" | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 4 | #include "tool_errors.hpp" | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 5 | #include "updater.hpp" | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 6 | #include "updater_mock.hpp" | 
| Patrick Venture | 7dad86f | 2019-05-17 08:52:20 -0700 | [diff] [blame] | 7 | #include "util.hpp" | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 8 |  | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 9 | #include <blobs-ipmid/blobs.hpp> | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 10 | #include <ipmiblob/blob_errors.hpp> | 
| Patrick Venture | 664c5bc | 2019-03-07 08:09:45 -0800 | [diff] [blame] | 11 | #include <ipmiblob/test/blob_interface_mock.hpp> | 
| Patrick Venture | 9b37b09 | 2020-05-28 20:58:57 -0700 | [diff] [blame] | 12 |  | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 13 | #include <string> | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 14 | #include <vector> | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 15 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 16 | #include <gmock/gmock.h> | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 17 | #include <gtest/gtest.h> | 
|  | 18 |  | 
| Patrick Venture | 9b534f0 | 2018-12-13 16:10:02 -0800 | [diff] [blame] | 19 | namespace host_tool | 
|  | 20 | { | 
|  | 21 |  | 
| Patrick Venture | 7dcca5d | 2019-05-15 12:32:33 -0700 | [diff] [blame] | 22 | using ::testing::_; | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 23 | using ::testing::Return; | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 24 | using ::testing::Throw; | 
| Patrick Venture | b58f561 | 2019-05-07 09:22:07 -0700 | [diff] [blame] | 25 | using ::testing::TypedEq; | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 26 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 27 | class UpdateHandlerTest : public ::testing::Test | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 28 | { | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 29 | protected: | 
|  | 30 | const std::uint16_t session = 0xbeef; | 
|  | 31 |  | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 32 | DataInterfaceMock handlerMock; | 
|  | 33 | ipmiblob::BlobInterfaceMock blobMock; | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 34 | UpdateHandler updater{&blobMock, &handlerMock}; | 
|  | 35 | }; | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 36 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 37 | TEST_F(UpdateHandlerTest, CheckAvailableSuccess) | 
|  | 38 | { | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 39 | EXPECT_CALL(blobMock, getBlobList()) | 
| Patrick Venture | 7dad86f | 2019-05-17 08:52:20 -0700 | [diff] [blame] | 40 | .WillOnce( | 
| Patrick Venture | 1d5a31c | 2019-05-20 11:38:22 -0700 | [diff] [blame] | 41 | Return(std::vector<std::string>({ipmi_flash::staticLayoutBlobId}))); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 42 |  | 
| Patrick Venture | 1d5a31c | 2019-05-20 11:38:22 -0700 | [diff] [blame] | 43 | EXPECT_TRUE(updater.checkAvailable(ipmi_flash::staticLayoutBlobId)); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 44 | } | 
|  | 45 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 46 | TEST_F(UpdateHandlerTest, CheckAvailableFailure) | 
|  | 47 | { | 
|  | 48 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 49 | .WillOnce(Return(std::vector<std::string>())); | 
|  | 50 |  | 
|  | 51 | EXPECT_FALSE(updater.checkAvailable(ipmi_flash::staticLayoutBlobId)); | 
|  | 52 | } | 
|  | 53 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 54 | TEST_F(UpdateHandlerTest, SendFileSuccess) | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 55 | { | 
|  | 56 | /* Call sendFile to verify it does what we expect. */ | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 57 | std::string firmwareImage = "image.bin"; | 
|  | 58 |  | 
|  | 59 | std::uint16_t supported = | 
|  | 60 | static_cast<std::uint16_t>( | 
| Patrick Venture | 84778b8 | 2019-06-26 20:11:09 -0700 | [diff] [blame] | 61 | ipmi_flash::FirmwareFlags::UpdateFlags::lpc) | | 
|  | 62 | static_cast<std::uint16_t>( | 
|  | 63 | ipmi_flash::FirmwareFlags::UpdateFlags::openWrite); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 64 |  | 
|  | 65 | EXPECT_CALL(handlerMock, supportedType()) | 
| Patrick Venture | 84778b8 | 2019-06-26 20:11:09 -0700 | [diff] [blame] | 66 | .WillOnce(Return(ipmi_flash::FirmwareFlags::UpdateFlags::lpc)); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 67 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 68 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::staticLayoutBlobId, supported)) | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 69 | .WillOnce(Return(session)); | 
|  | 70 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 71 | EXPECT_CALL(handlerMock, sendContents(firmwareImage, session)) | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 72 | .WillOnce(Return(true)); | 
|  | 73 |  | 
|  | 74 | EXPECT_CALL(blobMock, closeBlob(session)).Times(1); | 
|  | 75 |  | 
| Patrick Venture | 1d5a31c | 2019-05-20 11:38:22 -0700 | [diff] [blame] | 76 | updater.sendFile(ipmi_flash::staticLayoutBlobId, firmwareImage); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 77 | } | 
|  | 78 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 79 | TEST_F(UpdateHandlerTest, SendFileExceptsOnBlobOpening) | 
|  | 80 | { | 
|  | 81 | std::string firmwareImage = "image.bin"; | 
|  | 82 |  | 
|  | 83 | std::uint16_t supported = | 
|  | 84 | static_cast<std::uint16_t>( | 
|  | 85 | ipmi_flash::FirmwareFlags::UpdateFlags::lpc) | | 
|  | 86 | static_cast<std::uint16_t>( | 
|  | 87 | ipmi_flash::FirmwareFlags::UpdateFlags::openWrite); | 
|  | 88 |  | 
|  | 89 | EXPECT_CALL(handlerMock, supportedType()) | 
|  | 90 | .WillOnce(Return(ipmi_flash::FirmwareFlags::UpdateFlags::lpc)); | 
|  | 91 |  | 
|  | 92 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::staticLayoutBlobId, supported)) | 
|  | 93 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 94 |  | 
|  | 95 | EXPECT_THROW( | 
|  | 96 | updater.sendFile(ipmi_flash::staticLayoutBlobId, firmwareImage), | 
|  | 97 | ToolException); | 
|  | 98 | } | 
|  | 99 |  | 
|  | 100 | TEST_F(UpdateHandlerTest, SendFileHandlerFailureCausesException) | 
|  | 101 | { | 
|  | 102 | std::string firmwareImage = "image.bin"; | 
|  | 103 |  | 
|  | 104 | std::uint16_t supported = | 
|  | 105 | static_cast<std::uint16_t>( | 
|  | 106 | ipmi_flash::FirmwareFlags::UpdateFlags::lpc) | | 
|  | 107 | static_cast<std::uint16_t>( | 
|  | 108 | ipmi_flash::FirmwareFlags::UpdateFlags::openWrite); | 
|  | 109 |  | 
|  | 110 | EXPECT_CALL(handlerMock, supportedType()) | 
| Willy Tu | 1b23b77 | 2023-03-15 01:33:03 -0700 | [diff] [blame] | 111 | .WillRepeatedly(Return(ipmi_flash::FirmwareFlags::UpdateFlags::lpc)); | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 112 |  | 
|  | 113 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::staticLayoutBlobId, supported)) | 
| Willy Tu | 1b23b77 | 2023-03-15 01:33:03 -0700 | [diff] [blame] | 114 | .WillRepeatedly(Return(session)); | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 115 |  | 
|  | 116 | EXPECT_CALL(handlerMock, sendContents(firmwareImage, session)) | 
| Willy Tu | 1b23b77 | 2023-03-15 01:33:03 -0700 | [diff] [blame] | 117 | .WillRepeatedly(Return(false)); | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 118 |  | 
| Willy Tu | 1b23b77 | 2023-03-15 01:33:03 -0700 | [diff] [blame] | 119 | EXPECT_CALL(blobMock, closeBlob(session)).Times(3); | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 120 |  | 
|  | 121 | EXPECT_THROW( | 
|  | 122 | updater.sendFile(ipmi_flash::staticLayoutBlobId, firmwareImage), | 
|  | 123 | ToolException); | 
|  | 124 | } | 
|  | 125 |  | 
| Willy Tu | 1b23b77 | 2023-03-15 01:33:03 -0700 | [diff] [blame] | 126 | TEST_F(UpdateHandlerTest, SendFileHandlerPassWithRetries) | 
|  | 127 | { | 
|  | 128 | std::string firmwareImage = "image.bin"; | 
|  | 129 |  | 
|  | 130 | std::uint16_t supported = | 
|  | 131 | static_cast<std::uint16_t>( | 
|  | 132 | ipmi_flash::FirmwareFlags::UpdateFlags::lpc) | | 
|  | 133 | static_cast<std::uint16_t>( | 
|  | 134 | ipmi_flash::FirmwareFlags::UpdateFlags::openWrite); | 
|  | 135 |  | 
|  | 136 | EXPECT_CALL(handlerMock, supportedType()) | 
|  | 137 | .WillRepeatedly(Return(ipmi_flash::FirmwareFlags::UpdateFlags::lpc)); | 
|  | 138 |  | 
|  | 139 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::staticLayoutBlobId, supported)) | 
|  | 140 | .WillRepeatedly(Return(session)); | 
|  | 141 |  | 
|  | 142 | EXPECT_CALL(handlerMock, sendContents(firmwareImage, session)) | 
|  | 143 | .WillOnce(Return(false)) | 
|  | 144 | .WillOnce(Return(false)) | 
|  | 145 | .WillOnce(Return(true)); | 
|  | 146 |  | 
|  | 147 | EXPECT_CALL(blobMock, closeBlob(session)).Times(3); | 
|  | 148 |  | 
|  | 149 | updater.sendFile(ipmi_flash::staticLayoutBlobId, firmwareImage); | 
|  | 150 | } | 
|  | 151 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 152 | TEST_F(UpdateHandlerTest, VerifyFileHandleReturnsTrueOnSuccess) | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 153 | { | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 154 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::verifyBlobId, _)) | 
| Patrick Venture | aa32a36 | 2018-12-13 10:52:33 -0800 | [diff] [blame] | 155 | .WillOnce(Return(session)); | 
| Patrick Venture | 55646de | 2019-05-16 10:06:26 -0700 | [diff] [blame] | 156 | EXPECT_CALL(blobMock, commit(session, _)).WillOnce(Return()); | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 157 | ipmiblob::StatResponse verificationResponse = {}; | 
|  | 158 | /* the other details of the response are ignored, and should be. */ | 
|  | 159 | verificationResponse.metadata.push_back( | 
|  | 160 | static_cast<std::uint8_t>(ipmi_flash::ActionStatus::success)); | 
| Patrick Venture | 7dcca5d | 2019-05-15 12:32:33 -0700 | [diff] [blame] | 161 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 162 | EXPECT_CALL(blobMock, getStat(TypedEq<std::uint16_t>(session))) | 
|  | 163 | .WillOnce(Return(verificationResponse)); | 
|  | 164 | EXPECT_CALL(blobMock, closeBlob(session)).WillOnce(Return()); | 
|  | 165 |  | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 166 | EXPECT_TRUE(updater.verifyFile(ipmi_flash::verifyBlobId, false)); | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 167 | } | 
|  | 168 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 169 | TEST_F(UpdateHandlerTest, VerifyFileHandleSkipsPollingIfIgnoreStatus) | 
|  | 170 | { | 
|  | 171 | /* if ignoreStatus, it'll skip polling for a verification result. */ | 
|  | 172 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::verifyBlobId, _)) | 
|  | 173 | .WillOnce(Return(session)); | 
|  | 174 | EXPECT_CALL(blobMock, commit(session, _)).WillOnce(Return()); | 
|  | 175 |  | 
|  | 176 | EXPECT_CALL(blobMock, closeBlob(session)).WillOnce(Return()); | 
|  | 177 |  | 
|  | 178 | EXPECT_TRUE(updater.verifyFile(ipmi_flash::verifyBlobId, true)); | 
|  | 179 | } | 
|  | 180 |  | 
|  | 181 | TEST_F(UpdateHandlerTest, VerifyFileConvertsOpenBlobExceptionToToolException) | 
|  | 182 | { | 
|  | 183 | /* On open, it can except and this is converted to a ToolException. */ | 
|  | 184 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::verifyBlobId, _)) | 
|  | 185 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 186 | EXPECT_THROW(updater.verifyFile(ipmi_flash::verifyBlobId, false), | 
|  | 187 | ToolException); | 
|  | 188 | } | 
|  | 189 |  | 
|  | 190 | TEST_F(UpdateHandlerTest, VerifyFileCommitExceptionForwards) | 
|  | 191 | { | 
|  | 192 | /* On commit, it can except. */ | 
|  | 193 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::verifyBlobId, _)) | 
|  | 194 | .WillOnce(Return(session)); | 
|  | 195 | EXPECT_CALL(blobMock, commit(session, _)) | 
|  | 196 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 197 | EXPECT_THROW(updater.verifyFile(ipmi_flash::verifyBlobId, false), | 
|  | 198 | ToolException); | 
|  | 199 | } | 
|  | 200 |  | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 201 | TEST_F(UpdateHandlerTest, ReadVerisonReturnExpected) | 
|  | 202 | { | 
|  | 203 | /* It can return as expected, when polling and readBytes succeeds. */ | 
|  | 204 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::biosVersionBlobId, _)) | 
|  | 205 | .WillOnce(Return(session)); | 
|  | 206 | ipmiblob::StatResponse readVersionResponse = {}; | 
| Patrick Williams | 1038836 | 2023-05-10 07:51:09 -0500 | [diff] [blame^] | 207 | readVersionResponse.blob_state = blobs::StateFlags::open_read | | 
|  | 208 | blobs::StateFlags::committed; | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 209 | readVersionResponse.size = 10; | 
|  | 210 | EXPECT_CALL(blobMock, getStat(TypedEq<std::uint16_t>(session))) | 
|  | 211 | .WillOnce(Return(readVersionResponse)); | 
|  | 212 | std::vector<uint8_t> resp = {0x2d, 0xfe}; | 
|  | 213 | EXPECT_CALL(blobMock, readBytes(session, 0, _)).WillOnce(Return(resp)); | 
|  | 214 |  | 
|  | 215 | EXPECT_CALL(blobMock, closeBlob(session)).WillOnce(Return()); | 
|  | 216 | EXPECT_EQ(resp, updater.readVersion(ipmi_flash::biosVersionBlobId)); | 
|  | 217 | } | 
|  | 218 |  | 
|  | 219 | TEST_F(UpdateHandlerTest, ReadVersionExceptionWhenPollingSucceedsReadBytesFails) | 
|  | 220 | { | 
|  | 221 | /* On readBytes, it can except. */ | 
|  | 222 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::biosVersionBlobId, _)) | 
|  | 223 | .WillOnce(Return(session)); | 
|  | 224 | ipmiblob::StatResponse readVersionResponse = {}; | 
| Patrick Williams | 1038836 | 2023-05-10 07:51:09 -0500 | [diff] [blame^] | 225 | readVersionResponse.blob_state = blobs::StateFlags::open_read | | 
|  | 226 | blobs::StateFlags::committed; | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 227 | readVersionResponse.size = 10; | 
|  | 228 | EXPECT_CALL(blobMock, getStat(TypedEq<std::uint16_t>(session))) | 
|  | 229 | .WillOnce(Return(readVersionResponse)); | 
|  | 230 | EXPECT_CALL(blobMock, readBytes(session, 0, _)) | 
|  | 231 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 232 | EXPECT_CALL(blobMock, closeBlob(session)).WillOnce(Return()); | 
|  | 233 | EXPECT_THROW(updater.readVersion(ipmi_flash::biosVersionBlobId), | 
|  | 234 | ToolException); | 
|  | 235 | } | 
|  | 236 |  | 
| William A. Kennington III | f88bcf3 | 2021-10-14 02:15:10 -0700 | [diff] [blame] | 237 | TEST_F(UpdateHandlerTest, ReadVersionReturnsErrorIfPollingFails) | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 238 | { | 
| William A. Kennington III | f88bcf3 | 2021-10-14 02:15:10 -0700 | [diff] [blame] | 239 | /* It can throw an error, when polling fails. */ | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 240 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::biosVersionBlobId, _)) | 
|  | 241 | .WillOnce(Return(session)); | 
|  | 242 | ipmiblob::StatResponse readVersionResponse = {}; | 
|  | 243 | readVersionResponse.blob_state = blobs::StateFlags::commit_error; | 
|  | 244 | EXPECT_CALL(blobMock, getStat(TypedEq<std::uint16_t>(session))) | 
|  | 245 | .WillOnce(Return(readVersionResponse)); | 
|  | 246 | EXPECT_CALL(blobMock, closeBlob(session)).WillOnce(Return()); | 
| William A. Kennington III | f88bcf3 | 2021-10-14 02:15:10 -0700 | [diff] [blame] | 247 | EXPECT_THROW(updater.readVersion(ipmi_flash::biosVersionBlobId), | 
|  | 248 | ToolException); | 
| Jie Yang | 328f520 | 2021-03-16 00:52:07 -0700 | [diff] [blame] | 249 | } | 
|  | 250 |  | 
|  | 251 | TEST_F(UpdateHandlerTest, ReadVersionCovertsOpenBlobExceptionToToolException) | 
|  | 252 | { | 
|  | 253 | /* On open, it can except and this is converted to a ToolException. */ | 
|  | 254 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::biosVersionBlobId, _)) | 
|  | 255 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 256 | EXPECT_THROW(updater.readVersion(ipmi_flash::biosVersionBlobId), | 
|  | 257 | ToolException); | 
|  | 258 | } | 
|  | 259 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 260 | TEST_F(UpdateHandlerTest, CleanArtifactsSkipsCleanupIfUnableToOpen) | 
|  | 261 | { | 
|  | 262 | /* It only tries to commit if it's able to open the blob.  However, if | 
|  | 263 | * committing fails, this error is ignored. | 
|  | 264 | */ | 
|  | 265 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::cleanupBlobId, _)) | 
|  | 266 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 267 | EXPECT_CALL(blobMock, commit(_, _)).Times(0); | 
|  | 268 | EXPECT_CALL(blobMock, closeBlob(_)).Times(0); | 
|  | 269 |  | 
|  | 270 | updater.cleanArtifacts(); | 
|  | 271 | } | 
|  | 272 |  | 
|  | 273 | TEST_F(UpdateHandlerTest, CleanArtifactsIfOpenDoesClose) | 
|  | 274 | { | 
|  | 275 | /* The closeBlob call is called even if commit excepts. */ | 
|  | 276 | std::uint16_t session = 0xa5eb; | 
|  | 277 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::cleanupBlobId, _)) | 
|  | 278 | .WillOnce(Return(session)); | 
|  | 279 | EXPECT_CALL(blobMock, commit(session, _)) | 
|  | 280 | .WillOnce(Throw(ipmiblob::BlobException("asdf"))); | 
|  | 281 | EXPECT_CALL(blobMock, closeBlob(session)); | 
|  | 282 |  | 
|  | 283 | updater.cleanArtifacts(); | 
|  | 284 | } | 
|  | 285 |  | 
|  | 286 | TEST_F(UpdateHandlerTest, CleanArtifactsSuccessPath) | 
|  | 287 | { | 
|  | 288 | std::uint16_t session = 0xa5eb; | 
|  | 289 | EXPECT_CALL(blobMock, openBlob(ipmi_flash::cleanupBlobId, _)) | 
|  | 290 | .WillOnce(Return(session)); | 
|  | 291 | EXPECT_CALL(blobMock, commit(session, _)); | 
|  | 292 | EXPECT_CALL(blobMock, closeBlob(session)); | 
|  | 293 |  | 
|  | 294 | updater.cleanArtifacts(); | 
|  | 295 | } | 
|  | 296 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 297 | class UpdaterTest : public ::testing::Test | 
|  | 298 | { | 
|  | 299 | protected: | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 300 | static constexpr char image[] = "image.bin"; | 
|  | 301 | static constexpr char signature[] = "signature.bin"; | 
|  | 302 | static constexpr char layout[] = "static"; | 
|  | 303 | static constexpr char path[] = "/flash/static"; | 
|  | 304 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 305 | ipmiblob::BlobInterfaceMock blobMock; | 
|  | 306 | std::uint16_t session = 0xbeef; | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 307 | bool defaultIgnore = false; | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 308 | }; | 
|  | 309 |  | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 310 | TEST_F(UpdaterTest, UpdateMainReturnsSuccessIfAllSuccess) | 
|  | 311 | { | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 312 | UpdateHandlerMock handler; | 
|  | 313 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 314 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(true)); | 
|  | 315 | EXPECT_CALL(handler, sendFile(path, image)).WillOnce(Return()); | 
|  | 316 | EXPECT_CALL(handler, sendFile(ipmi_flash::hashBlobId, signature)) | 
|  | 317 | .WillOnce(Return()); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 318 | EXPECT_CALL(handler, verifyFile(ipmi_flash::verifyBlobId, defaultIgnore)) | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 319 | .WillOnce(Return(true)); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 320 | EXPECT_CALL(handler, verifyFile(ipmi_flash::updateBlobId, defaultIgnore)) | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 321 | .WillOnce(Return(true)); | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 322 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 323 | .WillOnce(Return(std::vector<std::string>({}))); | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 324 |  | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 325 | updaterMain(&handler, &blobMock, image, signature, layout, defaultIgnore); | 
|  | 326 | } | 
|  | 327 |  | 
|  | 328 | TEST_F(UpdaterTest, UpdateMainReturnsSuccessIfAllSuccessWithDeleteActiveBlob) | 
|  | 329 | { | 
|  | 330 | UpdateHandlerMock handler; | 
|  | 331 |  | 
|  | 332 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(true)); | 
|  | 333 | EXPECT_CALL(handler, sendFile(path, image)).WillOnce(Return()); | 
|  | 334 | EXPECT_CALL(handler, sendFile(ipmi_flash::hashBlobId, signature)) | 
|  | 335 | .WillOnce(Return()); | 
|  | 336 | EXPECT_CALL(handler, verifyFile(ipmi_flash::verifyBlobId, defaultIgnore)) | 
|  | 337 | .WillOnce(Return(true)); | 
|  | 338 | EXPECT_CALL(handler, verifyFile(ipmi_flash::updateBlobId, defaultIgnore)) | 
|  | 339 | .WillOnce(Return(true)); | 
|  | 340 | EXPECT_CALL(handler, cleanArtifacts()).WillOnce(Return()); | 
|  | 341 | EXPECT_CALL(blobMock, deleteBlob(ipmi_flash::activeImageBlobId)) | 
| Patrick Venture | 2862421 | 2021-11-09 09:04:03 -0800 | [diff] [blame] | 342 | .WillOnce(Return(true)); | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 343 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 344 | .WillOnce(Return(std::vector<std::string>( | 
|  | 345 | {ipmi_flash::staticLayoutBlobId, ipmi_flash::activeImageBlobId}))); | 
|  | 346 |  | 
|  | 347 | updaterMain(&handler, &blobMock, image, signature, layout, defaultIgnore); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 348 | } | 
|  | 349 |  | 
|  | 350 | TEST_F(UpdaterTest, UpdateMainReturnsSuccessWithIgnoreUpdate) | 
|  | 351 | { | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 352 | UpdateHandlerMock handler; | 
|  | 353 | bool updateIgnore = true; | 
|  | 354 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 355 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(true)); | 
|  | 356 | EXPECT_CALL(handler, sendFile(path, image)).WillOnce(Return()); | 
|  | 357 | EXPECT_CALL(handler, sendFile(ipmi_flash::hashBlobId, signature)) | 
|  | 358 | .WillOnce(Return()); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 359 | EXPECT_CALL(handler, verifyFile(ipmi_flash::verifyBlobId, defaultIgnore)) | 
|  | 360 | .WillOnce(Return(true)); | 
|  | 361 | EXPECT_CALL(handler, verifyFile(ipmi_flash::updateBlobId, updateIgnore)) | 
|  | 362 | .WillOnce(Return(true)); | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 363 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 364 | .WillOnce(Return(std::vector<std::string>({}))); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 365 |  | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 366 | updaterMain(&handler, &blobMock, image, signature, layout, updateIgnore); | 
| Patrick Venture | 1f09d41 | 2019-06-19 16:01:06 -0700 | [diff] [blame] | 367 | } | 
| Patrick Venture | 9b534f0 | 2018-12-13 16:10:02 -0800 | [diff] [blame] | 368 |  | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 369 | TEST_F(UpdaterTest, UpdateMainCleansUpOnFailure) | 
|  | 370 | { | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 371 | UpdateHandlerMock handler; | 
|  | 372 |  | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 373 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(true)); | 
|  | 374 | EXPECT_CALL(handler, sendFile(path, image)).WillOnce(Return()); | 
|  | 375 | EXPECT_CALL(handler, sendFile(ipmi_flash::hashBlobId, signature)) | 
|  | 376 | .WillOnce(Return()); | 
| Brandon Kim | 6749ba1 | 2019-09-19 13:31:37 -0700 | [diff] [blame] | 377 | EXPECT_CALL(handler, verifyFile(ipmi_flash::verifyBlobId, defaultIgnore)) | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 378 | .WillOnce(Return(false)); | 
|  | 379 | EXPECT_CALL(handler, cleanArtifacts()).WillOnce(Return()); | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 380 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 381 | .WillOnce(Return(std::vector<std::string>({}))); | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 382 |  | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 383 | EXPECT_THROW(updaterMain(&handler, &blobMock, image, signature, layout, | 
|  | 384 | defaultIgnore), | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 385 | ToolException); | 
|  | 386 | } | 
|  | 387 |  | 
|  | 388 | TEST_F(UpdaterTest, UpdateMainExceptsOnUpdateBlobFailure) | 
|  | 389 | { | 
|  | 390 | UpdateHandlerMock handler; | 
|  | 391 |  | 
|  | 392 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(true)); | 
|  | 393 | EXPECT_CALL(handler, sendFile(path, image)).WillOnce(Return()); | 
|  | 394 | EXPECT_CALL(handler, sendFile(ipmi_flash::hashBlobId, signature)) | 
|  | 395 | .WillOnce(Return()); | 
|  | 396 | EXPECT_CALL(handler, verifyFile(ipmi_flash::verifyBlobId, defaultIgnore)) | 
|  | 397 | .WillOnce(Return(true)); | 
|  | 398 | EXPECT_CALL(handler, verifyFile(ipmi_flash::updateBlobId, defaultIgnore)) | 
|  | 399 | .WillOnce(Return(false)); | 
|  | 400 | EXPECT_CALL(handler, cleanArtifacts()).WillOnce(Return()); | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 401 | EXPECT_CALL(blobMock, getBlobList()) | 
|  | 402 | .WillOnce(Return(std::vector<std::string>({}))); | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 403 |  | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 404 | EXPECT_THROW(updaterMain(&handler, &blobMock, image, signature, layout, | 
|  | 405 | defaultIgnore), | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 406 | ToolException); | 
|  | 407 | } | 
|  | 408 |  | 
|  | 409 | TEST_F(UpdaterTest, UpdateMainExceptsIfAvailableNotFound) | 
|  | 410 | { | 
|  | 411 | UpdateHandlerMock handler; | 
|  | 412 |  | 
|  | 413 | EXPECT_CALL(handler, checkAvailable(path)).WillOnce(Return(false)); | 
|  | 414 |  | 
| Willy Tu | 203ad80 | 2021-09-09 20:06:36 -0700 | [diff] [blame] | 415 | EXPECT_THROW(updaterMain(&handler, &blobMock, image, signature, layout, | 
|  | 416 | defaultIgnore), | 
| Patrick Venture | 8cdf964 | 2020-09-30 09:41:51 -0700 | [diff] [blame] | 417 | ToolException); | 
| Patrick Venture | 5f2fcc4 | 2019-06-20 07:21:05 -0700 | [diff] [blame] | 418 | } | 
|  | 419 |  | 
| Patrick Venture | 9b534f0 | 2018-12-13 16:10:02 -0800 | [diff] [blame] | 420 | } // namespace host_tool |