• Binbin's avatar
    Fix timing issue in new ACL log test (#11781) · 5a3cdddd
    Binbin authored
    There is a timing issue in the new ACL log test:
    ```
    *** [err]: ACL LOG aggregates similar errors together and assigns unique entry-id to new errors in tests/unit/acl.tcl
    Expected 1675382873989 < 1675382873989 (context: type eval line 15 cmd {assert {$timestamp_last_update_original < $timestamp_last_updated_after_update}} proc ::test)
    ```
    
    Looking at the test code, we will check the `timestamp-last-updated` before
    and after a new ACL error occurs. Actually `WRONGPASS` errors can be executed
    very quickly on fast machines. For example, in the this case, the execution is
    completed within one millisecond.
    
    The error is easy to reproduce, if we reduce the number of the for loops, for
    example set to 2, and using --loop and --stop. Avoid this timing issue by adding
    an `after 1` before the new errors.
    
    The test was introduced in #11477.
    5a3cdddd
acl.tcl 41.2 KB