<div dir="ltr">Raphael, this sounds like a good idea.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Fr., 30. Aug. 2019 um 10:44 Uhr schrieb Raphael “Teemperor” Isemann via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
I have to admit I’m getting a bit confused lately where to put tests. Especially for testing LLDB commands it’s not obvious where to put files as we test some commands directly in the top-level test folder (e.g. quit, help, settings), some are in /functionalities with a _command suffix (e.g. target), some are in /functionalities without any suffix (e.g. register), some tests are split by subcommand (process, frame) and some are in the top-level folder with the _command prefix (e.g. expression). This makes it hard to figure out where to find or create tests for specific commands. Also setting a LIT_FILTER for jus testing CommandObject* changes is not possible.<br>
<br>
I would propose we restructure at least the command tests into “test/commands/${command_name}/${subcommand_name_or_functionality}/“ such as “test/commands/process/launch”. The LIT_FILTER for these things would be “commands/“.<br>
<br>
I don’t see any disadvantages from this as<br>
* downstreams usually doesn’t fiddle around with the existing tests, so there should hopefully be no merge conflicts from this.<br>
* git blame can handle this change as we only move files/directories and don’t touch their contents.<br>
* it’s very little work to actually do this.<br>
<br>
I’m not sure if there is a need to restructure any other tests but I think if there are no objections in this thread, then I assume everyone can just take a few seconds and restructure their own tests.<br>
<br>
Cheers,<br>
<br>
- Raphael<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>