[clang-tools-extra] r344650 - [clang-doc] Add unit tests for serialization

Mikael Holmén via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 17 01:58:31 PDT 2018


Hi Julie,

clang 3.6.0 complains on this commit:

/usr/bin/clang++  -march=corei7  -DGTEST_HAS_RTTI=0 
-DGTEST_HAS_TR1_TUPLE=0 -DGTEST_LANG_CXX11=1 -D_DEBUG -D_GNU_SOURCE 
-D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS 
-Itools/clang/tools/extra/unittests/clang-doc 
-I../tools/clang/tools/extra/unittests/clang-doc 
-I../tools/clang/include -Itools/clang/include -I/usr/include/libxml2 
-Iinclude -I../include -I../tools/clang/tools/extra/clang-doc 
-I../utils/unittest/googletest/include 
-I../utils/unittest/googlemock/include 
-I/proj/flexasic/app/valgrind/3.11.0/include  -fPIC 
-fvisibility-inlines-hidden -Werror -Werror=date-time -std=c++11 -Wall 
-Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual 
-Wmissing-field-initializers -pedantic -Wno-long-long 
-Wcovered-switch-default -Wnon-virtual-dtor -Wdelete-non-virtual-dtor 
-Wstring-conversion -fdiagnostics-color -ffunction-sections 
-fdata-sections -fno-common -Woverloaded-virtual -Wno-nested-anon-types 
-O3    -UNDEBUG  -Wno-variadic-macros 
-Wno-gnu-zero-variadic-macro-arguments -fno-exceptions -fno-rtti -MMD 
-MT 
tools/clang/tools/extra/unittests/clang-doc/CMakeFiles/ClangDocTests.dir/BitcodeTest.cpp.o 
-MF 
tools/clang/tools/extra/unittests/clang-doc/CMakeFiles/ClangDocTests.dir/BitcodeTest.cpp.o.d 
-o 
tools/clang/tools/extra/unittests/clang-doc/CMakeFiles/ClangDocTests.dir/BitcodeTest.cpp.o 
-c ../tools/clang/tools/extra/unittests/clang-doc/BitcodeTest.cpp
In file included from 
../tools/clang/tools/extra/unittests/clang-doc/BitcodeTest.cpp:12:
../tools/clang/tools/extra/unittests/clang-doc/ClangDocTest.h:25:14: 
error: suggest braces around initialization of subobject 
[-Werror,-Wmissing-braces]
     SymbolID{1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1};
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
              {                                                         }
1 error generated.

Regards,
Mikael

