Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(157)

Side by Side Diff: pkg/analysis_server/test/edit/format_test.dart

Issue 1099633004: Fix error handling for formatting and update the spec (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library test.edit.format; 5 library test.edit.format;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 8
9 import 'package:analysis_server/src/edit/edit_domain.dart'; 9 import 'package:analysis_server/src/edit/edit_domain.dart';
10 import 'package:analysis_server/src/plugin/server_plugin.dart'; 10 import 'package:analysis_server/src/plugin/server_plugin.dart';
11 import 'package:analysis_server/src/protocol.dart'; 11 import 'package:analysis_server/src/protocol.dart';
12 import 'package:analyzer/src/plugin/plugin_impl.dart'; 12 import 'package:analyzer/src/plugin/plugin_impl.dart';
13 import 'package:unittest/unittest.dart' hide ERROR; 13 import 'package:unittest/unittest.dart' hide ERROR;
14 14
15 import '../analysis_abstract.dart'; 15 import '../analysis_abstract.dart';
16 import '../mocks.dart';
16 import '../reflective_tests.dart'; 17 import '../reflective_tests.dart';
17 18
18 main() { 19 main() {
19 groupSep = ' | '; 20 groupSep = ' | ';
20 runReflectiveTests(FormatTest); 21 runReflectiveTests(FormatTest);
21 } 22 }
22 23
23 @reflectiveTest 24 @reflectiveTest
24 class FormatTest extends AbstractAnalysisTest { 25 class FormatTest extends AbstractAnalysisTest {
25 @override 26 @override
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 expect(edit.replacement, equals(''' 81 expect(edit.replacement, equals('''
81 main() { 82 main() {
82 int x = 3; 83 int x = 3;
83 } 84 }
84 ''')); 85 '''));
85 expect(formatResult.selectionOffset, equals(0)); 86 expect(formatResult.selectionOffset, equals(0));
86 expect(formatResult.selectionLength, equals(3)); 87 expect(formatResult.selectionLength, equals(3));
87 }); 88 });
88 } 89 }
89 90
91 Future test_withErrors() {
92 addTestFile('''
93 main() { int x =
94 ''');
95 return waitForTasksFinished().then((_) {
96 Request request = new EditFormatParams(testFile, 0, 3).toRequest('0');
97 Response response = handler.handleRequest(request);
98 expect(response, isResponseFailure('0'));
99 });
100 }
101
90 EditFormatResult _formatAt(int selectionOffset, int selectionLength) { 102 EditFormatResult _formatAt(int selectionOffset, int selectionLength) {
91 Request request = new EditFormatParams( 103 Request request = new EditFormatParams(
92 testFile, selectionOffset, selectionLength).toRequest('0'); 104 testFile, selectionOffset, selectionLength).toRequest('0');
93 Response response = handleSuccessfulRequest(request); 105 Response response = handleSuccessfulRequest(request);
94 return new EditFormatResult.fromResponse(response); 106 return new EditFormatResult.fromResponse(response);
95 } 107 }
96 } 108 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698