incremental improvement to top level code completion results

pkg/analysis_services/test/completion/top_level_computer_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library test.services.completion.toplevel;
 
 import 'package:analysis_services/completion/completion_suggestion.dart';
 import 'package:analysis_services/src/completion/top_level_computer.dart';
 import 'package:analysis_testing/reflective_tests.dart';
 import 'package:unittest/unittest.dart';
 
 import 'completion_test_util.dart';
 
 main() {
   groupSep = ' | ';
   runReflectiveTests(TopLevelComputerTest);
 }
 
 @ReflectiveTestCase()
 class TopLevelComputerTest extends AbstractCompletionTest {
 
-  test_class() {
-    addTestUnit('class B {boolean v;}');
+  void addTestUnit(String content) {
+    super.addTestUnit(content);
+    computer = new TopLevelComputer(searchEngine, testUnit);
+  }
+
+  test_class_1() {
+    addUnit('/testA.dart', 'var T1; class A {bool x;}');
+    addUnit('/testB.dart', 'class B {bool y;}');
+    addTestUnit('import "/testA.dart"; class C {bool v;^}');
     return compute().then((_) {
-      assertHasResult(CompletionSuggestionKind.CLASS, 'B');
+      assertHasResult(CompletionSuggestionKind.CLASS, 'A');
+      assertHasResult(CompletionSuggestionKind.CLASS, 'B', CompletionRelevance.LOW);
+      assertHasResult(CompletionSuggestionKind.CLASS, 'C');
+      assertHasResult(CompletionSuggestionKind.CLASS, 'Object');
+      assertHasResult(CompletionSuggestionKind.TOP_LEVEL_VARIABLE, 'T1');

[scheglov, 2014/08/07 02:42:26]

Is a top-level variable suggestion useful here?

I think we want only types at this point of AST.

[danrubel, 2014/08/08 20:28:39]

Top level variables in one library can be accessed by another, so I think we
should include them.

[scheglov, 2014/08/08 20:58:43]

Correct me if I'm wrong, but I don't think we can reference a top-level variable
inside of a class body.
We can only define some class member at this point.

[danrubel, 2014/08/09 10:23:03]

Ah, now I understand what you are saying. Yes, this is a misleading test which I
will correct. I have not yet begun filtering suggestions based upon completion
offset.

+      assertNoResult('x');
+      assertNoResult('y');
       assertNoResult('v');
     });
   }
-
-  @override
-  void setUp() {
-    super.setUp();
-    computer = new TopLevelComputer(searchEngine);
-  }
 }