Change TestCompletion to only ever look inside of BaseDir
authorFrederic Riss <friss@apple.com>
Tue, 4 Sep 2018 23:09:49 +0000 (23:09 +0000)
committerFrederic Riss <friss@apple.com>
Tue, 4 Sep 2018 23:09:49 +0000 (23:09 +0000)
commit412f24dd2d987973b0b3be40957574e682bf1530
tree200ea474a15e16fb27f42b36c21f2531faa3e260
parent018ce562a9d8a7f3bab26ca140248fc00cea83d4
Change TestCompletion to only ever look inside of BaseDir

TestCompletion was failing quite frequently on our Linux bots. Some tracing
revealed that when we are iterating BaseDir we are not getting all the entries.
More specifically, we are sometimes missing the entry corresponding to the
TestCompletion directory that the first test in DirCompletionAbsolute is
looking for. BaseDir is the directory where lit is creating all the temporary
files. The semantics of opendir/readdir are unclear when it comes to iterating
over a directory that changes contents, but it seems like on Linux you might
fail to list an entry even if it was there before opendir and is still present
throughout the iteration. Changing the test to only look inside of the test-
specific directory seems to fix the instability.

This commit also removes some assertions that were added to try to track down
this issue.

llvm-svn: 341425
lldb/unittests/Interpreter/TestCompletion.cpp