tree 08a5749615dd4feb4001b86306d05acd0c120c77
parent 20a045f6854efe2dd11d5bb717ba72093034c83b
author Shawn McCarney <shawnmm@us.ibm.com> 1736795362 -0600
committer Shawn McCarney <shawnmm@us.ibm.com> 1736906747 -0600

Fix CI error

The test case doUpdateOnePSUNotPresent() recently started failing in CI
on the meson pass that contains the following option:
* -Db_sanitize=address,undefined

This option turns on memory checking, and that results in slightly
different gtest execution.

There appears to be one or more gmock bugs when ON_CALL and EXPECT_CALL
are used on the same mock method. Modify doUpdateOnePSUNotPresent() to
use ON_CALL instead of EXPECT_CALL. This is consistent with the other
methods in this file.

Also wrap the SdBusMock objects in test cases with NiceMock<> to
simplify future test case debug. The test cases do not set many
expectations on the sdbusplus functions that are called, and this
results in a huge number of 'Uninteresting mock function call' warning
messages. Using NiceMock eliminates these warnings.

Tested:
* Verified all automated test cases pass

Change-Id: I9f7ef8f314378ad928579c3b5707aab4fa091a23
Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>