[all-commits] [llvm/llvm-project] d1211f: [SyntaxTree] Split tests for expressions

Eduardo Caldas via All-commits all-commits at lists.llvm.org
Thu Aug 13 01:18:47 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: d1211fd1ec037f88c2cc855bc850200948c76940
      https://github.com/llvm/llvm-project/commit/d1211fd1ec037f88c2cc855bc850200948c76940
  Author: Eduardo Caldas <ecaldas at google.com>
  Date:   2020-08-13 (Thu, 13 Aug 2020)

  Changed paths:
    M clang/unittests/Tooling/Syntax/TreeTest.cpp

  Log Message:
  -----------
  [SyntaxTree] Split tests for expressions

We do that because:
* Big tests generated big tree dumps that could hardly serve as documentation.
* In most cases the tests didn't share setup, thus there was not much addition in lines of code.

We split tests for:
* `UserDefinedLiteral`
* `NestedBinaryOperator`
* `UserDefinedBinaryOperator`
* `UserDefinedPrefixOperator`
* `QualifiedId`

Differential Revision: https://reviews.llvm.org/D85819


  Commit: 833c2b6be26bcdf90aac2f6de6e345bcd858149e
      https://github.com/llvm/llvm-project/commit/833c2b6be26bcdf90aac2f6de6e345bcd858149e
  Author: Eduardo Caldas <ecaldas at google.com>
  Date:   2020-08-13 (Thu, 13 Aug 2020)

  Changed paths:
    M clang/unittests/Tooling/Syntax/TreeTest.cpp

  Log Message:
  -----------
  [SyntaxTree] Rename tests following `TestSuite_TestCase` + nits


Compare: https://github.com/llvm/llvm-project/compare/8199cd8a550d...833c2b6be26b


More information about the All-commits mailing list