Skip to content

[Test] Optimize log module test cases (fix memory safety + improve coverage)#1

Open
JueBaby1120 wants to merge 1 commit intoZEMINGMA:masterfrom
JueBaby1120:pr1
Open

[Test] Optimize log module test cases (fix memory safety + improve coverage)#1
JueBaby1120 wants to merge 1 commit intoZEMINGMA:masterfrom
JueBaby1120:pr1

Conversation

@JueBaby1120
Copy link

优化目的
提升日志模块测试用例的健壮性、完整性和可维护性,修复潜在内存安全问题,确保日志功能测试覆盖核心场景。

核心改动

  1. 修复内存安全隐患:log_loop 函数添加空指针校验,避免非法解引用崩溃;
  2. 规范接口调用:给 LogTest::init 增加默认日志文件路径,解决参数声明与调用不一致问题;
  3. 增强测试覆盖:
    • 日志轮转测试:验证文件生成、轮转大小阈值、配置生效;
    • 日志级别测试:覆盖默认模块/指定模块/非默认模块场景;
  4. 优化资源管理:测试用例结束后显式销毁日志模块+清理测试文件,避免资源泄漏;
  5. 代码规范优化:统一命名风格,补充关键注释,提升可读性。

测试验证

  • 本地编译通过(基于 miniob 现有编译环境);
  • 所有测试用例执行通过(LogRotateTest.RotateBySize + LogLevelTest.CheckEnableStatus);
  • 无内存泄漏(valgrind 检测无 invalid read/write)。
    依赖说明
  • 依赖项目已有的 common/file/file.h 工具类(用于文件操作),无新增第三方依赖。

@ZEMINGMA
Copy link
Owner

ZEMINGMA commented Nov 21, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants