add .clang-format
Change-Id: I94ce26d595367e08d6fb3734535bcd855f1b1473
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/test/fanpwm_unittest.cpp b/test/fanpwm_unittest.cpp
index 63520d3..11dcee9 100644
--- a/test/fanpwm_unittest.cpp
+++ b/test/fanpwm_unittest.cpp
@@ -1,11 +1,11 @@
#include "fan_pwm.hpp"
-
#include "hwmonio_mock.hpp"
+#include <sdbusplus/test/sdbus_mock.hpp>
+#include <string>
+
#include <gmock/gmock.h>
#include <gtest/gtest.h>
-#include <sdbusplus/test/sdbus_mock.hpp>
-#include <string>
using ::testing::_;
using ::testing::Invoke;
@@ -19,55 +19,38 @@
// Handle basic expectations we'll need for all these tests, if it's found that
// this is helpful for more tests, it can be promoted in scope.
-void SetupDbusObject(
- sdbusplus::SdBusMock *sdbus_mock,
- const std::string& path,
- const std::string& intf,
- const std::string property = "")
+void SetupDbusObject(sdbusplus::SdBusMock *sdbus_mock, const std::string &path,
+ const std::string &intf, const std::string property = "")
{
EXPECT_CALL(*sdbus_mock,
- sd_bus_add_object_vtable(
- IsNull(),
- NotNull(),
- StrEq(path),
- StrEq(intf),
- NotNull(),
- NotNull()))
- .WillOnce(Return(0));
+ sd_bus_add_object_vtable(IsNull(), NotNull(), StrEq(path),
+ StrEq(intf), NotNull(), NotNull()))
+ .WillOnce(Return(0));
if (property.empty())
{
EXPECT_CALL(*sdbus_mock,
- sd_bus_emit_properties_changed_strv(
- IsNull(),
- StrEq(path),
- StrEq(intf),
- NotNull()))
- .WillOnce(Return(0));
+ sd_bus_emit_properties_changed_strv(IsNull(), StrEq(path),
+ StrEq(intf), NotNull()))
+ .WillOnce(Return(0));
}
else
{
EXPECT_CALL(*sdbus_mock,
- sd_bus_emit_properties_changed_strv(
- IsNull(),
- StrEq(path),
- StrEq(intf),
- NotNull()))
- .WillOnce(
- Invoke([=](sd_bus *bus,
- const char *path,
- const char *interface,
- char **names) {
+ sd_bus_emit_properties_changed_strv(IsNull(), StrEq(path),
+ StrEq(intf), NotNull()))
+ .WillOnce(Invoke([=](sd_bus *bus, const char *path,
+ const char *interface, char **names) {
EXPECT_STREQ(property.c_str(), names[0]);
return 0;
- })
- );
+ }));
}
return;
}
-TEST(FanPwmTest, BasicConstructorDeferredTest) {
+TEST(FanPwmTest, BasicConstructorDeferredTest)
+{
// Attempt to just instantiate one.
// NOTE: This test's goal is to figure out what's minimally required to
@@ -87,16 +70,12 @@
SetupDbusObject(&sdbus_mock, objPath, FanPwmIntf, FanPwmProp);
- hwmon::FanPwm f(std::move(hwmonio_mock),
- devPath,
- id,
- bus_mock,
- objPath.c_str(),
- defer,
- target);
+ hwmon::FanPwm f(std::move(hwmonio_mock), devPath, id, bus_mock,
+ objPath.c_str(), defer, target);
}
-TEST(FanPwmTest, BasicConstructorNotDeferredTest) {
+TEST(FanPwmTest, BasicConstructorNotDeferredTest)
+{
// Attempt to just instantiate one.
// NOTE: This test's goal is to figure out what's minimally required to
@@ -116,24 +95,18 @@
SetupDbusObject(&sdbus_mock, objPath, FanPwmIntf, FanPwmProp);
- EXPECT_CALL(sdbus_mock,
- sd_bus_emit_object_added(IsNull(), StrEq("asdf")))
- .WillOnce(Return(0));
+ EXPECT_CALL(sdbus_mock, sd_bus_emit_object_added(IsNull(), StrEq("asdf")))
+ .WillOnce(Return(0));
- EXPECT_CALL(sdbus_mock,
- sd_bus_emit_object_removed(IsNull(), StrEq("asdf")))
- .WillOnce(Return(0));
+ EXPECT_CALL(sdbus_mock, sd_bus_emit_object_removed(IsNull(), StrEq("asdf")))
+ .WillOnce(Return(0));
- hwmon::FanPwm f(std::move(hwmonio_mock),
- devPath,
- id,
- bus_mock,
- objPath.c_str(),
- defer,
- target);
+ hwmon::FanPwm f(std::move(hwmonio_mock), devPath, id, bus_mock,
+ objPath.c_str(), defer, target);
}
-TEST(FanPwmTest, WriteTargetValue) {
+TEST(FanPwmTest, WriteTargetValue)
+{
// Create a FanPwm and write a value to the object.
sdbusplus::SdBusMock sdbus_mock;
@@ -154,43 +127,29 @@
hwmonio::HwmonIOMock *hwmonio =
reinterpret_cast<hwmonio::HwmonIOMock *>(hwmonio_mock.get());
- hwmon::FanPwm f(std::move(hwmonio_mock),
- devPath,
- id,
- bus_mock,
- objPath.c_str(),
- defer,
- target);
+ hwmon::FanPwm f(std::move(hwmonio_mock), devPath, id, bus_mock,
+ objPath.c_str(), defer, target);
target = 0x64;
- EXPECT_CALL(*hwmonio, write(static_cast<uint32_t>(target),
- StrEq("pwm"),
- StrEq("the_id"),
- _,
- hwmonio::retries,
- hwmonio::delay));
+ EXPECT_CALL(*hwmonio,
+ write(static_cast<uint32_t>(target), StrEq("pwm"),
+ StrEq("the_id"), _, hwmonio::retries, hwmonio::delay));
EXPECT_CALL(sdbus_mock,
sd_bus_emit_properties_changed_strv(
- IsNull(),
- StrEq("asdf"),
- StrEq(FanPwmIntf),
- NotNull()))
- .WillOnce(
- Invoke([&](sd_bus *bus,
- const char *path,
- const char *interface,
- char **names) {
+ IsNull(), StrEq("asdf"), StrEq(FanPwmIntf), NotNull()))
+ .WillOnce(Invoke([&](sd_bus *bus, const char *path,
+ const char *interface, char **names) {
EXPECT_EQ(0, strncmp("Target", names[0], 6));
return 0;
- })
- );
+ }));
EXPECT_EQ(target, f.target(target));
}
-TEST(FanPwmTest, WriteTargetValueNoUpdate) {
+TEST(FanPwmTest, WriteTargetValueNoUpdate)
+{
// Create a FanPwm and write a value to the object that was the previous
// value.
@@ -209,13 +168,8 @@
SetupDbusObject(&sdbus_mock, objPath, FanPwmIntf, FanPwmProp);
- hwmon::FanPwm f(std::move(hwmonio_mock),
- devPath,
- id,
- bus_mock,
- objPath.c_str(),
- defer,
- target);
+ hwmon::FanPwm f(std::move(hwmonio_mock), devPath, id, bus_mock,
+ objPath.c_str(), defer, target);
EXPECT_EQ(target, f.target(target));
}