willem/webserv
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: tests update StdoutChannel and logging methods to remove unnecessary parameters

Browse Source
This commit is contained in:
whaffman 2025-10-09 22:28:08 +02:00
parent 9c343d872e
commit fc1340ac7f
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/handler/test_handlers.cpp
View File

@ -1,7 +1,7 @@
#include <webserv/handler/FileHandler.hpp>
#include <webserv/handler/ErrorHandler.hpp>
#include <webserv/handler/MIMETypes.hpp>
#include <webserv/handler/URIParser.hpp>
#include <webserv/handler/URI.hpp>
#include <gtest/gtest.h>

10
tests/log/test_log.cpp
View File

@ -78,7 +78,7 @@ TEST_F(LogTest, LogLevelToColoredString)
TEST_F(LogTest, StdoutChannelConstruction)
{
StdoutChannel channel(Log::Level::Info);
StdoutChannel channel();
// If we reach here without exception, construction was successful
SUCCEED();
}
@ -115,16 +115,16 @@ TEST_F(LogTest, StaticLoggingMethodsWithContext)
TEST_F(LogTest, SetStdoutChannel)
{
EXPECT_NO_THROW(Log::setStdoutChannel(Log::Level::Debug));
EXPECT_NO_THROW(Log::setStdoutChannel());
// Setting again to see if it handles multiple calls
EXPECT_NO_THROW(Log::setStdoutChannel(Log::Level::Info));
EXPECT_NO_THROW(Log::setStdoutChannel());
}
TEST_F(LogTest, SetFileChannel)
{
EXPECT_NO_THROW(Log::setFileChannel("test_log.log", std::ios_base::trunc, Log::Level::Info));
EXPECT_NO_THROW(Log::setFileChannel("test_log.log", std::ios_base::trunc));
// Setting again to see if it handles multiple calls
EXPECT_NO_THROW(Log::setFileChannel("test_log.log", std::ios_base::app, Log::Level::Debug));
EXPECT_NO_THROW(Log::setFileChannel("test_log.log", std::ios_base::app));
}
// Note: More comprehensive tests would involve checking actual log outputs,