<div dir="ltr">Sorry about that. I wasn't familiar with the python bindings.<div>Your bisect is correct, we changed the behavior that test is testing: now an approximate match will be returned.</div><div><br><div>I guess either:</div><div> - just remove that test (depending on how we feel about smoke-testing vs exhaustively testing wrapperl like this)</div></div><div> - change it to verify exactly what the fuzzy-matched result is, e.g.</div><div>    - add -DFEATURE=0 to the first compile command for "Project.cpp" in the test CDB so it's easy to recognize</div><div>    - make the lookup key "/home/john.doe/MyProject/Project.h" so it's obvious what the match should be</div><div>    - verify that the lookup result has "-DFEATURE=0" and "Project.h" in its args<br></div><div>It seems to me that test_lookup_succeed should also verify what gets returned (otherwise it's not really testing anything at this point).</div><div><br></div><div>I can try to put together a patch, though I don't currently have a setup to test it - let me know if this would be useful!</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 12, 2018 at 8:38 AM Michał Górny via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mgorny added a comment.<br>
<br>
Unless my bisect is mistaken, this broke Clang Python binding test:<br>
<a href="https://github.com/llvm-mirror/clang/blob/master/bindings/python/tests/cindex/test_cdb.py#L34" rel="noreferrer" target="_blank">https://github.com/llvm-mirror/clang/blob/master/bindings/python/tests/cindex/test_cdb.py#L34</a><br>
<br>
The test apparently assumes that compilation database will not return anything for a file that does not exist. Could you please suggest how the test should be adjusted for this change?<br>
<br>
<br>
Repository:<br>
  rC Clang<br>
<br>
<a href="https://reviews.llvm.org/D51729" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51729</a><br>
<br>
<br>
<br>
</blockquote></div>