commit fdeb864820b17a7b6d4e26c64f9fcad74fa2415f
author Patrick Venture <venture@google.com> Mon Jun 03 14:30:34 2019 -0700
committer Patrick Venture <venture@google.com> Tue Jun 04 18:15:09 2019 +0000
tree 5feaf5d367b3c519e1228aed7616413b0bd9b08f
parent e312f39b98d0f6568f7ce7ac3d5f06087ca58de3

test: firmware updateCompleted: commit(session)

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I186865bd5178e29b3df5a122fb9b33db0b9ba6d4

test/firmware_state_updatecompleted_unittest.cpp

diff --git a/test/firmware_state_updatecompleted_unittest.cpp b/test/firmware_state_updatecompleted_unittest.cpp
index 98b9a56..324a307 100644
--- a/test/firmware_state_updatecompleted_unittest.cpp
+++ b/test/firmware_state_updatecompleted_unittest.cpp
@@ -155,13 +155,33 @@
 }
 
 /*
+ * commit(session) - returns failure
+ */
+TEST_F(FirmwareHandlerUpdateCompletedTest,
+       CommitOnUpdateBlobAfterSuccessReturnsFailure)
+{
+    getToUpdateCompleted(ActionStatus::success);
+
+    EXPECT_CALL(*updateMockPtr, triggerUpdate()).Times(0);
+    EXPECT_FALSE(handler->commit(session, {}));
+}
+
+TEST_F(FirmwareHandlerUpdateCompletedTest,
+       CommitOnUpdateBlobAfterFailureReturnsFailure)
+{
+    getToUpdateCompleted(ActionStatus::failed);
+
+    EXPECT_CALL(*updateMockPtr, triggerUpdate()).Times(0);
+    EXPECT_FALSE(handler->commit(session, {}));
+}
+
+/*
  * There are the following calls (parameters may vary):
  * canHandleBlob(blob)
  * getBlobIds
  * deleteBlob(blob)
  * close(session)
  * read(session)
- * commit(session)
  */
 
 } // namespace