Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(52)

Unified Diff: editor/tools/plugins/com.google.dart.tools.ui_test/src-ui/editor/refactoring/RenameRefactoringTest.java

Issue 18548007: UI tests for 'Rename' and 'Extract Local Variable' refactorings. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fixes for review comments Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.tools.ui_test/src-ui/editor/refactoring/RenameRefactoringTest.java
diff --git a/editor/tools/plugins/com.google.dart.tools.ui_test/src-ui/editor/refactoring/RenameRefactoringTest.java b/editor/tools/plugins/com.google.dart.tools.ui_test/src-ui/editor/refactoring/RenameRefactoringTest.java
new file mode 100644
index 0000000000000000000000000000000000000000..16befacf3566779090d952b26c738198fed71c86
--- /dev/null
+++ b/editor/tools/plugins/com.google.dart.tools.ui_test/src-ui/editor/refactoring/RenameRefactoringTest.java
@@ -0,0 +1,163 @@
+/*
+ * Copyright (c) 2013, the Dart project authors.
+ *
+ * Licensed under the Eclipse Public License v1.0 (the "License"); you may not use this file except
+ * in compliance with the License. You may obtain a copy of the License at
+ *
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Unless required by applicable law or agreed to in writing, software distributed under the License
+ * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
+ * or implied. See the License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package editor.refactoring;
+
+import com.google.dart.tools.ui.actions.JdtActionConstants;
+import com.google.dart.ui.test.driver.ShellClosedOperation;
+import com.google.dart.ui.test.driver.ShellOperation;
+import com.google.dart.ui.test.driver.Operation;
+import com.google.dart.ui.test.helpers.WizardDialogHelper;
+import com.google.dart.ui.test.util.UiContext;
+
+import editor.AbstractDartEditorTest;
+
+
+import org.eclipse.jface.action.IAction;
+
+import java.util.concurrent.TimeUnit;
+
+/**
+ * Test for the "Rename" refactoring.
+ */
+public final class RenameRefactoringTest extends AbstractDartEditorTest {
+ private static class WizardHelper extends WizardDialogHelper {
+ public WizardHelper(UiContext context) {
+ super(context);
+ }
+
+ public void setName(String name) {
+ context.getTextByLabel("New name:").setText(name);
+ }
+ }
+
+ public void test_renameClass() throws Exception {
+ openTestEditor(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "class A {}",
+ "main() {",
+ " A v = new A();",
+ "}");
+ findAndStartRename("A v");
+ // animate wizard dialog
+ String wizardName = "Rename Class";
+ addOperation(new ShellOperation(wizardName) {
+ @Override
+ public void run(UiContext context) throws Exception {
+ WizardHelper helper = new WizardHelper(context);
+ // invalid name
+ helper.setName("-A");
+ helper.assertMessage("Class name must not start with '-'.");
+ // set new name
+ helper.setName("A2");
+ helper.assertNoMessage();
+ // done
+ helper.closeOK();
+ }
+ });
+ addOperation(new ShellClosedOperation(wizardName));
+ // validate result
+ runUiOperations(60, TimeUnit.SECONDS);
+ assertTestUnitContent(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "class A2 {}",
+ "main() {",
+ " A2 v = new A2();",
+ "}");
+ }
+
+ public void test_renameField() throws Exception {
+ openTestEditor(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "class A {",
+ " int f;",
+ "}",
+ "foo(A a) {",
+ " a.f = 42;",
+ "}");
+ findAndStartRename("f;");
+ // animate wizard dialog
+ String wizardName = "Rename Field";
+ addOperation(new ShellOperation(wizardName) {
+ @Override
+ public void run(UiContext context) throws Exception {
+ WizardHelper helper = new WizardHelper(context);
+ // set new name
+ helper.setName("newName");
+ helper.assertNoMessage();
+ // done
+ helper.closeOK();
+ }
+ });
+ addOperation(new ShellClosedOperation(wizardName));
+ // validate result
+ runUiOperations(60, TimeUnit.SECONDS);
+ assertTestUnitContent(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "class A {",
+ " int newName;",
+ "}",
+ "foo(A a) {",
+ " a.newName = 42;",
+ "}");
+ }
+
+ public void test_renameTopLevelVariable() throws Exception {
+ openTestEditor(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "var myVar;",
+ "main() {",
+ " myVar = 42;",
+ "}");
+ findAndStartRename("myVar;");
+ // animate wizard dialog
+ String wizardName = "Rename Top-Level Variable";
+ addOperation(new ShellOperation(wizardName) {
+ @Override
+ public void run(UiContext context) throws Exception {
+ WizardHelper helper = new WizardHelper(context);
+ // set new name
+ helper.setName("newName");
+ helper.assertNoMessage();
+ // done
+ helper.closeOK();
+ }
+ });
+ addOperation(new ShellClosedOperation(wizardName));
+ // validate result
+ runUiOperations(60, TimeUnit.SECONDS);
+ assertTestUnitContent(
+ "// filler filler filler filler filler filler filler filler filler filler",
+ "var newName;",
+ "main() {",
+ " newName = 42;",
+ "}");
+ }
+
+ private void findAndStartRename(String pattern) throws Exception {
+ selectOffset(pattern);
+ // run "Rename" action
+ final IAction renameAction = getEditorAction(JdtActionConstants.RENAME);
+ addOperation(new Operation() {
+ @Override
+ public boolean isReady(UiContext context) throws Exception {
+ return renameAction.isEnabled();
+ }
+
+ @Override
+ public void run(UiContext context) throws Exception {
+ renameAction.run();
+ }
+ });
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698