• Binbin's avatar
    Fix module redact test for valgrind (#10432) · 61b7e591
    Binbin authored
    The new module redact test will fail with valgrind:
    ```
    [err]: modules can redact arguments in tests/unit/moduleapi/auth.tcl
    Expected 'slowlog reset' to be equal to 'auth.redact 1 (redacted) 3 (redacted)' (context: type eval line 12 cmd {assert_equal {slowlog reset} [lindex [lindex [r slowlog get] 2] 3]} proc ::test)
    ```
    
    The reason is that with `slowlog-log-slower-than 10000`,
    `slowlog get` will have a chance to exceed 10ms.
    
    Made two changes to avoid failure:
    1. change `slowlog-log-slower-than` from 10000 to -1, distable it.
    2. assert to use the previous execution result.
    
    In theory, the second one can actually be left unchanged, but i
    think it will be better if it is changed.
    61b7e591
auth.tcl 2.97 KB