ProxySQL测试用例中的逻辑错误分析与修复

在ProxySQL数据库管理工具的持续集成测试过程中，开发团队发现了一个关于查询规则快速路由算法的测试用例存在逻辑错误。这个测试用例名为test_query_rules_fast_routing_algorithm-t，其核心功能是验证ProxySQL的查询路由功能是否按预期工作。

问题现象

测试用例在执行过程中报告了多个"not ok"错误，但令人困惑的是，错误信息中显示预期值(Exp)和实际值(Act)完全相同。例如，测试输出显示：

not ok 90 - 2025-02-14 23:26:45.518200 - Number of 'global' entries in SQLite3 'debug_log' should match issued queries - Exp: 20, Act: 20

从表面看，预期值和实际值都是20，测试应该通过而非失败。这种矛盾现象表明测试逻辑本身存在问题。

问题根源分析

通过检查测试代码发现，问题出在测试断言逻辑上。测试中使用了ok()宏来进行断言验证，其逻辑是检查匹配行数(matching_rows)是否等于查询范围差值(rng_end - rng_init)。然而，错误信息却显示的是匹配行数(matched_lines.size())与查询范围差值的比较。

这种不一致导致了测试逻辑与实际显示信息的矛盾。具体表现为：

1. 断言比较的是matching_rows == rng_end - rng_init
2. 但错误信息显示的是matched_lines.size()与rng_end - rng_init的比较

解决方案

开发团队通过以下方式修复了这个问题：

1. 统一断言比较和错误信息显示的内容，确保两者检查的是相同的指标
2. 修复了测试逻辑中的多个不一致问题
3. 改进了测试的其他方面，使其更加健壮和可靠

技术启示

这个案例展示了测试开发中几个重要的经验教训：

1. 断言与错误信息一致性：测试断言中比较的内容应该与错误信息中显示的内容完全一致，否则会导致混淆和难以诊断的问题。

2. 测试代码质量：即使是测试代码本身也需要保持高质量，因为它直接影响对产品功能的验证有效性。

3. 持续集成的重要性：通过持续集成系统能够及时发现这类问题，避免它们进入生产环境。

4. 测试可读性：清晰的测试输出对于快速定位问题至关重要，不一致的错误信息会大大增加调试难度。

ProxySQL作为高性能的MySQL代理，其测试套件的严谨性直接关系到产品的可靠性。这次修复不仅解决了一个具体的测试问题，也提高了整个测试框架的质量，为后续开发提供了更可靠的保障。