(TBR) update completion test to expect pass now that this is fixed #23001

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 2101 matching lines @@
 import 'dart:math' as math show PI;
 main() {
   math.!1
 }''',
         <String>["1+PI", "1-LN10"],
         failingTests: '1');
 
     sources.clear();
     sources["/lib.dart"] = '''
 library lib
-class _A 
+class _A
   foo() {}
 
 class A extends _A {
 }''';
     buildTests(
         'test_memberOfPrivateClass_otherLibrary',
         '''
 import 'lib.dart';
 main(A a) {
   a.!1
@@ skipping 711 matching lines @@
   test() {
     topvar.!1top();
     field.!2top();
   }
 }''',
         <String>["1+top", "2+top"],
         failingTests: '12');
 
     // test analysis of untyped fields and top-level vars
     buildTests('test035', '''class Y {final x='hi';mth() {x.!1length;}}''',
-        <String>["1+length"], failingTests: '1');
+        <String>["1+length"]);
 
     // TODO(scheglov) decide what to do with Type for untyped field (not
     // supported by the new store)
     // test analysis of untyped fields and top-level vars
     buildTests(
         'test036',
         '''
 class A1 {
   var field;
   A1() : field = 0;
@@ skipping 127 matching lines @@
       } else {
         ++expectedPassCount;
         tester(testName, () {
           CompletionTestCase test = new CompletionTestCase();
           return test.runTest(spec, extraFiles);
         });
       }
     }
   }
 }