tree 5d14fe03f9301308a16aa4e5fa513cd08b77e813
parent 84614882e7b15f97c6de899506f3be827c67748f
author Brandon Wyman <bjwyman@gmail.com> 1589405158 -0500
committer Shawn McCarney <shawnmm@us.ibm.com> 1589840072 +0000

psu-ng: Updates for clearFaults

It was discovered that the power supplies may not always have the
in1_crit "file" available in the HWMON directory. This file may be
missing if certain commands are not supported. It was also discovered
that it can be missing if the power supply falsely indicates it supports
PEC, but actually does not. Changing the readString call to read from
the in1_input file, which typically should be present (READ_VIN).

Move the clearFaults function from the hpp file to the cpp file, as:
1. It is getting a bit lengthy.
2. Adding a try/catch for ReadFailure to avoid crashing the application
   if there is a ReadFailure, which is in the phosphor::logging
   namespace already used in the cpp file.

Update the test case to expect a read from in1_input, and return a value
that more closely resembles an actual READ_VIN value.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I9747317c68040cdce6bb80922e3928be55376c44