test: sensors: host
Tests for sensors/host
Change-Id: I760825c666c711d6f9c394ceefe35f8151383785
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/test/sensor_host_unittest.cpp b/test/sensor_host_unittest.cpp
new file mode 100644
index 0000000..5e8af4b
--- /dev/null
+++ b/test/sensor_host_unittest.cpp
@@ -0,0 +1,129 @@
+#include "sensors/host.hpp"
+
+#include <chrono>
+#include <gmock/gmock.h>
+#include <gtest/gtest.h>
+#include <memory>
+#include <sdbusplus/test/sdbus_mock.hpp>
+#include <string>
+#include <vector>
+
+#include "test/helpers.hpp"
+
+using ::testing::IsNull;
+using ::testing::Return;
+using ::testing::StrEq;
+
+TEST(HostSensorTest, BoringConstructorTest) {
+ // WARN: The host sensor is not presently meant to be created this way,
+ // TODO: Can I move the constructor into private?
+}
+
+TEST(HostSensorTest, CreateHostTempSensorTest) {
+ // The normal case for this sensor is to be a temperature sensor, where
+ // the value is treated as a margin sensor.
+
+ sdbusplus::SdBusMock sdbus_mock;
+ auto bus_mock = sdbusplus::get_mocked_new(&sdbus_mock);
+ std::string name = "fleeting0";
+ int64_t timeout = 1;
+ const char *objPath = "/asdf/asdf0";
+ bool defer = false;
+ std::string interface = "xyz.openbmc_project.Sensor.Value";
+
+ // Scale is the only property we change in the code. Also,
+ // emit_object_added isn't called twice.
+ // Temperature is the default for type, and everything, so those aren't
+ // updated.
+ std::vector<std::string> properties = {"Scale"};
+ int i;
+
+ // The CreateTemp updates all the properties, however, only Scale is set
+ // to non-default.
+ SetupDbusObject(
+ &sdbus_mock,
+ defer,
+ objPath,
+ interface,
+ properties,
+ &i);
+
+ // This is called during object destruction.
+ EXPECT_CALL(sdbus_mock,
+ sd_bus_emit_object_removed(IsNull(), StrEq(objPath)))
+ .WillOnce(Return(0));
+
+ std::unique_ptr<Sensor> s = HostSensor::CreateTemp(
+ name, timeout, bus_mock, objPath, defer);
+}
+
+TEST(HostSensorTest, VerifyWriteThenReadMatches) {
+ // Verify that when value is updated, the information matches
+ // what we expect when read back.
+
+ sdbusplus::SdBusMock sdbus_mock;
+ auto bus_mock = sdbusplus::get_mocked_new(&sdbus_mock);
+ std::string name = "fleeting0";
+ int64_t timeout = 1;
+ const char *objPath = "/asdf/asdf0";
+ bool defer = false;
+ std::string interface = "xyz.openbmc_project.Sensor.Value";
+
+ // Scale is the only property we change in the code. Also,
+ // emit_object_added isn't called twice.
+ // Temperature is the default for type, and everything, so those aren't
+ // updated.
+ std::vector<std::string> properties = {"Scale"};
+ int i;
+
+ SetupDbusObject(
+ &sdbus_mock,
+ defer,
+ objPath,
+ interface,
+ properties,
+ &i);
+
+ EXPECT_CALL(sdbus_mock,
+ sd_bus_emit_object_removed(IsNull(), StrEq(objPath)))
+ .WillOnce(Return(0));
+
+ std::unique_ptr<Sensor> s = HostSensor::CreateTemp(
+ name, timeout, bus_mock, objPath, defer);
+
+ // Value is updated from dbus calls only (normally).
+ HostSensor *hs = static_cast<HostSensor *>(s.get());
+ int64_t new_value = 2;
+
+ ReadReturn r = hs->read();
+ EXPECT_EQ(r.value, 0);
+
+ EXPECT_CALL(sdbus_mock,
+ sd_bus_emit_properties_changed_strv(
+ IsNull(),
+ StrEq(objPath),
+ StrEq(interface),
+ NotNull()))
+ .WillOnce(
+ Invoke([=](sd_bus *bus,
+ const char *path,
+ const char *interface,
+ char **names) {
+ EXPECT_STREQ("Value", names[0]);
+ return 0;
+ })
+ );
+
+ std::chrono::high_resolution_clock::time_point t1 =
+ std::chrono::high_resolution_clock::now();
+
+ hs->value(new_value);
+ r = hs->read();
+ EXPECT_EQ(r.value, new_value * 0.001);
+
+ auto duration = std::chrono::duration_cast<std::chrono::seconds>(
+ t1 - r.updated).count();
+
+ // Verify it was updated within the last second.
+ EXPECT_TRUE(duration < 1);
+}