[PATCH] D91102: [clangd] Enhance Clangd rename testing coverage

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev created this revision.
kbobyrev added a reviewer: hokein.
Herald added subscribers: cfe-commits, usaxena95, arphaman.
Herald added a project: clang.
kbobyrev requested review of this revision.
Herald added subscribers: MaskRay, ilya-biryukov.

We plan to eliminate error-prone and obsolete Clang-Rename API from Clangd. To
do that, we will introduce Decl canonicalization rules that will make renaming
code simpler and easier to maintain (D71880 <https://reviews.llvm.org/D71880>).

To ensure smooth transition to the new implementation, many Clang-Rename tests
will be moved to Clangd to prevent any possible regressions. This patch is the
first in the chain of test migration patches. It improves existing tests,
abstracts GoogleTest fixture for future usage and validates the approach. Other
tests will be ported in the subsequent patches.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91102

Files:
  clang-tools-extra/clangd/unittests/RenameTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91102.303978.patch
Type: text/x-patch
Size: 7840 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201109/ec01b9b0/attachment.bin>