prevent-abbreviations: Fix optional parameter (#764)

Author: fisker

rules/utils/rename-identifier.js

@@ -28,7 +28,7 @@ function renameIdentifier(identifier, name, fixer, sourceCode) {
 
 	// `typeAnnotation`
 	if (identifier.typeAnnotation) {
-		return fixer.replaceText(identifier, `${name}${sourceCode.getText(identifier.typeAnnotation)}`);
+		return fixer.replaceText(identifier, `${name}${identifier.optional ? '?' : ''}${sourceCode.getText(identifier.typeAnnotation)}`);
 	}
 
 	return fixer.replaceText(identifier, name);

test/prevent-abbreviations.js

@@ -1720,6 +1720,13 @@ typescriptRuleTester.run('prevent-abbreviations', rule, {
 			code,
 			output: code.replace('prop', 'property'),
 			errors: createErrors()
-		}))
+		})),
+
+		// #763
+		{
+			code: 'const foo = (extraParams?: string) => {}',
+			output: 'const foo = (extraParameters?: string) => {}',
+			errors: createErrors()
+		}
 	]
 });