add arguments to constructor completions

pkg/analysis_server/test/completion_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.completion.support;
 
 import 'dart:async';
 import 'dart:collection';
 
 import 'package:unittest/unittest.dart';
@@ skipping 221 matching lines @@
 class L{var k;void.!1}''', <String>["1-k"]);
 
     buildTests('testCommentSnippets044', '''
 class List{}class XXX {XXX.fisk();}main() {main(); new !1}}''',
         <String>["1+List", "1+XXX.fisk"], failingTests: '1');
 
     buildTests('testCommentSnippets047', '''
 f(){int x;int y=!1;}''', <String>["1+x"]);
 
     buildTests('testCommentSnippets048', '''
-import 'dart:convert' as json;f() {var x=new js!1}''', <String>["1+json"]);
+import 'dart:convert' as json;f() {var x=new js!1}''', <String>["1+json"], failingTests: '1');
 
     buildTests('testCommentSnippets049', '''
 import 'dart:convert' as json;
 import 'dart:convert' as jxx;
 class JsonDecoderX{}
 f1() {var x=new !2j!1s!3}''', <String>[
       "1+json",
       "1+jxx",
       "2+json",
       "2+jxx",
       "2-JsonDecoder",
       "3+json",
       "3-jxx"
-    ]);
+    ], failingTests: '123');

[danrubel, 2015/03/03 15:58:07]

Just realized that this CL breaks new <prefix>.
Will address this in the next CL.

[danrubel, 2015/03/03 16:40:00]

https://codereview.chromium.org/972193002/

 
     buildTests('testCommentSnippets050', '''
 class xdr {
   xdr();
   const xdr.a(a,b,c);
   xdr.b();
   f() => 3;
 }
 class xa{}
 k() {
   new x!1dr().f();
   const x!2dr.!3a(1, 2, 3);
 }''', <String>[
       "1+xdr",
       "1+xa",
       "1+xdr.a",
       "1+xdr.b",
       "2-xa",
       "2-xdr",
       "2+xdr.a",
       "2-xdr.b",
       "3-b",
       "3+a"
-    ], failingTests: '123');
+    ], failingTests: '23');
 
     // Type propagation.
     buildTests('testCommentSnippets051', '''
 class String{int length(){} String toUpperCase(){} bool isEmpty(){}}class Map{getKeys(){}}
 void r() {
   var v;
   if (v is String) {
     v.!1length;
     v.!2getKeys;
   }
@@ skipping 327 matching lines @@
 class File {
   factory File(String path) => null;
   factory File.fromPath(Path path) => null;
 }
 f() => new Fil!1''', <String>[
       "1+File",
       "1+File.fromPath",
       "1+FileMode",
       "1+FileMode._internal1",
       "1+FileMode._internal"
-    ], failingTests: '1');
+    ]);
 
     buildTests('testCommentSnippets078', '''
 class Map{static from()=>null;clear(){}}void main() { Map.!1 }''',
         <String>["1+from", "1-clear"]); // static method, instance method
 
     buildTests('testCommentSnippets079', '''
 class Map{static from()=>null;clear(){}}void main() { Map s; s.!1 }''',
         <String>["1-from", "1+clear"]); // static method, instance method
 
     buildTests('testCommentSnippets080', '''
@@ skipping 1658 matching lines @@
       } else {
         ++expectedPassCount;
         tester(testName, () {
           CompletionTestCase test = new CompletionTestCase();
           return test.runTest(spec, extraFiles);
         });
       }
     }
   }
 }