[Lldb-commits] [lldb] Make sure that a `Progress` "completed" update is always reported at destruction (PR #102097)

via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 6 09:51:07 PDT 2024


royitaqi wrote:

> LGTM. Could we cover this scenario by the existing unit test?

Yes, definitely.

At the time of the patch I didn't find any existing unit test files by searching for "ProgressTest.cpp". Just now I realize there is the [ProgressReportTest.cpp](https://github.com/llvm/llvm-project/blob/main/lldb/unittests/Core/ProgressReportTest.cpp). I will read it and try to add a test for this scenario.

https://github.com/llvm/llvm-project/pull/102097


More information about the lldb-commits mailing list