[Lldb-commits] [lldb] [lldb][test] Remove duplicate testcase names in API test-suite (PR #97043)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 28 11:37:45 PDT 2024


JDevlieghere wrote:

> > > LGTM. This has definitely come up in the past. If you feel motivated, I'm sure there must be a way to detect this issue in Python and we could have assert/warning/error that captures this at the dotest level.
> > 
> > 
> > Agreed, making it part of `dotest` would be amazing. Maybe someone with better python knowledge has some ideas (@medismailben @kastiglione ?). In the meantime I'll have a think of how one might do that
> 
> I think instead of having this be part of our test-suite, I'd run it as part of PR testing in a GitHub action

The test suite would/will run as part of PR testing. Are you saying you would **only** run it at PR time? Why wait and not find out at your test? We already collect all the test names to run them as separate lit tests (similar but different problem) and I bet we also already iterate over the tests inside a file too to build the variants. 

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


More information about the lldb-commits mailing list