Version 1.7.0-dev.3.2 .

editor/tools/plugins/com.google.dart.engine.services_test/src/com/google/dart/engine/services/internal/refactoring/ExtractMethodRefactoringImplTest.java

 /*
  * 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
@@ skipping 125 matching lines @@
     setSelectionFromStartEndComments();
     createRefactoring();
     // check conditions
     assertRefactoringStatus(
         refactoringStatus,
         RefactoringStatusSeverity.ERROR,
         "Created method will shadow method 'A.res'.",
         findRangeIdentifier("res() {} // marker"));
   }
 
+  public void test_bad_conflict_topLevel_alreadyDeclaresFunction() throws Exception {
+    parseTestUnit(
+        "// filler filler filler filler filler filler filler filler filler filler",
+        "void res() {}",
+        "main() {",
+        "// start",
+        "  print(0);",
+        "// end",
+        "}",
+        "");
+    setSelectionFromStartEndComments();
+    createRefactoring();
+    // check conditions
+    assertRefactoringStatus(
+        refactoringStatus,
+        RefactoringStatusSeverity.ERROR,
+        "Library already declares function with name 'res'.",
+        findRangeIdentifier("res() {}"));
+  }
+
   // TODO(scheglov) waiting for "library namespace" in Engine
 //  public void test_bad_conflict_method_willHideTopLevel() throws Exception {
 //    indexTestUnit(
 //        "// filler filler filler filler filler filler filler filler filler filler",
 //        "void res() {} // marker",
 //        "class B {",
 //        "  main() {",
 //        "// start",
 //        "    print(0);",
 //        "// end",
@@ skipping 13 matching lines @@
 //        findRangeIdentifier("res() {} // marker"));
 ////    assertTrue(refactoringStatus.hasError());
 ////    {
 ////      String msg = refactoringStatus.getMessageMatchingSeverity(RefactoringStatus.ERROR);
 ////      assertEquals(
 ////          "Usage of function 'res' in file 'Test/Test.dart' in library 'Test' will be shadowed by created function",
 ////          msg);
 ////    }
 //  }
 
-  public void test_bad_conflict_topLevel_alreadyDeclaresFunction() throws Exception {
-    parseTestUnit(
-        "// filler filler filler filler filler filler filler filler filler filler",
-        "void res() {}",
-        "main() {",
-        "// start",
-        "  print(0);",
-        "// end",
-        "}",
-        "");
-    setSelectionFromStartEndComments();
-    createRefactoring();
-    // check conditions
-    assertRefactoringStatus(
-        refactoringStatus,
-        RefactoringStatusSeverity.ERROR,
-        "Library already declares function with name 'res'.",
-        findRangeIdentifier("res() {}"));
-  }
-
   public void test_bad_conflict_topLevel_willHideInheritedMemberUsage() throws Exception {
     indexTestUnit(
         "// filler filler filler filler filler filler filler filler filler filler",
         "class A {",
         "  void res() {}",
         "}",
         "class B extends A {",
         "  foo() {",
         "    res(); // marker",
         "  }",
@@ skipping 2344 matching lines @@
   /**
    * Checks that all conditions are <code>OK</code> and applying {@link Change} to the
    * {@link #testUnit} is same source as given lines.
    */
   protected final void assertSuccessfulRefactoring(String... lines) throws Exception {
     assertRefactoringStatus(refactoringStatus, RefactoringStatusSeverity.OK, null);
     Change change = refactoring.createChange(pm);
     assertTestChangeResult(change, makeSource(lines));
   }
 
+  @Override
+  protected void tearDown() throws Exception {
+    refactoring = null;
+    refactoringStatus = null;
+    super.tearDown();
+  }
+
   /**
    * Creates {@link ExtractMethodRefactoring} in {@link #refactoring}.
    */
   private void createRefactoring() throws Exception {
     int selectionLength = selectionEnd - selectionStart;
     AssistContext context = new AssistContext(
         searchEngine,
         analysisContext,
         null,
         testSource,
@@ skipping 19 matching lines @@
   }
 
   /**
    * Sets selection to the start of the first occurrence of the given string.
    */
   private void setSelectionString(String pattern) {
     selectionStart = findOffset(pattern);
     selectionEnd = findEnd(pattern);
   }
 }