More fixes for failures on the Windows bot

pkg/analysis_server/test/services/refactoring/naming_conventions_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.services.refactoring.naming_conventions;
 
 import 'package:analysis_server/src/protocol.dart'
     show RefactoringProblemSeverity;
 import 'package:analysis_server/src/services/refactoring/naming_conventions.dart';
 import 'package:analysis_server/src/services/refactoring/refactoring.dart';
 import 'package:test_reflective_loader/test_reflective_loader.dart';
-import 'package:unittest/unittest.dart';
 
+import '../../utils.dart';
 import 'abstract_refactoring.dart';
 
 main() {
-  groupSep = ' | ';
+  initializeTestEnvironment();
   defineReflectiveTests(NamingConventionsTest);
 }
 
 @reflectiveTest
 class NamingConventionsTest extends RefactoringTest {
   @override
   Refactoring get refactoring => null;
 
   void test_validateClassName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
@@ skipping 15 matching lines @@
 
   void test_validateClassName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateClassName("New-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not contain '-'.");
   }
 
   void test_validateClassName_notIdentifierStart() {
     assertRefactoringStatus(
         validateClassName("-NewName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Class name must begin with an uppercase letter or underscore.");
+        expectedMessage:
+            "Class name must begin with an uppercase letter or underscore.");
   }
 
   void test_validateClassName_null() {
     assertRefactoringStatus(
         validateClassName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not be null.");
   }
 
   void test_validateClassName_OK() {
     assertRefactoringStatusOK(validateClassName("NewName"));
@@ skipping 13 matching lines @@
 
   void test_validateClassName_trailingBlanks() {
     assertRefactoringStatus(
         validateClassName("NewName "), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not start or end with a blank.");
   }
 
   void test_validateConstructorName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateConstructorName("NewName"), RefactoringProblemSeverity.WARNING,
-        expectedMessage: "Constructor name should start with a lowercase letter.");
+        expectedMessage:
+            "Constructor name should start with a lowercase letter.");
   }
 
   void test_validateConstructorName_empty() {
     assertRefactoringStatusOK(validateConstructorName(""));
   }
 
   void test_validateConstructorName_leadingBlanks() {
     assertRefactoringStatus(
         validateConstructorName(" newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Constructor name must not start or end with a blank.");
+        expectedMessage:
+            "Constructor name must not start or end with a blank.");
   }
 
   void test_validateConstructorName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateConstructorName("na-me"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constructor name must not contain '-'.");
   }
 
   void test_validateConstructorName_notIdentifierStart() {
     assertRefactoringStatus(
         validateConstructorName("2name"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Constructor name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Constructor name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateConstructorName_null() {
     assertRefactoringStatus(
         validateConstructorName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constructor name must not be null.");
   }
 
   void test_validateConstructorName_OK() {
     assertRefactoringStatusOK(validateConstructorName("newName"));
   }
 
   void test_validateConstructorName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateConstructorName("_newName"));
   }
 
   void test_validateConstructorName_trailingBlanks() {
     assertRefactoringStatus(
         validateConstructorName("newName "), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Constructor name must not start or end with a blank.");
+        expectedMessage:
+            "Constructor name must not start or end with a blank.");
   }
 
   void test_validateFieldName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateFieldName("NewName"), RefactoringProblemSeverity.WARNING,
         expectedMessage: "Field name should start with a lowercase letter.");
   }
 
   void test_validateFieldName_empty() {
     assertRefactoringStatus(
         validateFieldName(""), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not be empty.");
   }
 
   void test_validateFieldName_leadingBlanks() {
     assertRefactoringStatus(
         validateFieldName(" newName"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not start or end with a blank.");
   }
 
   void test_validateFieldName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateFieldName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not contain '-'.");
   }
 
   void test_validateFieldName_notIdentifierStart() {
     assertRefactoringStatus(
         validateFieldName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Field name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Field name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateFieldName_null() {
     assertRefactoringStatus(
         validateFieldName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not be null.");
   }
 
   void test_validateFieldName_OK() {
     assertRefactoringStatusOK(validateFieldName("newName"));
@@ skipping 33 matching lines @@
 
   void test_validateFunctionName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateFunctionName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not contain '-'.");
   }
 
   void test_validateFunctionName_notIdentifierStart() {
     assertRefactoringStatus(
         validateFunctionName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Function name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Function name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateFunctionName_null() {
     assertRefactoringStatus(
         validateFunctionName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not be null.");
   }
 
   void test_validateFunctionName_OK() {
     assertRefactoringStatusOK(validateFunctionName("newName"));
   }
 
   void test_validateFunctionName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateFunctionName("_newName"));
   }
 
   void test_validateFunctionName_OK_middleUnderscore() {
     assertRefactoringStatusOK(validateFunctionName("new_name"));
   }
 
   void test_validateFunctionName_trailingBlanks() {
     assertRefactoringStatus(
         validateFunctionName("newName "), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not start or end with a blank.");
   }
 
   void test_validateFunctionTypeAliasName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(validateFunctionTypeAliasName("newName"),
         RefactoringProblemSeverity.WARNING,
-        expectedMessage: "Function type alias name should start with an uppercase letter.");
+        expectedMessage:
+            "Function type alias name should start with an uppercase letter.");
   }
 
   void test_validateFunctionTypeAliasName_empty() {
     assertRefactoringStatus(
         validateFunctionTypeAliasName(""), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function type alias name must not be empty.");
   }
 
   void test_validateFunctionTypeAliasName_leadingBlanks() {
     assertRefactoringStatus(validateFunctionTypeAliasName(" NewName"),
         RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Function type alias name must not start or end with a blank.");
+        expectedMessage:
+            "Function type alias name must not start or end with a blank.");
   }
 
   void test_validateFunctionTypeAliasName_notIdentifierMiddle() {
     assertRefactoringStatus(validateFunctionTypeAliasName("New-Name"),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function type alias name must not contain '-'.");
   }
 
   void test_validateFunctionTypeAliasName_notIdentifierStart() {
     assertRefactoringStatus(validateFunctionTypeAliasName("-NewName"),
         RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Function type alias name must begin with an uppercase letter or underscore.");
+        expectedMessage:
+            "Function type alias name must begin with an uppercase letter or underscore.");
   }
 
   void test_validateFunctionTypeAliasName_null() {
     assertRefactoringStatus(
         validateFunctionTypeAliasName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function type alias name must not be null.");
   }
 
   void test_validateFunctionTypeAliasName_OK() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("NewName"));
   }
 
   void test_validateFunctionTypeAliasName_OK_leadingDollar() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("\$NewName"));
   }
 
   void test_validateFunctionTypeAliasName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("_NewName"));
   }
 
   void test_validateFunctionTypeAliasName_OK_middleDollar() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("New\$Name"));
   }
 
   void test_validateFunctionTypeAliasName_trailingBlanks() {
     assertRefactoringStatus(validateFunctionTypeAliasName("NewName "),
         RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Function type alias name must not start or end with a blank.");
+        expectedMessage:
+            "Function type alias name must not start or end with a blank.");
   }
 
   void test_validateImportPrefixName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateImportPrefixName("NewName"), RefactoringProblemSeverity.WARNING,
