[libclang] Fix test LibclangReparseTest.FileName when TMPDIR is set to a symbolic link

[petpav01]

Summary

Commit r329515 recently added new test LibclangReparseTest.FileName. This test fails in an environment which has TMPDIR set to a symbolic link that points to an actual directory.

Example:

build$ ln -s /tmp mytmp
build$ TMPDIR=$PWD/mytmp ./tools/clang/unittests/libclang/libclangTests
[...]
[----------] 5 tests from LibclangReparseTest
[ RUN      ] LibclangReparseTest.FileName
.../tools/clang/unittests/libclang/LibclangTest.cpp:497: Failure
Value of: strstr(clang_getCString(cxname), CppName.c_str())
  Actual: false
Expected: true
[  FAILED  ] LibclangReparseTest.FileName (10 ms)
[...]

CppName is the original path in form $tmpdir/libclang-test-xxxxxx/main.cpp. Value cxname is obtained through clang_File_tryGetRealPathName() which gives the real path with the symlink resolved. The test fails because these two paths do not match.

The proposed patch addresses the problem by checking only that the value returned by clang_File_tryGetRealPathName() ends with "main.cpp".

Additionally, the patch makes the previous assertion in the test that checks result of clang_getFileName() stricter. It newly verifies that the name returned by the function is exactly same as what was given to clang_parseTranslationUnit()/clang_getFile().

[MaskRay]

Thanks!

unittests/libclang/LibclangTest.cpp

 //===- unittests/libclang/LibclangTest.cpp --- libclang tests -------------===//
 //
 //                     The LLVM Compiler Infrastructure
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 
 #include "clang-c/Index.h"
+#include "llvm/ADT/StringRef.h"
 #include "llvm/Support/Debug.h"
 #include "llvm/Support/FileSystem.h"
 #include "llvm/Support/Path.h"
 #include "llvm/Support/raw_ostream.h"
 #include "gtest/gtest.h"
 #include <fstream>
 #include <functional>
 #include <map>

 TEST_F(LibclangReparseTest, FileName) {
   std::string CppName = "main.cpp";
   WriteFile(CppName, "int main() {}");
   ClangTU = clang_parseTranslationUnit(Index, CppName.c_str(), nullptr, 0,
                                        nullptr, 0, TUFlags);
   CXFile cxf = clang_getFile(ClangTU, CppName.c_str());
 
   CXString cxname = clang_getFileName(cxf);
-  ASSERT_TRUE(strstr(clang_getCString(cxname), CppName.c_str()));
+  ASSERT_STREQ(clang_getCString(cxname), CppName.c_str());
   clang_disposeString(cxname);
 
   cxname = clang_File_tryGetRealPathName(cxf);
-  ASSERT_TRUE(strstr(clang_getCString(cxname), CppName.c_str()));
+  ASSERT_TRUE(llvm::StringRef(clang_getCString(cxname)).endswith("main.cpp"));
   clang_disposeString(cxname);
 }
 
 TEST_F(LibclangReparseTest, Reparse) {
   const char *HeaderTop = "#ifndef H\n#define H\nstruct Foo { int bar;";
   const char *HeaderBottom = "\n};\n#endif\n";
   const char *CppFile = "#include \"HeaderFile.h\"\nint main() {"
                          " Foo foo; foo.bar = 7; foo.baz = 8; }\n";