r194494 - Add test for injecting diagnostic consumers into a ClangTool.

[jahanian]

More details on the failure of this test on a VS buildbot machine. Hope this helps.

FAIL: Clang-Unit :: Tooling/Release/ToolingTests.exe/ClangToolTest.InjectDiagnosticConsumerInBuildASTs (6620 of 6748)
  ******************** TEST 'Clang-Unit :: Tooling/Release/ToolingTests.exe/ClangToolTest.InjectDiagnosticConsumerInBuildASTs' FAILED ********************
  Note: Google Test filter = ClangToolTest.InjectDiagnosticConsumerInBuildASTs

  [==========] Running 1 test from 1 test case.

  [----------] Global test environment set-up.

  [----------] 1 test from ClangToolTest

  [ RUN      ] ClangToolTest.InjectDiagnosticConsumerInBuildASTs
..\..\..\..\..\..\src\tools\clang\unittests\Tooling\ToolingTest.cpp(332): error : Value of: ASTs.size() [C:\public\zorg\buildbot\smooshlab\slave-0.8\...\clang-build\223482\tools\clang\test\check-clang.vcxproj]

    Actual: 0

  Expected: 1u

  Which is: 1

  [  FAILED  ] ClangToolTest.InjectDiagnosticConsumerInBuildASTs (0 ms)

  [----------] 1 test from ClangToolTest (0 ms total)



  [----------] Global test environment tear-down

  [==========] 1 test from 1 test case ran. (0 ms total)

  [  PASSED  ] 0 tests.

  [  FAILED  ] 1 test, listed below:

  [  FAILED  ] ClangToolTest.InjectDiagnosticConsumerInBuildASTs



   1 FAILED TEST

  Error while processing C:\a.cc.

Failing Tests (1):
      Clang-Unit :: Tooling/Release/ToolingTests.exe/ClangToolTest.InjectDiagnosticConsumerInBuildASTs

- Thank you 
   - Fariborz

On Nov 12, 2013, at 11:11 AM, jahanian <fjahanian at apple.com> wrote:

> Test is failing on Wondows buildbot.
> Failing Tests (1):
>       Clang-Unit :: Tooling/Release/ToolingTests.exe/ClangToolTest.InjectDiagnosticConsumerInBuildASTs
> - Fariborz
> 
> On Nov 12, 2013, at 9:53 AM, Manuel Klimek <klimek at google.com> wrote:
> 
>> Author: klimek
>> Date: Tue Nov 12 11:53:18 2013
>> New Revision: 194494
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=194494&view=rev
>> Log:
>> Add test for injecting diagnostic consumers into a ClangTool.
>> 
>> As suggested by pcc on 194226.
>> 
>> Modified:
>>    cfe/trunk/unittests/Tooling/ToolingTest.cpp
>> 
>> Modified: cfe/trunk/unittests/Tooling/ToolingTest.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Tooling/ToolingTest.cpp?rev=194494&r1=194493&r2=194494&view=diff
>> ==============================================================================
>> --- cfe/trunk/unittests/Tooling/ToolingTest.cpp (original)
>> +++ cfe/trunk/unittests/Tooling/ToolingTest.cpp Tue Nov 12 11:53:18 2013
>> @@ -321,5 +321,17 @@ TEST(ClangToolTest, InjectDiagnosticCons
>>   EXPECT_EQ(1u, Consumer.NumDiagnosticsSeen);
>> }
>> 
>> +TEST(ClangToolTest, InjectDiagnosticConsumerInBuildASTs) {
>> +  FixedCompilationDatabase Compilations("/", std::vector<std::string>());
>> +  ClangTool Tool(Compilations, std::vector<std::string>(1, "/a.cc"));
>> +  Tool.mapVirtualFile("/a.cc", "int x = undeclared;");
>> +  TestDiagnosticConsumer Consumer;
>> +  Tool.setDiagnosticConsumer(&Consumer);
>> +  std::vector<ASTUnit*> ASTs;
>> +  Tool.buildASTs(ASTs);
>> +  EXPECT_EQ(1u, ASTs.size());
>> +  EXPECT_EQ(1u, Consumer.NumDiagnosticsSeen);
>> +}
>> +
>> } // end namespace tooling
>> } // end namespace clang
>> 
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131112/80d473d8/attachment.html>