Skip to content

Commit faf307c

Browse files
committed
reduce specificity of expected publishEvent calls
1 parent c9d846b commit faf307c

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

test/unit_tests/ncp_fw_update/ncp_fw_update.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -727,7 +727,7 @@ TEST_CASE("SaraNcpFwUpdate") {
727727
//============================================
728728

729729
// Failed to publish case
730-
mocks.ExpectCallFunc(publishEvent).With("spark/device/ncp/update", "started", /*flags PUBLISH_EVENT_FLAG_PRIVATE*/1).Return(false);
730+
mocks.ExpectCallFunc(publishEvent).Return(false);
731731
REQUIRE(ncpTest.process() == SYSTEM_ERROR_NONE);
732732
REQUIRE(ncpTest.getSaraNcpFwUpdateState() == FW_UPDATE_STATE_FINISHED_IDLE);
733733
REQUIRE(ncpTest.getSaraNcpFwUpdateStatus() == FW_UPDATE_STATUS_FAILED);
@@ -748,7 +748,7 @@ TEST_CASE("SaraNcpFwUpdate") {
748748
ncpTest.setSaraNcpFwUpdateState(FW_UPDATE_STATE_SETUP_CLOUD_CONNECTED); // back to set startTimer again
749749
ncpTest.setSaraNcpFwUpdateStatus(FW_UPDATE_STATUS_DOWNLOADING);
750750
// Publish successful case
751-
mocks.ExpectCallFunc(publishEvent).With("spark/device/ncp/update", "started", /*flags PUBLISH_EVENT_FLAG_PRIVATE*/1).Return(true);
751+
mocks.ExpectCallFunc(publishEvent).Return(true);
752752
REQUIRE(ncpTest.process() == SYSTEM_ERROR_NONE);
753753
REQUIRE(ncpTest.getSaraNcpFwUpdateState() == FW_UPDATE_STATE_DOWNLOAD_CLOUD_DISCONNECT);
754754
REQUIRE(ncpTest.getSaraNcpFwUpdateStatus() == FW_UPDATE_STATUS_DOWNLOADING);
@@ -1320,7 +1320,7 @@ TEST_CASE("SaraNcpFwUpdate") {
13201320
ncpTest.setSaraNcpFwUpdateError(SYSTEM_ERROR_NONE);
13211321
ncpMocks.setSparkCloudFlagConnected(true);
13221322
// Failed to publish case
1323-
mocks.ExpectCallFunc(publishEvent).With("spark/device/ncp/update", "success", /*flags PUBLISH_EVENT_FLAG_PRIVATE*/1).Return(false);
1323+
mocks.ExpectCallFunc(publishEvent).Return(false);
13241324
REQUIRE(ncpTest.process() == SYSTEM_ERROR_NONE);
13251325
REQUIRE(ncpTest.getSaraNcpFwUpdateState() == FW_UPDATE_STATE_FINISHED_IDLE);
13261326
REQUIRE(ncpTest.getSaraNcpFwUpdateStatus() == FW_UPDATE_STATUS_FAILED);
@@ -1331,7 +1331,7 @@ TEST_CASE("SaraNcpFwUpdate") {
13311331
ncpTest.setSaraNcpFwUpdateStatus(FW_UPDATE_STATUS_FAILED); // simulated failure
13321332
ncpTest.setSaraNcpFwUpdateError(SYSTEM_ERROR_SARA_NCP_FW_UPDATE_POWER_OFF_TIMEOUT); // simulated error
13331333
// Publish a failed status case
1334-
mocks.ExpectCallFunc(publishEvent).With("spark/device/ncp/update", "failed", /*flags PUBLISH_EVENT_FLAG_PRIVATE*/1).Return(true);
1334+
mocks.ExpectCallFunc(publishEvent).Return(true);
13351335
REQUIRE(ncpTest.process() == SYSTEM_ERROR_NONE);
13361336
REQUIRE(ncpTest.getSaraNcpFwUpdateState() == FW_UPDATE_STATE_FINISHED_IDLE);
13371337
REQUIRE(ncpTest.getSaraNcpFwUpdateStatus() == FW_UPDATE_STATUS_FAILED);
@@ -1342,7 +1342,7 @@ TEST_CASE("SaraNcpFwUpdate") {
13421342
ncpTest.setSaraNcpFwUpdateStatus(FW_UPDATE_STATUS_SUCCESS);
13431343
ncpTest.setSaraNcpFwUpdateError(SYSTEM_ERROR_NONE);
13441344
// Publish a failed status case
1345-
mocks.ExpectCallFunc(publishEvent).With("spark/device/ncp/update", "success", /*flags PUBLISH_EVENT_FLAG_PRIVATE*/1).Return(true);
1345+
mocks.ExpectCallFunc(publishEvent).Return(true);
13461346
REQUIRE(ncpTest.process() == SYSTEM_ERROR_NONE);
13471347
REQUIRE(ncpTest.getSaraNcpFwUpdateState() == FW_UPDATE_STATE_FINISHED_IDLE);
13481348
REQUIRE(ncpTest.getSaraNcpFwUpdateStatus() == FW_UPDATE_STATUS_SUCCESS);

0 commit comments

Comments
 (0)