-        expectedMessage: "Import prefix name should start with a lowercase letter.");
+        expectedMessage:
+            "Import prefix name should start with a lowercase letter.");
   }
 
   void test_validateImportPrefixName_empty() {
     assertRefactoringStatusOK(validateImportPrefixName(""));
   }
 
   void test_validateImportPrefixName_leadingBlanks() {
     assertRefactoringStatus(
         validateImportPrefixName(" newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Import prefix name must not start or end with a blank.");
+        expectedMessage:
+            "Import prefix name must not start or end with a blank.");
   }
 
   void test_validateImportPrefixName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateImportPrefixName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Import prefix name must not contain '-'.");
   }
 
   void test_validateImportPrefixName_notIdentifierStart() {
     assertRefactoringStatus(
         validateImportPrefixName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Import prefix name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Import prefix name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateImportPrefixName_null() {
     assertRefactoringStatus(
         validateImportPrefixName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Import prefix name must not be null.");
   }
 
   void test_validateImportPrefixName_OK() {
     assertRefactoringStatusOK(validateImportPrefixName("newName"));
   }
 
   void test_validateImportPrefixName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateImportPrefixName("_newName"));
   }
 
   void test_validateImportPrefixName_OK_middleUnderscore() {
     assertRefactoringStatusOK(validateImportPrefixName("new_name"));
   }
 
   void test_validateImportPrefixName_trailingBlanks() {
     assertRefactoringStatus(
         validateImportPrefixName("newName "), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Import prefix name must not start or end with a blank.");
+        expectedMessage:
+            "Import prefix name must not start or end with a blank.");
   }
 
   void test_validateLabelName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateLabelName("NewName"), RefactoringProblemSeverity.WARNING,
         expectedMessage: "Label name should start with a lowercase letter.");
   }
 
   void test_validateLabelName_empty() {
     assertRefactoringStatus(
         validateLabelName(""), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not be empty.");
   }
 
   void test_validateLabelName_leadingBlanks() {
     assertRefactoringStatus(
         validateLabelName(" newName"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not start or end with a blank.");
   }
 
   void test_validateLabelName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateLabelName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not contain '-'.");
   }
 
   void test_validateLabelName_notIdentifierStart() {
     assertRefactoringStatus(
         validateLabelName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Label name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Label name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateLabelName_null() {
     assertRefactoringStatus(
         validateLabelName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not be null.");
   }
 
   void test_validateLabelName_OK() {
     assertRefactoringStatusOK(validateLabelName("newName"));
@@ skipping 25 matching lines @@
         validateLibraryName(" "), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name must not be blank.");
   }
 
   void test_validateLibraryName_blank_identifier() {
     assertRefactoringStatus(
         validateLibraryName("my..name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name identifier must not be empty.");
     assertRefactoringStatus(
         validateLibraryName("my. .name"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Library name identifier must not start or end with a blank.");
+        expectedMessage:
+            "Library name identifier must not start or end with a blank.");
   }
 
   void test_validateLibraryName_hasUpperCase() {
     assertRefactoringStatus(
         validateLibraryName("my.newName"), RefactoringProblemSeverity.WARNING,
-        expectedMessage: "Library name should consist of lowercase identifier separated by dots.");
+        expectedMessage:
+            "Library name should consist of lowercase identifier separated by dots.");
   }
 
   void test_validateLibraryName_leadingBlanks() {
     assertRefactoringStatus(
         validateLibraryName("my. name"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Library name identifier must not start or end with a blank.");
+        expectedMessage:
+            "Library name identifier must not start or end with a blank.");
   }
 
   void test_validateLibraryName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateLibraryName("my.ba-d.name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name identifier must not contain '-'.");
   }
 
   void test_validateLibraryName_notIdentifierStart() {
     assertRefactoringStatus(
         validateLibraryName("my.2bad.name"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Library name identifier must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Library name identifier must begin with a lowercase letter or underscore.");
   }
 
   void test_validateLibraryName_null() {
     assertRefactoringStatus(
         validateLibraryName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name must not be null.");
   }
 
   void test_validateLibraryName_OK_oneIdentifier() {
     assertRefactoringStatusOK(validateLibraryName("name"));
   }
 
   void test_validateLibraryName_OK_severalIdentifiers() {
     assertRefactoringStatusOK(validateLibraryName("my.library.name"));
   }
 
   void test_validateLibraryName_trailingBlanks() {
     assertRefactoringStatus(
         validateLibraryName("my.bad .name"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Library name identifier must not start or end with a blank.");
+        expectedMessage:
+            "Library name identifier must not start or end with a blank.");
   }
 
   void test_validateMethodName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateMethodName("NewName"), RefactoringProblemSeverity.WARNING,
         expectedMessage: "Method name should start with a lowercase letter.");
   }
 
   void test_validateMethodName_empty() {
     assertRefactoringStatus(
@@ skipping 15 matching lines @@
 
   void test_validateMethodName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateMethodName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not contain '-'.");
   }
 
   void test_validateMethodName_notIdentifierStart() {
     assertRefactoringStatus(
         validateMethodName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Method name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Method name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateMethodName_null() {
     assertRefactoringStatus(
         validateMethodName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not be null.");
   }
 
   void test_validateMethodName_OK() {
     assertRefactoringStatusOK(validateMethodName("newName"));
   }
 
   void test_validateMethodName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateMethodName("_newName"));
   }
 
   void test_validateMethodName_OK_middleUnderscore() {
     assertRefactoringStatusOK(validateMethodName("new_name"));
   }
 
   void test_validateMethodName_trailingBlanks() {
     assertRefactoringStatus(
         validateMethodName("newName "), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not start or end with a blank.");
   }
 
   void test_validateParameterName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateParameterName("NewName"), RefactoringProblemSeverity.WARNING,
-        expectedMessage: "Parameter name should start with a lowercase letter.");
+        expectedMessage:
+            "Parameter name should start with a lowercase letter.");
   }
 
   void test_validateParameterName_empty() {
     assertRefactoringStatus(
         validateParameterName(""), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not be empty.");
   }
 
   void test_validateParameterName_leadingBlanks() {
     assertRefactoringStatus(
         validateParameterName(" newName"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not start or end with a blank.");
   }
 
   void test_validateParameterName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateParameterName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not contain '-'.");
   }
 
   void test_validateParameterName_notIdentifierStart() {
     assertRefactoringStatus(
         validateParameterName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Parameter name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Parameter name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateParameterName_null() {
     assertRefactoringStatus(
         validateParameterName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not be null.");
   }
 
   void test_validateParameterName_OK() {
     assertRefactoringStatusOK(validateParameterName("newName"));
@@ skipping 33 matching lines @@
 
   void test_validateVariableName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateVariableName("new-Name"), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not contain '-'.");
   }
 
   void test_validateVariableName_notIdentifierStart() {
     assertRefactoringStatus(
         validateVariableName("2newName"), RefactoringProblemSeverity.FATAL,
-        expectedMessage: "Variable name must begin with a lowercase letter or underscore.");
+        expectedMessage:
+            "Variable name must begin with a lowercase letter or underscore.");
   }
 
   void test_validateVariableName_null() {
     assertRefactoringStatus(
         validateVariableName(null), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not be null.");
   }
 
   void test_validateVariableName_OK() {
     assertRefactoringStatusOK(validateVariableName("newName"));
@@ skipping 10 matching lines @@
   void test_validateVariableName_OK_middleUnderscore() {
     assertRefactoringStatusOK(validateVariableName("new_name"));
   }
 
   void test_validateVariableName_trailingBlanks() {
     assertRefactoringStatus(
         validateVariableName("newName "), RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not start or end with a blank.");
   }
 }