Start running new completion tests.

Start running new completion tests.

R=brianwilkerson@google.com, danrubel@google.com

Committed: https://code.google.com/p/dart/source/detail?r=42210

[Paul Berry, 2014-12-08 22:59:57]

paulberry@google.com changed reviewers:
+ brianwilkerson@google.com, danrubel@google.com

[Paul Berry, 2014-12-08 22:59:57]

Several pieces of functionality still need to be implemented:

- Negative tests don't yet work properly, because they assume the completion
result is filtered, and it isn't (by design).  For now I've just disabled all
the negative tests.

- For some reason test case 'test018' prevents the Dart VM from exiting.  For
now I've just commented out this test.

- It would be nice to print out a summary of the number of tests which are
expected to pass/fail before running the tests, so that we can use this to track
implementation progress.

I plan to address these in future CL's.

Dan: note that once this lands, we'll have to update the "failingTests:"
arguments as bugs are fixed.  If a test case is expected to fail and it passes,
this will be reported as a failure, so it should be pretty easy to keep these
arguments up to date.

[danrubel, 2014-12-08 23:17:06]

lgtm

[Brian Wilkerson, 2014-12-09 00:12:28]

LGTM

https://codereview.chromium.org/781333003/diff/1/pkg/analysis_server/test/com...
File pkg/analysis_server/test/completion_test.dart (right):

https://codereview.chromium.org/781333003/diff/1/pkg/analysis_server/test/com...
pkg/analysis_server/test/completion_test.dart:2137: // exiting, for some unknown
reason.  TODO(paulberry): fix this.
It didn't have a problem when I ran it locally (before these changes). I'm very
interested to know what changed.

[Paul Berry, 2014-12-09 16:44:19]

Committed patchset #1 (id:1) manually as 42210 (presubmit successful).