Ver Fonte

Adding flush test.

Sam Jaffe há 6 anos atrás
pai
commit
db23e2f659
2 ficheiros alterados com 23 adições e 0 exclusões
  1. 13 0
      test/c_logger_test.cxx
  2. 10 0
      test/logger_test.cxx

+ 13 - 0
test/c_logger_test.cxx

@@ -20,13 +20,25 @@ struct t_logger : public c_logger {
 
 using CLoggerTest = LoggerTest;
 
+TEST_F(CLoggerTest, FlushesOnClose) {
+  EXPECT_CALL(*pimpl, flush()).Times(1);
+  t_logger("", pimpl);
+}
+
+TEST_F(CLoggerTest, FlushesOnFlushCall) {
+  EXPECT_CALL(*pimpl, flush()).Times(2);
+  t_logger("", pimpl).flush();
+}
+
 TEST_F(CLoggerTest, LogsWithFmtCode) {
   using testing::_;
   using testing::HasSubstr;
+  // TODO: Eq
   EXPECT_CALL(*pimpl, write(_, HasSubstr("5"))).Times(1);
   t_logger("", pimpl).errorf("%d", 5);
 }
 
+// TODO: This is wrong
 TEST_F(CLoggerTest, FmtLogHasNameInHeader) {
   using testing::_;
   using testing::HasSubstr;
@@ -34,6 +46,7 @@ TEST_F(CLoggerTest, FmtLogHasNameInHeader) {
   t_logger("TEST", pimpl).errorf("%d", 5);
 }
 
+// TODO: This is wrong
 TEST_F(CLoggerTest, FmtLogHasLevelInHeader) {
   using testing::_;
   using testing::HasSubstr;

+ 10 - 0
test/logger_test.cxx

@@ -18,6 +18,16 @@ struct t_logger : public logger {
 };
 }
 
+TEST_F(LoggerTest, FlushesOnClose) {
+  EXPECT_CALL(*pimpl, flush()).Times(1);
+  t_logger("", pimpl);
+}
+
+TEST_F(LoggerTest, FlushesOnFlushCall) {
+  EXPECT_CALL(*pimpl, flush()).Times(2);
+  t_logger("", pimpl).flush();
+}
+
 TEST_F(LoggerTest, LogsWithBraceFmtCode) {
   using testing::Field;
   EXPECT_CALL(*pimpl, write(Field(&logpacket::message, "5"))).Times(1);