On 10/17/2018 01:06 AM, Julie Hockett via cfe-commits wrote:
> Author: juliehockett
> Date: Tue Oct 16 16:06:42 2018
> New Revision: 344650
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=344650&view=rev
> Log:
> [clang-doc] Add unit tests for serialization
> 
> Adds unit tests for the Serialize library.
> 
> This is part of a move to convert clang-doc's tests to a more
> maintainable unit test framework, with a smaller number of integration
> tests to maintain and more granular failure feedback.
> 
> Differential Revision: https://reviews.llvm.org/D53081
> 
> Added:
>      clang-tools-extra/trunk/unittests/clang-doc/
>      clang-tools-extra/trunk/unittests/clang-doc/CMakeLists.txt
>      clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.cpp
>      clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.h
>      clang-tools-extra/trunk/unittests/clang-doc/SerializeTest.cpp
> Modified:
>      clang-tools-extra/trunk/unittests/CMakeLists.txt
> 
> Modified: clang-tools-extra/trunk/unittests/CMakeLists.txt
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/CMakeLists.txt?rev=344650&r1=344649&r2=344650&view=diff
> ==============================================================================
> --- clang-tools-extra/trunk/unittests/CMakeLists.txt (original)
> +++ clang-tools-extra/trunk/unittests/CMakeLists.txt Tue Oct 16 16:06:42 2018
> @@ -16,6 +16,7 @@ endif()
>   
>   add_subdirectory(change-namespace)
>   add_subdirectory(clang-apply-replacements)
> +add_subdirectory(clang-doc)
>   add_subdirectory(clang-move)
>   add_subdirectory(clang-query)
>   add_subdirectory(clang-tidy)
> 
> Added: clang-tools-extra/trunk/unittests/clang-doc/CMakeLists.txt
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-doc/CMakeLists.txt?rev=344650&view=auto
> ==============================================================================
> --- clang-tools-extra/trunk/unittests/clang-doc/CMakeLists.txt (added)
> +++ clang-tools-extra/trunk/unittests/clang-doc/CMakeLists.txt Tue Oct 16 16:06:42 2018
> @@ -0,0 +1,29 @@
> +set(LLVM_LINK_COMPONENTS
> +  support
> +  BitReader
> +  BitWriter
> +  )
> +
> +get_filename_component(CLANG_DOC_SOURCE_DIR
> +  ${CMAKE_CURRENT_SOURCE_DIR}/../../clang-doc REALPATH)
> +include_directories(
> +  ${CLANG_DOC_SOURCE_DIR}
> +  )
> +
> +add_extra_unittest(ClangDocTests
> +  ClangDocTest.cpp
> +  SerializeTest.cpp
> +  )
> +
> +target_link_libraries(ClangDocTests
> +  PRIVATE
> +  clangAST
> +  clangASTMatchers
> +  clangBasic
> +  clangDoc
> +  clangFormat
> +  clangFrontend
> +  clangRewrite
> +  clangTooling
> +  clangToolingCore
> +  )
> 
> Added: clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.cpp?rev=344650&view=auto
> ==============================================================================
> --- clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.cpp (added)
> +++ clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.cpp Tue Oct 16 16:06:42 2018
> @@ -0,0 +1,182 @@
> +//===-- clang-doc/ClangDocTest.cpp ----------------------------------------===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===----------------------------------------------------------------------===//
> +
> +#include "Representation.h"
> +#include "clang/AST/RecursiveASTVisitor.h"
> +#include "gtest/gtest.h"
> +
> +namespace clang {
> +namespace doc {
> +
> +NamespaceInfo *InfoAsNamespace(Info *I) {
> +  assert(I->IT == InfoType::IT_namespace);
> +  return static_cast<NamespaceInfo *>(I);
> +}
> +
> +RecordInfo *InfoAsRecord(Info *I) {
> +  assert(I->IT == InfoType::IT_record);
> +  return static_cast<RecordInfo *>(I);
> +}
> +
> +FunctionInfo *InfoAsFunction(Info *I) {
> +  assert(I->IT == InfoType::IT_function);
> +  return static_cast<FunctionInfo *>(I);
> +}
> +
> +EnumInfo *InfoAsEnum(Info *I) {
> +  assert(I->IT == InfoType::IT_enum);
> +  return static_cast<EnumInfo *>(I);
> +}
> +
> +void CheckCommentInfo(CommentInfo &Expected, CommentInfo &Actual) {
> +  EXPECT_EQ(Expected.Kind, Actual.Kind);
> +  EXPECT_EQ(Expected.Text, Actual.Text);
> +  EXPECT_EQ(Expected.Name, Actual.Name);
> +  EXPECT_EQ(Expected.Direction, Actual.Direction);
> +  EXPECT_EQ(Expected.ParamName, Actual.ParamName);
> +  EXPECT_EQ(Expected.CloseName, Actual.CloseName);
> +  EXPECT_EQ(Expected.SelfClosing, Actual.SelfClosing);
> +  EXPECT_EQ(Expected.Explicit, Actual.Explicit);
> +
> +  ASSERT_EQ(Expected.AttrKeys.size(), Actual.AttrKeys.size());
> +  for (size_t Idx = 0; Idx < Actual.AttrKeys.size(); ++Idx)
> +    EXPECT_EQ(Expected.AttrKeys[Idx], Actual.AttrKeys[Idx]);
> +
> +  ASSERT_EQ(Expected.AttrValues.size(), Actual.AttrValues.size());
> +  for (size_t Idx = 0; Idx < Actual.AttrValues.size(); ++Idx)
> +    EXPECT_EQ(Expected.AttrValues[Idx], Actual.AttrValues[Idx]);
> +
> +  ASSERT_EQ(Expected.Args.size(), Actual.Args.size());
> +  for (size_t Idx = 0; Idx < Actual.Args.size(); ++Idx)
> +    EXPECT_EQ(Expected.Args[Idx], Actual.Args[Idx]);
> +
> +  ASSERT_EQ(Expected.Children.size(), Actual.Children.size());
> +  for (size_t Idx = 0; Idx < Actual.Children.size(); ++Idx)
> +    CheckCommentInfo(*Expected.Children[Idx], *Actual.Children[Idx]);
> +}
> +
> +void CheckReference(Reference &Expected, Reference &Actual) {
> +  EXPECT_EQ(Expected.Name, Actual.Name);
> +  EXPECT_EQ(Expected.RefType, Actual.RefType);
> +}
> +
> +void CheckTypeInfo(TypeInfo *Expected, TypeInfo *Actual) {
> +  CheckReference(Expected->Type, Actual->Type);
> +}
> +
> +void CheckFieldTypeInfo(FieldTypeInfo *Expected, FieldTypeInfo *Actual) {
> +  CheckTypeInfo(Expected, Actual);
> +  EXPECT_EQ(Expected->Name, Actual->Name);
> +}
> +
> +void CheckMemberTypeInfo(MemberTypeInfo *Expected, MemberTypeInfo *Actual) {
> +  CheckFieldTypeInfo(Expected, Actual);
> +  EXPECT_EQ(Expected->Access, Actual->Access);
> +}
> +
> +void CheckBaseInfo(Info *Expected, Info *Actual) {
> +  EXPECT_EQ(size_t(20), Actual->USR.size());
> +  EXPECT_EQ(Expected->Name, Actual->Name);
> +  ASSERT_EQ(Expected->Namespace.size(), Actual->Namespace.size());
> +  for (size_t Idx = 0; Idx < Actual->Namespace.size(); ++Idx)
> +    CheckReference(Expected->Namespace[Idx], Actual->Namespace[Idx]);
> +  ASSERT_EQ(Expected->Description.size(), Actual->Description.size());
> +  for (size_t Idx = 0; Idx < Actual->Description.size(); ++Idx)
> +    CheckCommentInfo(Expected->Description[Idx], Actual->Description[Idx]);
> +}
> +
> +void CheckSymbolInfo(SymbolInfo *Expected, SymbolInfo *Actual) {
> +  CheckBaseInfo(Expected, Actual);
> +  EXPECT_EQ(Expected->DefLoc.hasValue(), Actual->DefLoc.hasValue());
> +  if (Expected->DefLoc.hasValue() && Actual->DefLoc.hasValue()) {
> +    EXPECT_EQ(Expected->DefLoc->LineNumber, Actual->DefLoc->LineNumber);
> +    EXPECT_EQ(Expected->DefLoc->Filename, Actual->DefLoc->Filename);
> +  }
> +  ASSERT_EQ(Expected->Loc.size(), Actual->Loc.size());
> +  for (size_t Idx = 0; Idx < Actual->Loc.size(); ++Idx)
> +    EXPECT_EQ(Expected->Loc[Idx], Actual->Loc[Idx]);
> +}
> +
> +void CheckFunctionInfo(FunctionInfo *Expected, FunctionInfo *Actual) {
> +  CheckSymbolInfo(Expected, Actual);
> +
> +  EXPECT_EQ(Expected->IsMethod, Actual->IsMethod);
> +  CheckReference(Expected->Parent, Actual->Parent);
> +  CheckTypeInfo(&Expected->ReturnType, &Actual->ReturnType);
> +
> +  ASSERT_EQ(Expected->Params.size(), Actual->Params.size());
> +  for (size_t Idx = 0; Idx < Actual->Params.size(); ++Idx)
> +    EXPECT_EQ(Expected->Params[Idx], Actual->Params[Idx]);
> +
> +  EXPECT_EQ(Expected->Access, Actual->Access);
> +}
> +
> +void CheckEnumInfo(EnumInfo *Expected, EnumInfo *Actual) {
> +  CheckSymbolInfo(Expected, Actual);
> +
> +  EXPECT_EQ(Expected->Scoped, Actual->Scoped);
> +  ASSERT_EQ(Expected->Members.size(), Actual->Members.size());
> +  for (size_t Idx = 0; Idx < Actual->Members.size(); ++Idx)
> +    EXPECT_EQ(Expected->Members[Idx], Actual->Members[Idx]);
> +}
> +
> +void CheckNamespaceInfo(NamespaceInfo *Expected, NamespaceInfo *Actual) {
> +  CheckBaseInfo(Expected, Actual);
> +
> +  ASSERT_EQ(Expected->ChildNamespaces.size(), Actual->ChildNamespaces.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildNamespaces.size(); ++Idx)
> +    EXPECT_EQ(Expected->ChildNamespaces[Idx], Actual->ChildNamespaces[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildRecords.size(), Actual->ChildRecords.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildRecords.size(); ++Idx)
> +    EXPECT_EQ(Expected->ChildRecords[Idx], Actual->ChildRecords[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildFunctions.size(), Actual->ChildFunctions.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildFunctions.size(); ++Idx)
> +    CheckFunctionInfo(&Expected->ChildFunctions[Idx],
> +                      &Actual->ChildFunctions[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildEnums.size(), Actual->ChildEnums.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildEnums.size(); ++Idx)
> +    CheckEnumInfo(&Expected->ChildEnums[Idx], &Actual->ChildEnums[Idx]);
> +}
> +
> +void CheckRecordInfo(RecordInfo *Expected, RecordInfo *Actual) {
> +  CheckSymbolInfo(Expected, Actual);
> +
> +  EXPECT_EQ(Expected->TagType, Actual->TagType);
> +
> +  ASSERT_EQ(Expected->Members.size(), Actual->Members.size());
> +  for (size_t Idx = 0; Idx < Actual->Members.size(); ++Idx)
> +    EXPECT_EQ(Expected->Members[Idx], Actual->Members[Idx]);
> +
> +  ASSERT_EQ(Expected->Parents.size(), Actual->Parents.size());
> +  for (size_t Idx = 0; Idx < Actual->Parents.size(); ++Idx)
> +    CheckReference(Expected->Parents[Idx], Actual->Parents[Idx]);
> +
> +  ASSERT_EQ(Expected->VirtualParents.size(), Actual->VirtualParents.size());
> +  for (size_t Idx = 0; Idx < Actual->VirtualParents.size(); ++Idx)
> +    CheckReference(Expected->VirtualParents[Idx], Actual->VirtualParents[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildRecords.size(), Actual->ChildRecords.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildRecords.size(); ++Idx)
> +    EXPECT_EQ(Expected->ChildRecords[Idx], Actual->ChildRecords[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildFunctions.size(), Actual->ChildFunctions.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildFunctions.size(); ++Idx)
> +    CheckFunctionInfo(&Expected->ChildFunctions[Idx],
> +                      &Actual->ChildFunctions[Idx]);
> +
> +  ASSERT_EQ(Expected->ChildEnums.size(), Actual->ChildEnums.size());
> +  for (size_t Idx = 0; Idx < Actual->ChildEnums.size(); ++Idx)
> +    CheckEnumInfo(&Expected->ChildEnums[Idx], &Actual->ChildEnums[Idx]);
> +}
> +
> +} // namespace doc
> +} // namespace clang
> 
> Added: clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.h
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.h?rev=344650&view=auto
> ==============================================================================
> --- clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.h (added)
> +++ clang-tools-extra/trunk/unittests/clang-doc/ClangDocTest.h Tue Oct 16 16:06:42 2018
> @@ -0,0 +1,51 @@
> +//===-- clang-doc/ClangDocTest.h ------------------------------------------===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===----------------------------------------------------------------------===//
> +
> +#ifndef LLVM_CLANG_TOOLS_EXTRA_UNITTESTS_CLANG_DOC_CLANGDOCTEST_H
> +#define LLVM_CLANG_TOOLS_EXTRA_UNITTESTS_CLANG_DOC_CLANGDOCTEST_H
> +
> +#include "ClangDocTest.h"
> +#include "Representation.h"
> +#include "clang/AST/RecursiveASTVisitor.h"
> +#include "gtest/gtest.h"
> +
> +namespace clang {
> +namespace doc {
> +
> +using EmittedInfoList = std::vector<std::unique_ptr<Info>>;
> +
> +static const SymbolID EmptySID = SymbolID();
> +static const SymbolID NonEmptySID =
> +    SymbolID{1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1};
> +
> +NamespaceInfo *InfoAsNamespace(Info *I);
> +RecordInfo *InfoAsRecord(Info *I);
> +FunctionInfo *InfoAsFunction(Info *I);
> +EnumInfo *InfoAsEnum(Info *I);
> +
> +// Unlike the operator==, these functions explicitly does not check USRs, as
> +// that may change and it would be better to not rely on its implementation.
> +void CheckReference(Reference &Expected, Reference &Actual);
> +void CheckTypeInfo(TypeInfo *Expected, TypeInfo *Actual);
> +void CheckFieldTypeInfo(FieldTypeInfo *Expected, FieldTypeInfo *Actual);
> +void CheckMemberTypeInfo(MemberTypeInfo *Expected, MemberTypeInfo *Actual);
> +
> +// This function explicitly does not check USRs, as that may change and it would
> +// be better to not rely on its implementation.
> +void CheckBaseInfo(Info *Expected, Info *Actual);
> +void CheckSymbolInfo(SymbolInfo *Expected, SymbolInfo *Actual);
> +void CheckFunctionInfo(FunctionInfo *Expected, FunctionInfo *Actual);
> +void CheckEnumInfo(EnumInfo *Expected, EnumInfo *Actual);
> +void CheckNamespaceInfo(NamespaceInfo *Expected, NamespaceInfo *Actual);
> +void CheckRecordInfo(RecordInfo *Expected, RecordInfo *Actual);
> +
> +} // namespace doc
> +} // namespace clang
> +
> +#endif // LLVM_CLANG_TOOLS_EXTRA_UNITTESTS_CLANG_DOC_CLANGDOCTEST_H
> 
> Added: clang-tools-extra/trunk/unittests/clang-doc/SerializeTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-doc/SerializeTest.cpp?rev=344650&view=auto
> ==============================================================================
> --- clang-tools-extra/trunk/unittests/clang-doc/SerializeTest.cpp (added)
> +++ clang-tools-extra/trunk/unittests/clang-doc/SerializeTest.cpp Tue Oct 16 16:06:42 2018
> @@ -0,0 +1,346 @@
> +//===-- clang-doc/SerializeTest.cpp ---------------------------------------===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===----------------------------------------------------------------------===//
> +
> +#include "Serialize.h"
> +#include "ClangDocTest.h"
> +#include "Representation.h"
> +#include "clang/AST/Comment.h"
> +#include "clang/AST/RecursiveASTVisitor.h"
> +#include "gtest/gtest.h"
> +
> +namespace clang {
> +namespace doc {
> +
> +class ClangDocSerializeTestVisitor
> +    : public RecursiveASTVisitor<ClangDocSerializeTestVisitor> {
> +
> +  EmittedInfoList &EmittedInfos;
> +  bool Public;
> +
> +  comments::FullComment *getComment(const NamedDecl *D) const {
> +    if (RawComment *Comment =
> +            D->getASTContext().getRawCommentForDeclNoCache(D)) {
> +      Comment->setAttached();
> +      return Comment->parse(D->getASTContext(), nullptr, D);
> +    }
> +    return nullptr;
> +  }
> +
> +public:
> +  ClangDocSerializeTestVisitor(EmittedInfoList &EmittedInfos, bool Public)
> +      : EmittedInfos(EmittedInfos), Public(Public) {}
> +
> +  bool VisitNamespaceDecl(const NamespaceDecl *D) {
> +    auto I = serialize::emitInfo(D, getComment(D), /*Line=*/0,
> +                                 /*File=*/"test.cpp", Public);
> +    if (I)
> +      EmittedInfos.emplace_back(std::move(I));
> +    return true;
> +  }
> +
> +  bool VisitFunctionDecl(const FunctionDecl *D) {
> +    // Don't visit CXXMethodDecls twice
> +    if (dyn_cast<CXXMethodDecl>(D))
> +      return true;
> +    auto I = serialize::emitInfo(D, getComment(D), /*Line=*/0,
> +                                 /*File=*/"test.cpp", Public);
> +    if (I)
> +      EmittedInfos.emplace_back(std::move(I));
> +    return true;
> +  }
> +
> +  bool VisitCXXMethodDecl(const CXXMethodDecl *D) {
> +    auto I = serialize::emitInfo(D, getComment(D), /*Line=*/0,
> +                                 /*File=*/"test.cpp", Public);
> +    if (I)
> +      EmittedInfos.emplace_back(std::move(I));
> +    return true;
> +  }
> +
> +  bool VisitRecordDecl(const RecordDecl *D) {
> +    auto I = serialize::emitInfo(D, getComment(D), /*Line=*/0,
> +                                 /*File=*/"test.cpp", Public);
> +    if (I)
> +      EmittedInfos.emplace_back(std::move(I));
> +    return true;
> +  }
> +
> +  bool VisitEnumDecl(const EnumDecl *D) {
> +    auto I = serialize::emitInfo(D, getComment(D), /*Line=*/0,
> +                                 /*File=*/"test.cpp", Public);
> +    if (I)
> +      EmittedInfos.emplace_back(std::move(I));
> +    return true;
> +  }
> +};
> +
> +void ExtractInfosFromCode(StringRef Code, size_t NumExpectedInfos, bool Public,
> +                          EmittedInfoList &EmittedInfos) {
> +  auto ASTUnit = clang::tooling::buildASTFromCode(Code);
> +  auto TU = ASTUnit->getASTContext().getTranslationUnitDecl();
> +  ClangDocSerializeTestVisitor Visitor(EmittedInfos, Public);
> +  Visitor.TraverseTranslationUnitDecl(TU);
> +  ASSERT_EQ(NumExpectedInfos, EmittedInfos.size());
> +}
> +
> +void ExtractInfosFromCodeWithArgs(StringRef Code, size_t NumExpectedInfos,
> +                                  bool Public, EmittedInfoList &EmittedInfos,
> +                                  std::vector<std::string> &Args) {
> +  auto ASTUnit = clang::tooling::buildASTFromCodeWithArgs(Code, Args);
> +  auto TU = ASTUnit->getASTContext().getTranslationUnitDecl();
> +  ClangDocSerializeTestVisitor Visitor(EmittedInfos, Public);
> +  Visitor.TraverseTranslationUnitDecl(TU);
> +  ASSERT_EQ(NumExpectedInfos, EmittedInfos.size());
> +}
> +
> +// Test serialization of namespace declarations.
> +TEST(SerializeTest, emitNamespaceInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("namespace A { namespace B { void f() {} } }", 3,
> +                       /*Public=*/false, Infos);
> +
> +  NamespaceInfo *A = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedA(EmptySID, "A");
> +  CheckNamespaceInfo(&ExpectedA, A);
> +
> +  NamespaceInfo *B = InfoAsNamespace(Infos[1].get());
> +  NamespaceInfo ExpectedB(EmptySID, "B");
> +  ExpectedB.Namespace.emplace_back(EmptySID, "A", InfoType::IT_namespace);
> +  CheckNamespaceInfo(&ExpectedB, B);
> +
> +  NamespaceInfo *BWithFunction = InfoAsNamespace(Infos[2].get());
> +  NamespaceInfo ExpectedBWithFunction(EmptySID);
> +  FunctionInfo F;
> +  F.Name = "f";
> +  F.ReturnType = TypeInfo(EmptySID, "void", InfoType::IT_default);
> +  F.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  F.Namespace.emplace_back(EmptySID, "B", InfoType::IT_namespace);
> +  F.Namespace.emplace_back(EmptySID, "A", InfoType::IT_namespace);
> +  ExpectedBWithFunction.ChildFunctions.emplace_back(std::move(F));
> +  CheckNamespaceInfo(&ExpectedBWithFunction, BWithFunction);
> +}
> +
> +TEST(SerializeTest, emitAnonymousNamespaceInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("namespace { }", 1, /*Public=*/false, Infos);
> +
> +  NamespaceInfo *A = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedA(EmptySID);
> +  CheckNamespaceInfo(&ExpectedA, A);
> +}
> +
> +// Test serialization of record declarations.
> +TEST(SerializeTest, emitRecordInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode(R"raw(class E {
> +public:
> +  E() {}
> +protected:
> +  void ProtectedMethod();
> +};)raw", 3, /*Public=*/false, Infos);
> +
> +  RecordInfo *E = InfoAsRecord(Infos[0].get());
> +  RecordInfo ExpectedE(EmptySID, "E");
> +  ExpectedE.TagType = TagTypeKind::TTK_Class;
> +  ExpectedE.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  CheckRecordInfo(&ExpectedE, E);
> +
> +  RecordInfo *RecordWithEConstructor = InfoAsRecord(Infos[1].get());
> +  RecordInfo ExpectedRecordWithEConstructor(EmptySID);
> +  FunctionInfo EConstructor;
> +  EConstructor.Name = "E";
> +  EConstructor.Parent = Reference(EmptySID, "E", InfoType::IT_record);
> +  EConstructor.ReturnType = TypeInfo(EmptySID, "void", InfoType::IT_default);
> +  EConstructor.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  EConstructor.Namespace.emplace_back(EmptySID, "E", InfoType::IT_record);
> +  EConstructor.Access = AccessSpecifier::AS_public;
> +  EConstructor.IsMethod = true;
> +  ExpectedRecordWithEConstructor.ChildFunctions.emplace_back(
> +      std::move(EConstructor));
> +  CheckRecordInfo(&ExpectedRecordWithEConstructor, RecordWithEConstructor);
> +
> +  RecordInfo *RecordWithMethod = InfoAsRecord(Infos[2].get());
> +  RecordInfo ExpectedRecordWithMethod(EmptySID);
> +  FunctionInfo Method;
> +  Method.Name = "ProtectedMethod";
> +  Method.Parent = Reference(EmptySID, "E", InfoType::IT_record);
> +  Method.ReturnType = TypeInfo(EmptySID, "void", InfoType::IT_default);
> +  Method.Loc.emplace_back(0, llvm::SmallString<16>{"test.cpp"});
> +  Method.Namespace.emplace_back(EmptySID, "E", InfoType::IT_record);
> +  Method.Access = AccessSpecifier::AS_protected;
> +  Method.IsMethod = true;
> +  ExpectedRecordWithMethod.ChildFunctions.emplace_back(std::move(Method));
> +  CheckRecordInfo(&ExpectedRecordWithMethod, RecordWithMethod);
> +}
> +
> +// Test serialization of enum declarations.
> +TEST(SerializeTest, emitEnumInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("enum E { X, Y }; enum class G { A, B };", 2,
> +                       /*Public=*/false, Infos);
> +
> +  NamespaceInfo *NamespaceWithEnum = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedNamespaceWithEnum(EmptySID);
> +  EnumInfo E;
> +  E.Name = "E";
> +  E.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  E.Members.emplace_back("X");
> +  E.Members.emplace_back("Y");
> +  ExpectedNamespaceWithEnum.ChildEnums.emplace_back(std::move(E));
> +  CheckNamespaceInfo(&ExpectedNamespaceWithEnum, NamespaceWithEnum);
> +
> +  NamespaceInfo *NamespaceWithScopedEnum = InfoAsNamespace(Infos[1].get());
> +  NamespaceInfo ExpectedNamespaceWithScopedEnum(EmptySID);
> +  EnumInfo G;
> +  G.Name = "G";
> +  G.Scoped = true;
> +  G.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  G.Members.emplace_back("A");
> +  G.Members.emplace_back("B");
> +  ExpectedNamespaceWithScopedEnum.ChildEnums.emplace_back(std::move(G));
> +  CheckNamespaceInfo(&ExpectedNamespaceWithScopedEnum, NamespaceWithScopedEnum);
> +}
> +
> +TEST(SerializeTest, emitUndefinedRecordInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("class E;", 1, /*Public=*/false, Infos);
> +
> +  RecordInfo *E = InfoAsRecord(Infos[0].get());
> +  RecordInfo ExpectedE(EmptySID, "E");
> +  ExpectedE.TagType = TagTypeKind::TTK_Class;
> +  ExpectedE.Loc.emplace_back(0, llvm::SmallString<16>{"test.cpp"});
> +  CheckRecordInfo(&ExpectedE, E);
> +}
> +
> +TEST(SerializeTest, emitRecordMemberInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("struct E { int I; };", 1, /*Public=*/false, Infos);
> +
> +  RecordInfo *E = InfoAsRecord(Infos[0].get());
> +  RecordInfo ExpectedE(EmptySID, "E");
> +  ExpectedE.TagType = TagTypeKind::TTK_Struct;
> +  ExpectedE.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  ExpectedE.Members.emplace_back("int", "I", AccessSpecifier::AS_public);
> +  CheckRecordInfo(&ExpectedE, E);
> +}
> +
> +TEST(SerializeTest, emitInternalRecordInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("class E { class G {}; };", 2, /*Public=*/false, Infos);
> +
> +  RecordInfo *E = InfoAsRecord(Infos[0].get());
> +  RecordInfo ExpectedE(EmptySID, "E");
> +  ExpectedE.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  ExpectedE.TagType = TagTypeKind::TTK_Class;
> +  CheckRecordInfo(&ExpectedE, E);
> +
> +  RecordInfo *G = InfoAsRecord(Infos[1].get());
> +  RecordInfo ExpectedG(EmptySID, "G");
> +  ExpectedG.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  ExpectedG.TagType = TagTypeKind::TTK_Class;
> +  ExpectedG.Namespace.emplace_back(EmptySID, "E", InfoType::IT_record);
> +  CheckRecordInfo(&ExpectedG, G);
> +}
> +
> +TEST(SerializeTest, emitPublicAnonymousNamespaceInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("namespace { class A; }", 0, /*Public=*/true, Infos);
> +}
> +
> +TEST(SerializeTest, emitPublicFunctionInternalInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("int F() { class G {}; return 0; };", 1, /*Public=*/true,
> +                       Infos);
> +
> +  NamespaceInfo *BWithFunction = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedBWithFunction(EmptySID);
> +  FunctionInfo F;
> +  F.Name = "F";
> +  F.ReturnType = TypeInfo(EmptySID, "int", InfoType::IT_default);
> +  F.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  ExpectedBWithFunction.ChildFunctions.emplace_back(std::move(F));
> +  CheckNamespaceInfo(&ExpectedBWithFunction, BWithFunction);
> +}
> +
> +TEST(SerializeTest, emitInlinedFunctionInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode("inline void F(int I) { };", 1, /*Public=*/true, Infos);
> +
> +  NamespaceInfo *BWithFunction = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedBWithFunction(EmptySID);
> +  FunctionInfo F;
> +  F.Name = "F";
> +  F.ReturnType = TypeInfo(EmptySID, "void", InfoType::IT_default);
> +  F.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  F.Params.emplace_back("int", "I");
> +  ExpectedBWithFunction.ChildFunctions.emplace_back(std::move(F));
> +  CheckNamespaceInfo(&ExpectedBWithFunction, BWithFunction);
> +}
> +
> +TEST(SerializeTest, emitInheritedRecordInfo) {
> +  EmittedInfoList Infos;
> +  ExtractInfosFromCode(
> +      "class F {}; class G{} ; class E : public F, virtual private G {};", 3,
> +      /*Public=*/false, Infos);
> +
> +  RecordInfo *F = InfoAsRecord(Infos[0].get());
> +  RecordInfo ExpectedF(EmptySID, "F");
> +  ExpectedF.TagType = TagTypeKind::TTK_Class;
> +  ExpectedF.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  CheckRecordInfo(&ExpectedF, F);
> +
> +  RecordInfo *G = InfoAsRecord(Infos[1].get());
> +  RecordInfo ExpectedG(EmptySID, "G");
> +  ExpectedG.TagType = TagTypeKind::TTK_Class;
> +  ExpectedG.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  CheckRecordInfo(&ExpectedG, G);
> +
> +  RecordInfo *E = InfoAsRecord(Infos[2].get());
> +  RecordInfo ExpectedE(EmptySID, "E");
> +  ExpectedE.Parents.emplace_back(EmptySID, "F", InfoType::IT_record);
> +  ExpectedE.VirtualParents.emplace_back(EmptySID, "G", InfoType::IT_record);
> +  ExpectedE.DefLoc = Location(0, llvm::SmallString<16>{"test.cpp"});
> +  ExpectedE.TagType = TagTypeKind::TTK_Class;
> +  CheckRecordInfo(&ExpectedE, E);
> +}
> +
> +TEST(SerializeTest, emitModulePublicLFunctions) {
> +  EmittedInfoList Infos;
> +  std::vector<std::string> Args;
> +  Args.push_back("-fmodules-ts");
> +  ExtractInfosFromCodeWithArgs(R"raw(export module M;
> +int moduleFunction(int x);
> +static int staticModuleFunction(int x);
> +export double exportedModuleFunction(double y);)raw",
> +                               2, /*Public=*/true, Infos, Args);
> +
> +  NamespaceInfo *BWithFunction = InfoAsNamespace(Infos[0].get());
> +  NamespaceInfo ExpectedBWithFunction(EmptySID);
> +  FunctionInfo F;
> +  F.Name = "moduleFunction";
> +  F.ReturnType = TypeInfo(EmptySID, "int", InfoType::IT_default);
> +  F.Loc.emplace_back(0, llvm::SmallString<16>{"test.cpp"});
> +  F.Params.emplace_back("int", "x");
> +  ExpectedBWithFunction.ChildFunctions.emplace_back(std::move(F));
> +  CheckNamespaceInfo(&ExpectedBWithFunction, BWithFunction);
> +
> +  NamespaceInfo *BWithExportedFunction = InfoAsNamespace(Infos[1].get());
> +  NamespaceInfo ExpectedBWithExportedFunction(EmptySID);
> +  FunctionInfo ExportedF;
> +  ExportedF.Name = "exportedModuleFunction";
> +  ExportedF.ReturnType = TypeInfo(EmptySID, "double", InfoType::IT_default);
> +  ExportedF.Loc.emplace_back(0, llvm::SmallString<16>{"test.cpp"});
> +  ExportedF.Params.emplace_back("double", "y");
> +  ExpectedBWithExportedFunction.ChildFunctions.emplace_back(
> +      std::move(ExportedF));
> +  CheckNamespaceInfo(&ExpectedBWithExportedFunction, BWithExportedFunction);
> +}
> +
> +} // namespace doc
> +} // end namespace clang
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
> 


More information about the cfe-commits mailing list