[PATCH] D89297: [clangd] Add a TestWorkspace utility

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 15 10:22:46 PDT 2020


nridge added a comment.

In D89297#2326838 <https://reviews.llvm.org/D89297#2326838>, @nridge wrote:

> In principle, it should be possible to make the same interface work with `FileIndex` under the hood, but I haven't figured out how. (Specifically, I haven't figured out how to get `FileIndex` to resolve `#include`s, because you can't pass a `MockFS` to it the way you can to `BackgroundIndex`.)

I should mention for completeness that the way we did tests like this in Eclipse CDT is... we actually wrote the test files to disk :) But I'm guessing, given the efforts that have been made to use `MockFS` in the test suite so far, that we don't want to do this for clangd.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89297/new/

https://reviews.llvm.org/D89297



More information about the cfe-commits mailing list