[PATCH] D53202: [python] [tests] Remove cdb lookup failure test

2018-10-12 Thread Michał Górny via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL344364: [python] [tests] Remove cdb lookup failure test (authored by mgorny, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D53202: [python] [tests] Remove cdb lookup failure test

2018-10-12 Thread Michał Górny via Phabricator via cfe-commits
mgorny added a comment. Thanks for the review! Repository: rC Clang https://reviews.llvm.org/D53202 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D53202: [python] [tests] Remove cdb lookup failure test

2018-10-12 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. We could verify the fuzzy-matched results, but we have pretty good testing on the C++ side, so a smoke test here seems fine. Repository: rC Clang https://reviews.llvm.org/D53202

[PATCH] D53202: [python] [tests] Remove cdb lookup failure test

2018-10-12 Thread Michał Górny via Phabricator via cfe-commits
mgorny created this revision. mgorny added reviewers: sammccall, bkramer, aadg. Herald added a subscriber: arphaman. Remove the test checking for compilation db lookup failure. Since r342228, JSONCompilationDatabasePlugin infers compile commands for missing files, therefore making the lookup