hyprwm/hyprutils
1
0
Fork 0
mirror of https://github.com/hyprwm/hyprutils.git synced 2025-12-24 17:10:03 +01:00
Code Issues Projects Releases Packages Wiki Activity

things

Browse source
This commit is contained in:
Vaxry 2025-11-23 01:17:27 +00:00
parent eb27ac56e3
commit fe829f2913
Signed by: vaxry
GPG key ID: 665806380871D640
3 changed files with 28 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/cli/ArgumentParser.cpp
View file

@ -99,7 +99,8 @@ std::vector<SArgumentKey>::iterator CArgumentParserImpl::getValue(const std::str
return it; return it;
} }
std::expected<void, std::string> CArgumentParserImpl::registerOption(const std::string_view& name, const std::string_view& abbrev, const std::string_view& description, eArgumentType type) { std::expected<void, std::string> CArgumentParserImpl::registerOption(const std::string_view& name, const std::string_view& abbrev, const std::string_view& description,
eArgumentType type) {
if (getValue(name) != m_values.end() || getValue(abbrev) != m_values.end()) if (getValue(name) != m_values.end() || getValue(abbrev) != m_values.end())
return std::unexpected("Value already exists"); return std::unexpected("Value already exists");

4
src/cli/Logger.cpp
View file

@ -6,7 +6,7 @@
using namespace Hyprutils; using namespace Hyprutils;
using namespace Hyprutils::CLI; using namespace Hyprutils::CLI;
CLogger::CLogger() { CLogger::CLogger() {
m_impl = Memory::makeUnique<CLoggerImpl>(this); m_impl = Memory::makeUnique<CLoggerImpl>(this);
} }
@ -21,7 +21,7 @@ void CLogger::setTime(bool enabled) {
} }
void CLogger::setEnableStdout(bool enabled) { void CLogger::setEnableStdout(bool enabled) {
m_impl->m_stdoutEnabled; m_impl->m_stdoutEnabled = enabled;
m_impl->updateParentShouldLog(); m_impl->updateParentShouldLog();
} }

24
tests/cli/Logger.cpp
View file

@ -24,6 +24,12 @@ TEST(CLI, Logger) {
EXPECT_EQ(logger.rollingLog(), "DEBUG ]: Hello!\nTRACE ]: Hello, Trace!"); EXPECT_EQ(logger.rollingLog(), "DEBUG ]: Hello!\nTRACE ]: Hello, Trace!");
logger.setEnableStdout(false);
logger.log(Hyprutils::CLI::LOG_ERR, "Error");
EXPECT_EQ(logger.rollingLog(), "DEBUG ]: Hello!\nTRACE ]: Hello, Trace!");
auto res = logger.setOutputFile("./loggerFile.log"); auto res = logger.setOutputFile("./loggerFile.log");
EXPECT_TRUE(res); EXPECT_TRUE(res);
@ -39,4 +45,22 @@ TEST(CLI, Logger) {
std::error_code ec; std::error_code ec;
std::filesystem::remove("./loggerFile.log", ec); std::filesystem::remove("./loggerFile.log", ec);
// TODO: maybe find a way to test the times and color?
logger.setEnableStdout(true);
logger.setTime(true);
logger.log(Hyprutils::CLI::LOG_WARN, "Timed warning!");
logger.setEnableColor(false);
logger.log(Hyprutils::CLI::LOG_CRIT, "rip");
// spam some logs to check rolling
for (size_t i = 0; i < 1000; ++i) {
logger.log(LOG_DEBUG, "Log log log!");
}
EXPECT_TRUE(logger.rollingLog().size() < 4096);
EXPECT_TRUE(logger.rollingLog().starts_with("DEBUG")); // test the breaking is done correctly
} }