On Fri, Jun 02, 2023 at 10:11:36AM +0200, Tim Düsterhus wrote: > Hi > > On 6/2/23 08:42, Willy Tarreau wrote: > > Thank you for this. I've added a comment in the file about it, and a > > regtest to detect when this happens, since (null) appears in the header > > line of the "show stat" output in this case. > > Nice, thank you. I thought about including a test for that, but didn't have > the time to think about how to properly test this. The CLI-based test is > also much cleaner than any Lua-based test I would likely have created.
Any test would fit anyway. What we want is just a safety belt to detect that the developer missed one place to update. If we face other similar ones in the future, that could be a nice way to add other similar guards against developers mistakes. Willy
