From 6dc96680f1e9db0d1c87ab26865a27ac80cefd3d Mon Sep 17 00:00:00 2001 From: Vishwanatha Subbanna Date: Mon, 6 Feb 2017 21:47:18 +0530 Subject: Add gtest cases to verify sd_event timer Change-Id: I6db746adc969a322f463c00143938c7c2339ff86 Signed-off-by: Vishwanatha Subbanna --- softoff/Makefile.am | 2 + softoff/test/Makefile.am | 12 ++++ softoff/test/utest.cpp | 164 +++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 178 insertions(+) create mode 100644 softoff/test/Makefile.am create mode 100644 softoff/test/utest.cpp (limited to 'softoff') diff --git a/softoff/Makefile.am b/softoff/Makefile.am index aafce74..b8266b3 100644 --- a/softoff/Makefile.am +++ b/softoff/Makefile.am @@ -25,3 +25,5 @@ xyz/openbmc_project/Ipmi/Internal/SoftPowerOff/server.cpp: ${top_srcdir}/xyz/ope xyz/openbmc_project/Ipmi/Internal/SoftPowerOff/server.hpp: ${top_srcdir}/xyz/openbmc_project/Ipmi/Internal/SoftPowerOff.interface.yaml @mkdir -p `dirname $@` $(SDBUSPLUSPLUS) -r $(top_srcdir) interface server-header xyz.openbmc_project.Ipmi.Internal.SoftPowerOff > $@ + +SUBDIRS = test diff --git a/softoff/test/Makefile.am b/softoff/test/Makefile.am new file mode 100644 index 0000000..2c65568 --- /dev/null +++ b/softoff/test/Makefile.am @@ -0,0 +1,12 @@ +AM_CPPFLAGS = -I$(top_srcdir)/softoff + +# Run all 'check' test programs +TESTS = $(check_PROGRAMS) + +# # Build/add utest to test suite +check_PROGRAMS = utest +utest_CPPFLAGS = -Igtest $(GTEST_CPPFLAGS) $(AM_CPPFLAGS) +utest_CXXFLAGS = $(PTHREAD_CFLAGS) +utest_LDFLAGS = -lgtest_main -lgtest $(PTHREAD_LIBS) $(OESDK_TESTCASE_FLAGS) $(SYSTEMD_LIBS) ${SDBUSPLUS_LIBS} +utest_SOURCES = utest.cpp +utest_LDADD = $(top_builddir)/softoff/timer.o diff --git a/softoff/test/utest.cpp b/softoff/test/utest.cpp new file mode 100644 index 0000000..a9dae9c --- /dev/null +++ b/softoff/test/utest.cpp @@ -0,0 +1,164 @@ +#include +#include +#include +#include "timer.hpp" + +using namespace phosphor::ipmi; + +class TimerTest : public ::testing::Test +{ + public: + // systemd event handler + sd_event* events; + + // Need this so that events can be initialized. + int rc; + + // Source of event + sd_event_source* eventSource; + + // Add a Timer Object + Timer timer; + + // Gets called as part of each TEST_F construction + TimerTest() + : rc(sd_event_default(&events)), + timer(events) + { + // Check for successful creation of + // event handler and timer object. + EXPECT_GE(rc, 0); + } + + // Gets called as part of each TEST_F destruction + ~TimerTest() + { + events = sd_event_unref(events); + } +}; + +/** @brief Makes sure that timer is expired and the + * callback handler gets invoked post 2 seconds + */ +TEST_F(TimerTest, timerExpiresAfter2seconds) +{ + using namespace std::chrono; + + auto time = duration_cast(seconds(2)); + EXPECT_GE(timer.startTimer(time), 0); + + // Waiting 2 seconds is enough here since we have + // already spent some usec now + int count = 0; + while(count < 2 && !timer.isExpired()) + { + // Returns -0- on timeout and positive number on dispatch + auto sleepTime = duration_cast(seconds(1)); + if(!sd_event_run(events, sleepTime.count())) + { + count++; + } + } + EXPECT_EQ(true, timer.isExpired()); + EXPECT_EQ(1, count); +} + +/** @brief Makes sure that timer is not expired + */ +TEST_F(TimerTest, timerNotExpiredAfter2Seconds) +{ + using namespace std::chrono; + + auto time = duration_cast(seconds(2)); + EXPECT_GE(timer.startTimer(time), 0); + + // Now turn off the timer post a 1 second sleep + sleep(1); + EXPECT_GE(timer.setTimer(SD_EVENT_OFF), 0); + + // Wait 2 seconds and see that timer is not expired + int count = 0; + while(count < 2) + { + // Returns -0- on timeout + auto sleepTime = duration_cast(seconds(1)); + if(!sd_event_run(events, sleepTime.count())) + { + count++; + } + } + EXPECT_EQ(false, timer.isExpired()); + + // 2 because of one more count that happens prior to exiting + EXPECT_EQ(2, count); +} + +/** @brief Makes sure that timer value is changed in between + * and that the new timer expires + */ +TEST_F(TimerTest, updateTimerAndExpectExpire) +{ + using namespace std::chrono; + + auto time = duration_cast(seconds(2)); + EXPECT_GE(timer.startTimer(time), 0); + + // Now sleep for a second and then set the new timeout value + sleep(1); + + // New timeout is 3 seconds from THIS point. + time = duration_cast(seconds(3)); + EXPECT_GE(timer.startTimer(time), 0); + + // Wait 3 seconds and see that timer is expired + int count = 0; + while(count < 3 && !timer.isExpired()) + { + // Returns -0- on timeout + auto sleepTime = duration_cast(seconds(1)); + if(!sd_event_run(events, sleepTime.count())) + { + count++; + } + } + EXPECT_EQ(true, timer.isExpired()); + EXPECT_EQ(2, count); +} + +/** @brief Makes sure that timer value is changed in between + * and turn off and make sure that timer does not expire + */ +TEST_F(TimerTest, updateTimerAndNeverExpire) +{ + using namespace std::chrono; + + auto time = duration_cast(seconds(2)); + EXPECT_GE(timer.startTimer(time), 0); + + // Now sleep for a second and then set the new timeout value + sleep(1); + + // New timeout is 2 seconds from THIS point. + time = duration_cast(seconds(2)); + EXPECT_GE(timer.startTimer(time), 0); + + // Now turn off the timer post a 1 second sleep + sleep(1); + EXPECT_GE(timer.setTimer(SD_EVENT_OFF), 0); + + // Wait 2 seconds and see that timer is expired + int count = 0; + while(count < 2) + { + // Returns -0- on timeout + auto sleepTime = duration_cast(seconds(1)); + if(!sd_event_run(events, sleepTime.count())) + { + count++; + } + } + EXPECT_EQ(false, timer.isExpired()); + + // 2 becase of one more count that happens prior to exiting + EXPECT_EQ(2, count); +} -- cgit v1.2.1