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

Side by Side Diff: pkg/analysis_server/test/services/refactoring/naming_conventions_test.dart

Issue 2879273002: Make server use the common protocol classes (Closed)
Patch Set: Created 3 years, 7 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
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 import 'package:analysis_server/protocol/protocol_generated.dart'
6 show RefactoringProblemSeverity;
7 import 'package:analysis_server/src/services/refactoring/naming_conventions.dart '; 5 import 'package:analysis_server/src/services/refactoring/naming_conventions.dart ';
8 import 'package:analysis_server/src/services/refactoring/refactoring.dart'; 6 import 'package:analysis_server/src/services/refactoring/refactoring.dart';
7 import 'package:analyzer_plugin/protocol/protocol_common.dart'
8 show RefactoringProblemSeverity;
9 import 'package:test_reflective_loader/test_reflective_loader.dart'; 9 import 'package:test_reflective_loader/test_reflective_loader.dart';
10 10
11 import 'abstract_refactoring.dart'; 11 import 'abstract_refactoring.dart';
12 12
13 main() { 13 main() {
14 defineReflectiveSuite(() { 14 defineReflectiveSuite(() {
15 defineReflectiveTests(NamingConventionsTest); 15 defineReflectiveTests(NamingConventionsTest);
16 }); 16 });
17 } 17 }
18 18
(...skipping 726 matching lines...) Expand 10 before | Expand all | Expand 10 after
745 void test_validateVariableName_OK_middleUnderscore() { 745 void test_validateVariableName_OK_middleUnderscore() {
746 assertRefactoringStatusOK(validateVariableName("new_name")); 746 assertRefactoringStatusOK(validateVariableName("new_name"));
747 } 747 }
748 748
749 void test_validateVariableName_trailingBlanks() { 749 void test_validateVariableName_trailingBlanks() {
750 assertRefactoringStatus( 750 assertRefactoringStatus(
751 validateVariableName("newName "), RefactoringProblemSeverity.FATAL, 751 validateVariableName("newName "), RefactoringProblemSeverity.FATAL,
752 expectedMessage: "Variable name must not start or end with a blank."); 752 expectedMessage: "Variable name must not start or end with a blank.");
753 } 753 }
754 } 754 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698