commit b3e48bca920006d5543549d547ebd284c7b3b124
author Bob King <Bob_King@wistron.com> Tue Feb 18 09:59:09 2020 +0800
committer Shawn McCarney <shawnmm@us.ibm.com> Fri Feb 21 13:45:43 2020 +0000
tree aed3dc2210c27dc60d3de43e5f05eba66ecdc83a
parent 8215be3fcef2dc9e438ae09864be271889c7da82

regulators: Fix test case failures.

Fix test case failures caused by new validator logic to check for invalid rule IDs.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6655916fe0c499262d7ce40105930fea389f256a

phosphor-regulators/test/validate-regulators-config_tests.cpp

diff --git a/phosphor-regulators/test/validate-regulators-config_tests.cpp b/phosphor-regulators/test/validate-regulators-config_tests.cpp
index 24d5212..bb47c65 100644
--- a/phosphor-regulators/test/validate-regulators-config_tests.cpp
+++ b/phosphor-regulators/test/validate-regulators-config_tests.cpp
@@ -51,6 +51,20 @@
               }
             }
           ]
+        },
+        {
+          "comments": [ "Reads sensors from a PMBus regulator rail" ],
+          "id": "read_sensors_rule",
+          "actions": [
+            {
+              "comments": [ "Read output voltage from READ_VOUT." ],
+              "pmbus_read_sensor": {
+                "type": "vout",
+                "command": "0x8B",
+                "format": "linear_16"
+              }
+            }
+          ]
         }
       ],
 
@@ -2342,8 +2356,8 @@
     // Invalid: test duplicate ID in rule.
     {
         json configFile = validConfigFile;
-        configFile["rules"][1]["id"] = "set_voltage_rule";
-        configFile["rules"][1]["actions"][0]["pmbus_write_vout_command"]
+        configFile["rules"][2]["id"] = "set_voltage_rule";
+        configFile["rules"][2]["actions"][0]["pmbus_write_vout_command"]
                   ["format"] = "linear";
         EXPECT_JSON_INVALID(configFile, "Error: Duplicate rule ID.", "");
     }