summaryrefslogtreecommitdiffstats
path: root/clang/unittests/Tooling/RefactoringActionRulesTest.cpp
diff options
context:
space:
mode:
authorAlex Lorenz <arphaman@gmail.com>2017-09-14 10:06:52 +0000
committerAlex Lorenz <arphaman@gmail.com>2017-09-14 10:06:52 +0000
commitb54ef6a2a4c84293f64f90ea344b129eabbb0cd1 (patch)
treebeea03e3a80eaf8bc338ebb66b247f3363562975 /clang/unittests/Tooling/RefactoringActionRulesTest.cpp
parentd7b3add7c4a25a558794bf1c1ca30f865b8c6dd7 (diff)
downloadbcm5719-llvm-b54ef6a2a4c84293f64f90ea344b129eabbb0cd1.tar.gz
bcm5719-llvm-b54ef6a2a4c84293f64f90ea344b129eabbb0cd1.zip
[refactor] add clang-refactor tool with initial testing support and
local-rename action This commit introduces the clang-refactor tool alongside the local-rename action which uses the existing renaming engine used by clang-rename. The tool doesn't actually perform the source transformations yet, it just provides testing support. This commit also moves only one test from clang-rename over to test/Refactor. I will continue to move the other tests throughout development of clang-refactor. The following options are supported by clang-refactor: -v: use verbose output -selection: The source range that corresponds to the portion of the source that's selected (currently only special command test:<file> is supported). Please note that a follow-up commit will migrate clang-refactor to libTooling's common option parser, so clang-refactor will be able to use the common interface with compilation database and options like -p, -extra-arg, etc. The testing support provided by clang-refactor is described below: When -selection=test:<file> is given, clang-refactor will parse the selection commands from that file. The selection commands are grouped and the specified refactoring action invoked by the tool. Each command in a group is expected to produce an identical result. The precise syntax for the selection commands is described in a comment in TestSupport.h. Differential Revision: https://reviews.llvm.org/D36574 llvm-svn: 313244
Diffstat (limited to 'clang/unittests/Tooling/RefactoringActionRulesTest.cpp')
-rw-r--r--clang/unittests/Tooling/RefactoringActionRulesTest.cpp20
1 files changed, 13 insertions, 7 deletions
diff --git a/clang/unittests/Tooling/RefactoringActionRulesTest.cpp b/clang/unittests/Tooling/RefactoringActionRulesTest.cpp
index 62c751ac12c..cf91b5fc8f4 100644
--- a/clang/unittests/Tooling/RefactoringActionRulesTest.cpp
+++ b/clang/unittests/Tooling/RefactoringActionRulesTest.cpp
@@ -57,7 +57,8 @@ createReplacements(const std::unique_ptr<RefactoringActionRule> &Rule,
TEST_F(RefactoringActionRulesTest, MyFirstRefactoringRule) {
auto ReplaceAWithB =
- [](std::pair<selection::SourceSelectionRange, int> Selection)
+ [](const RefactoringRuleContext &,
+ std::pair<selection::SourceSelectionRange, int> Selection)
-> Expected<AtomicChanges> {
const SourceManager &SM = Selection.first.getSources();
SourceLocation Loc = Selection.first.getRange().getBegin().getLocWithOffset(
@@ -71,7 +72,8 @@ TEST_F(RefactoringActionRulesTest, MyFirstRefactoringRule) {
class SelectionRequirement : public selection::Requirement {
public:
std::pair<selection::SourceSelectionRange, int>
- evaluateSelection(selection::SourceSelectionRange Selection) const {
+ evaluateSelection(const RefactoringRuleContext &,
+ selection::SourceSelectionRange Selection) const {
return std::make_pair(Selection, 20);
}
};
@@ -127,8 +129,10 @@ TEST_F(RefactoringActionRulesTest, MyFirstRefactoringRule) {
}
TEST_F(RefactoringActionRulesTest, ReturnError) {
- Expected<AtomicChanges> (*Func)(selection::SourceSelectionRange) =
- [](selection::SourceSelectionRange) -> Expected<AtomicChanges> {
+ Expected<AtomicChanges> (*Func)(const RefactoringRuleContext &,
+ selection::SourceSelectionRange) =
+ [](const RefactoringRuleContext &,
+ selection::SourceSelectionRange) -> Expected<AtomicChanges> {
return llvm::make_error<llvm::StringError>(
"Error", llvm::make_error_code(llvm::errc::invalid_argument));
};
@@ -155,13 +159,14 @@ TEST_F(RefactoringActionRulesTest, ReturnInitiationDiagnostic) {
class SelectionRequirement : public selection::Requirement {
public:
Expected<Optional<int>>
- evaluateSelection(selection::SourceSelectionRange Selection) const {
+ evaluateSelection(const RefactoringRuleContext &,
+ selection::SourceSelectionRange Selection) const {
return llvm::make_error<llvm::StringError>(
"bad selection", llvm::make_error_code(llvm::errc::invalid_argument));
}
};
auto Rule = createRefactoringRule(
- [](int) -> Expected<AtomicChanges> {
+ [](const RefactoringRuleContext &, int) -> Expected<AtomicChanges> {
llvm::report_fatal_error("Should not run!");
},
requiredSelection(SelectionRequirement()));
@@ -201,7 +206,8 @@ Optional<SymbolOccurrences> findOccurrences(RefactoringActionRule &Rule,
TEST_F(RefactoringActionRulesTest, ReturnSymbolOccurrences) {
auto Rule = createRefactoringRule(
- [](selection::SourceSelectionRange Selection)
+ [](const RefactoringRuleContext &,
+ selection::SourceSelectionRange Selection)
-> Expected<SymbolOccurrences> {
SymbolOccurrences Occurrences;
Occurrences.push_back(SymbolOccurrence(
OpenPOWER on IntegriCloud