Issue 18296. Harden the name checks.

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';
@@ skipping 40 matching lines @@
   void test_validateClassName_empty() {
     assertRefactoringStatus(
         validateClassName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not be empty.");
   }
 
   void test_validateClassName_leadingBlanks() {
     assertRefactoringStatus(
         validateClassName(" NewName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not start or end with a blank.");
   }
 
   void test_validateClassName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateClassName("New-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not contain '-'.");
   }
 
   void test_validateClassName_notIdentifierStart() {
     assertRefactoringStatus(
         validateClassName("-NewName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateClassName("NewName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Class name must not start or end with a blank.");
   }
   void test_validateConstantName_OK() {
     assertRefactoringStatusOK(validateConstantName("NAME"));
   }
 
   void test_validateConstantName_OK_digit() {
     assertRefactoringStatusOK(validateConstantName("NAME2"));
   }
 
   void test_validateConstantName_OK_underscoreLeading() {
     assertRefactoringStatusOK(validateConstantName("_NAME"));
   }
 
   void test_validateConstantName_OK_underscoreMiddle() {
     assertRefactoringStatusOK(validateConstantName("MY_NEW_NAME"));
   }
 
   void test_validateConstantName_empty() {
     assertRefactoringStatus(
         validateConstantName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constant name must not be empty.");
   }
 
   void test_validateConstantName_leadingBlanks() {
     assertRefactoringStatus(
         validateConstantName(" NewName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constant name must not start or end with a blank.");
   }
 
   void test_validateConstantName_notAllCaps() {
     assertRefactoringStatus(
         validateConstantName("NewName"),
         RefactoringProblemSeverity.WARNING,
         expectedMessage: "Constant name should be all uppercase with underscores.");
   }
 
   void test_validateConstantName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateConstantName("NA-ME"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constant name must not contain '-'.");
   }
 
   void test_validateConstantName_notIdentifierStart() {
     assertRefactoringStatus(
         validateConstantName("99_RED_BALLOONS"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage:
             "Constant name must begin with an uppercase letter or underscore.");
   }
 
   void test_validateConstantName_null() {
     assertRefactoringStatus(
         validateConstantName(null),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constant name must not be null.");
   }
 
   void test_validateConstantName_trailingBlanks() {
     assertRefactoringStatus(
         validateConstantName("NewName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constant name must not start or end with a blank.");
   }
 
   void test_validateConstructorName_OK() {
     assertRefactoringStatusOK(validateConstructorName("newName"));
   }
 
   void test_validateConstructorName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateConstructorName("_newName"));
   }
 
   void test_validateConstructorName_doesNotStartWithLowerCase() {
     assertRefactoringStatus(
         validateConstructorName("NewName"),
         RefactoringProblemSeverity.WARNING,
         expectedMessage: "Constructor name should start with a lowercase letter.");
   }
 
   void test_validateConstructorName_empty() {
     assertRefactoringStatusOK(validateConstructorName(""));
   }
 
   void test_validateConstructorName_leadingBlanks() {
     assertRefactoringStatus(
         validateConstructorName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constructor name must not start or end with a blank.");
   }
 
   void test_validateConstructorName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateConstructorName("na-me"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constructor name must not contain '-'.");
   }
 
   void test_validateConstructorName_notIdentifierStart() {
     assertRefactoringStatus(
         validateConstructorName("2name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateConstructorName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Constructor name must not start or end with a blank.");
   }
 
   void test_validateFieldName_OK() {
     assertRefactoringStatusOK(validateFieldName("newName"));
   }
 
   void test_validateFieldName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateFieldName("_newName"));
   }
@@ skipping 12 matching lines @@
   void test_validateFieldName_empty() {
     assertRefactoringStatus(
         validateFieldName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not be empty.");
   }
 
   void test_validateFieldName_leadingBlanks() {
     assertRefactoringStatus(
         validateFieldName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not start or end with a blank.");
   }
 
   void test_validateFieldName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateFieldName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not contain '-'.");
   }
 
   void test_validateFieldName_notIdentifierStart() {
     assertRefactoringStatus(
         validateFieldName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateFieldName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Field name must not start or end with a blank.");
   }
 
   void test_validateFunctionName_OK() {
     assertRefactoringStatusOK(validateFunctionName("newName"));
   }
 
   void test_validateFunctionName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateFunctionName("_newName"));
   }
@@ skipping 12 matching lines @@
   void test_validateFunctionName_empty() {
     assertRefactoringStatus(
         validateFunctionName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not be empty.");
   }
 
   void test_validateFunctionName_leadingBlanks() {
     assertRefactoringStatus(
         validateFunctionName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not start or end with a blank.");
   }
 
   void test_validateFunctionName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateFunctionName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not contain '-'.");
   }
 
   void test_validateFunctionName_notIdentifierStart() {
     assertRefactoringStatus(
         validateFunctionName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateFunctionName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function name must not start or end with a blank.");
   }
 
   void test_validateFunctionTypeAliasName_OK() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("NewName"));
   }
 
   void test_validateFunctionTypeAliasName_OK_leadingDollar() {
     assertRefactoringStatusOK(validateFunctionTypeAliasName("\$NewName"));
   }
@@ skipping 17 matching lines @@
   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.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage:
             "Function type alias name must not start or end with a blank.");
   }
 
   void test_validateFunctionTypeAliasName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateFunctionTypeAliasName("New-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Function type alias name must not contain '-'.");
   }
 
   void test_validateFunctionTypeAliasName_notIdentifierStart() {
     assertRefactoringStatus(
         validateFunctionTypeAliasName("-NewName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateFunctionTypeAliasName("NewName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage:
             "Function type alias name must not start or end with a blank.");
   }
 
   void test_validateImportPrefixName_OK() {
     assertRefactoringStatusOK(validateImportPrefixName("newName"));
   }
 
   void test_validateImportPrefixName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateImportPrefixName("_newName"));
@@ skipping 10 matching lines @@
         expectedMessage: "Import prefix name should start with a lowercase letter.");
   }
 
   void test_validateImportPrefixName_empty() {
     assertRefactoringStatusOK(validateImportPrefixName(""));
   }
 
   void test_validateImportPrefixName_leadingBlanks() {
     assertRefactoringStatus(
         validateImportPrefixName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Import prefix name must not start or end with a blank.");
   }
 
   void test_validateImportPrefixName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateImportPrefixName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Import prefix name must not contain '-'.");
   }
 
   void test_validateImportPrefixName_notIdentifierStart() {
     assertRefactoringStatus(
         validateImportPrefixName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateImportPrefixName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Import prefix name must not start or end with a blank.");
   }
 
   void test_validateLabelName_OK() {
     assertRefactoringStatusOK(validateLabelName("newName"));
   }
 
   void test_validateLabelName_OK_leadingDollar() {
     assertRefactoringStatusOK(validateLabelName("\$newName"));
   }
@@ skipping 16 matching lines @@
   void test_validateLabelName_empty() {
     assertRefactoringStatus(
         validateLabelName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not be empty.");
   }
 
   void test_validateLabelName_leadingBlanks() {
     assertRefactoringStatus(
         validateLabelName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not start or end with a blank.");
   }
 
   void test_validateLabelName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateLabelName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not contain '-'.");
   }
 
   void test_validateLabelName_notIdentifierStart() {
     assertRefactoringStatus(
         validateLabelName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateLabelName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Label name must not start or end with a blank.");
   }
 
   void test_validateLibraryName_OK_oneIdentifier() {
     assertRefactoringStatusOK(validateLibraryName("name"));
   }
 
   void test_validateLibraryName_OK_severalIdentifiers() {
     assertRefactoringStatusOK(validateLibraryName("my.library.name"));
   }
 
   void test_validateLibraryName_blank() {
     assertRefactoringStatus(
         validateLibraryName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name must not be blank.");
     assertRefactoringStatus(
         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.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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.");
   }
 
   void test_validateLibraryName_leadingBlanks() {
     assertRefactoringStatus(
         validateLibraryName("my. name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name identifier must not start or end with a blank.");
   }
 
   void test_validateLibraryName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateLibraryName("my.ba-d.name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name identifier must not contain '-'.");
   }
 
   void test_validateLibraryName_notIdentifierStart() {
     assertRefactoringStatus(
         validateLibraryName("my.2bad.name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateLibraryName("my.bad .name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Library name identifier must not start or end with a blank.");
   }
 
   void test_validateMethodName_OK() {
     assertRefactoringStatusOK(validateMethodName("newName"));
   }
 
   void test_validateMethodName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateMethodName("_newName"));
   }
@@ skipping 12 matching lines @@
   void test_validateMethodName_empty() {
     assertRefactoringStatus(
         validateMethodName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not be empty.");
   }
 
   void test_validateMethodName_leadingBlanks() {
     assertRefactoringStatus(
         validateMethodName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not start or end with a blank.");
   }
 
   void test_validateMethodName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateMethodName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not contain '-'.");
   }
 
   void test_validateMethodName_notIdentifierStart() {
     assertRefactoringStatus(
         validateMethodName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateMethodName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Method name must not start or end with a blank.");
   }
 
   void test_validateParameterName_OK() {
     assertRefactoringStatusOK(validateParameterName("newName"));
   }
 
   void test_validateParameterName_OK_leadingUnderscore() {
     assertRefactoringStatusOK(validateParameterName("_newName"));
   }
@@ skipping 12 matching lines @@
   void test_validateParameterName_empty() {
     assertRefactoringStatus(
         validateParameterName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not be empty.");
   }
 
   void test_validateParameterName_leadingBlanks() {
     assertRefactoringStatus(
         validateParameterName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not start or end with a blank.");
   }
 
   void test_validateParameterName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateParameterName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not contain '-'.");
   }
 
   void test_validateParameterName_notIdentifierStart() {
     assertRefactoringStatus(
         validateParameterName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateParameterName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Parameter name must not start or end with a blank.");
   }
 
   void test_validateVariableName_OK() {
     assertRefactoringStatusOK(validateVariableName("newName"));
   }
 
   void test_validateVariableName_OK_leadingDollar() {
     assertRefactoringStatusOK(validateVariableName("\$newName"));
   }
@@ skipping 16 matching lines @@
   void test_validateVariableName_empty() {
     assertRefactoringStatus(
         validateVariableName(""),
         RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not be empty.");
   }
 
   void test_validateVariableName_leadingBlanks() {
     assertRefactoringStatus(
         validateVariableName(" newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not start or end with a blank.");
   }
 
   void test_validateVariableName_notIdentifierMiddle() {
     assertRefactoringStatus(
         validateVariableName("new-Name"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not contain '-'.");
   }
 
   void test_validateVariableName_notIdentifierStart() {
     assertRefactoringStatus(
         validateVariableName("2newName"),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         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_trailingBlanks() {
     assertRefactoringStatus(
         validateVariableName("newName "),
-        RefactoringProblemSeverity.ERROR,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "Variable name must not start or end with a blank.");
   }
 }