Add refactoring actions tests

Author: IgnatSergeev

clang/test/Refactor/Edit/RedundandIf.cpp

@@ -0,0 +1,64 @@
+// RUN: clang-refactor redundand-if -location=test:%s %s -- 2>&1 | grep -v CHECK | FileCheck %s
+
+void bodyLocation() {
+    if (false) {
+        int thenVar;
+        /*loc body_loc=*/
+    }
+}
+
+// CHECK: 1 'body_loc' results:
+// CHECK: void bodyLocation() {
+// CHECK-NEXT: {{^     }}
+// CHECK-NEXT: }
+
+void ifFalse() {
+    if (/*loc false=*/false) {
+        int thenVar;
+    } else {
+        int elseVar;
+    }
+}
+
+// CHECK: 1 'false' results:
+// CHECK: void ifFalse() {
+// CHECK-NEXT: {
+// CHECK-NEXT: int elseVar;
+// CHECK-NEXT: }
+// CHECK-NEXT: }
+
+void ifFalseNoElse() {
+    if (/*loc false_no_else=*/false) {
+        int thenVar;
+    }
+}
+
+// CHECK: 1 'false_no_else' results:
+// CHECK: void ifFalseNoElse() {
+// CHECK-NEXT: {{^     }}
+// CHECK-NEXT: }
+
+void noMatchingLocation() {
+    /*loc no_matching_location=*/
+    if (false) {
+        int thenVar;
+    }
+}
+
+// CHECK: 1 'no_matching_location' results:
+// CHECK-NEXT: refactoring action can't be initiated without a matching location
+
+void ifTrue() {
+    if (/*loc true=*/true) {
+        int thenVar;
+    } else {
+        int elseVar;
+    }
+}
+
+// CHECK: 1 'true' results:
+// CHECK: void ifTrue() {
+// CHECK-NEXT: {
+// CHECK-NEXT: int thenVar;
+// CHECK-NEXT: }
+// CHECK-NEXT: }

clang/test/Refactor/Edit/RedundandWhile.cpp

@@ -0,0 +1,35 @@
+// RUN: clang-refactor redundand-while -location=test:%s %s -- 2>&1 | grep -v CHECK | FileCheck %s
+
+void bodyLocation() {
+    while (false) {
+        int thenVar;
+        /*loc body_loc=*/
+    }
+}
+
+// CHECK: 1 'body_loc' results:
+// CHECK: void bodyLocation() {
+// CHECK-NEXT: {{^     }}
+// CHECK-NEXT: }
+
+void conditionLoc() {
+    while (/*loc false=*/false) {
+        int thenVar;
+    }
+}
+
+// CHECK: 1 'false' results:
+// CHECK: void conditionLoc() {
+// CHECK-NEXT: {{^     }}
+// CHECK-NEXT: }
+
+void noMatchingLocation() {
+    /*loc no_matching_location=*/
+    while (false) {
+        int thenVar;
+    }
+}
+
+// CHECK: 1 'no_matching_location' results:
+// CHECK-NEXT: refactoring action can't be initiated without a matching location
+

clang/test/Refactor/Edit/SimplifyExpr.cpp

@@ -0,0 +1,56 @@
+// RUN: clang-refactor simplify-expr -location=test:%s %s -- 2>&1 | grep -v CHECK | FileCheck %s
+
+bool foo = true;
+bool bar = false /*loc and_false_lhs=*/ && foo;
+
+// CHECK: 1 'and_false_lhs' results:
+// CHECK: bool bar = false;
+
+bool foo2 = true;
+bool bar2 = foo2 /*loc and_false_rhs=*/ && false;
+
+// CHECK: 1 'and_false_rhs' results:
+// CHECK: bool bar2 = false;
+
+bool foo3 = true;
+bool bar3 = true /*loc and_true_lhs=*/ && foo3;
+
+// CHECK: 1 'and_true_lhs' results:
+// CHECK: bool bar3 = foo3;
+
+bool foo4 = true;
+bool bar4 = foo4 /*loc and_true_rhs=*/ && true;
+
+// CHECK: 1 'and_true_rhs' results:
+// CHECK: bool bar4 = foo4;
+
+bool foo5 = true;
+bool bar5 = /*loc no_matching_location=*/ true || foo5;
+
+// CHECK: 1 'no_matching_location' results:
+// CHECK-NEXT: refactoring action can't be initiated without a matching location
+
+bool foo6 = true;
+bool bar6 = false /*loc or_false_lhs=*/ || foo6;
+
+// CHECK: 1 'or_false_lhs' results:
+// CHECK: bool bar6 = foo6;
+
+bool foo7 = true;
+bool bar7 = foo7 /*loc or_false_rhs=*/ || false;
+
+// CHECK: 1 'or_false_rhs' results:
+// CHECK: bool bar7 = foo7;
+
+bool foo8 = true;
+bool bar8 = true /*loc or_true_lhs=*/ || foo8;
+
+// CHECK: 1 'or_true_lhs' results:
+// CHECK: bool bar8 = true;
+//
+bool foo9 = true;
+bool bar9 = foo9 /*loc or_true_rhs=*/ || true;
+
+// CHECK: 1 'or_true_rhs' results:
+// CHECK: bool bar9 = true;
+

clang/test/Refactor/Edit/UnusedField.cpp

@@ -0,0 +1,20 @@
+// RUN: clang-refactor unused-field -location=test:%s %s -- 2>&1 | grep -v CHECK | FileCheck %s
+
+struct Bar {
+    /*loc used=*/int member;
+    /*loc unused=*/int unusedMember;
+};
+
+void foo() {
+    Bar a;
+    a.member = 1;
+}
+
+// CHECK: 1 'unused' results:
+// CHECK: struct Bar {
+// CHECK-NEXT: /*loc used=*/int member;
+// CHECK-NEXT: /*loc unused=*/
+// CHECK-NEXT: };
+
+// CHECK: 1 'used' results:
+// CHECK-NEXT: refactoring action can't be initiated without a matching location

clang/test/Refactor/Edit/UnusedMethod.cpp

@@ -0,0 +1,20 @@
+// RUN: clang-refactor unused-method -location=test:%s %s -- 2>&1 | grep -v CHECK | FileCheck %s
+
+struct Bar {
+    /*loc used=*/void used() {}
+    /*loc unused=*/void unused() {}
+};
+
+void foo() {
+    Bar a;
+    a.used();
+}
+
+// CHECK: 1 'unused' results:
+// CHECK: struct Bar {
+// CHECK-NEXT: /*loc used=*/void used() {}
+// CHECK-NEXT: /*loc unused=*/
+// CHECK-NEXT: };
+
+// CHECK: 1 'used' results:
+// CHECK-NEXT: refactoring action can't be initiated without a matching location