Extract renameIdentifier to utils

rules/prevent-abbreviations.js

@@ -7,6 +7,11 @@ const getDocumentationUrl = require('./utils/get-documentation-url');
 const avoidCapture = require('./utils/avoid-capture');
 const cartesianProductSamples = require('./utils/cartesian-product-samples');
 const isSameNode = require('./utils/is-same-node');
+const isShorthandPropertyIdentifier = require('./utils/is-shorthand-property-identifier');
+const isAssignmentPatternShorthandPropertyIdentifier = require('./utils/is-assignment-pattern-shorthand-property-identifier');
+const isShorthandImportIdentifier = require('./utils/is-shorthand-import-identifier');
+const isShorthandExportIdentifier = require('./utils/is-shorthand-export-identifier');
+const renameIdentifier = require('./utils/rename-identifier')
 
 const isUpperCase = string => string === string.toUpperCase();
 const isUpperFirst = string => isUpperCase(string[0]);
@@ -398,70 +403,6 @@ const shouldFix = variable => {
 	return !variableIdentifiers(variable).some(isExportedIdentifier);
 };
 
-const isShorthandPropertyIdentifier = identifier => {
-	return (
-		identifier.parent.type === 'Property' &&
-		identifier.parent.shorthand &&
-		isSameNode(identifier, identifier.parent.key)
-	);
-};
-
-const isAssignmentPatternShorthandPropertyIdentifier = identifier => {
-	return (
-		identifier.parent.type === 'AssignmentPattern' &&
-		identifier.parent.left === identifier &&
-		identifier.parent.parent.type === 'Property' &&
-		isSameNode(identifier, identifier.parent.parent.key) &&
-		identifier.parent.parent.value === identifier.parent &&
-		identifier.parent.parent.shorthand
-	);
-};
-
-const isShorthandImportIdentifier = identifier => {
-	return (
-		identifier.parent.type === 'ImportSpecifier' &&
-		identifier.parent.imported.name === identifier.name &&
-		identifier.parent.local.name === identifier.name
-	);
-};
-
-const isShorthandExportIdentifier = identifier => {
-	return (
-		identifier.parent.type === 'ExportSpecifier' &&
-		identifier.parent.exported.name === identifier.name &&
-		identifier.parent.local.name === identifier.name
-	);
-};
-
-const fixIdentifier = (fixer, replacement, sourceCode) => identifier => {
-	if (
-		isShorthandPropertyIdentifier(identifier) ||
-		isAssignmentPatternShorthandPropertyIdentifier(identifier)
-	) {
-		return fixer.replaceText(identifier, `${identifier.name}: ${replacement}`);
-	}
-
-	if (isShorthandImportIdentifier(identifier)) {
-		return fixer.replaceText(identifier, `${identifier.name} as ${replacement}`);
-	}
-
-	if (isShorthandExportIdentifier(identifier)) {
-		return fixer.replaceText(identifier, `${replacement} as ${identifier.name}`);
-	}
-
-	// `TypeParameter` default value
-	if (identifier.default) {
-		return fixer.replaceText(identifier, `${replacement} = ${sourceCode.getText(identifier.default)}`);
-	}
-
-	// `typeAnnotation`
-	if (identifier.typeAnnotation) {
-		return fixer.replaceText(identifier, `${replacement}${sourceCode.getText(identifier.typeAnnotation)}`);
-	}
-
-	return fixer.replaceText(identifier, replacement);
-};
-
 const isDefaultOrNamespaceImportName = identifier => {
 	if (
 		identifier.parent.type === 'ImportDefaultSpecifier' &&
@@ -694,7 +635,7 @@ const create = context => {
 
 			problem.fix = fixer => {
 				return variableIdentifiers(variable)
-					.map(fixIdentifier(fixer, replacement, sourceCode));
+					.map(identifier => renameIdentifier(identifier, replacement, fixer, sourceCode));
 			};
 		}
 

rules/utils/is-assignment-pattern-shorthand-property-identifier.js

@@ -0,0 +1,11 @@
+'use strict';
+
+const isSameNode = require('./is-same-node');
+
+module.exports = identifier =>
+	identifier.parent.type === 'AssignmentPattern' &&
+	identifier.parent.left === identifier &&
+	identifier.parent.parent.type === 'Property' &&
+	isSameNode(identifier, identifier.parent.parent.key) &&
+	identifier.parent.parent.value === identifier.parent &&
+	identifier.parent.parent.shorthand

rules/utils/is-shorthand-export-identifier.js

@@ -0,0 +1,6 @@
+'use strict';
+
+module.exports = identifier =>
+		identifier.parent.type === 'ExportSpecifier' &&
+		identifier.parent.exported.name === identifier.name &&
+		identifier.parent.local.name === identifier.name

rules/utils/is-shorthand-import-identifier.js

@@ -0,0 +1,8 @@
+'use strict';
+
+const isSameNode = require('./is-same-node');
+
+module.exports = identifier =>
+	identifier.parent.type === 'ImportSpecifier' &&
+	identifier.parent.imported.name === identifier.name &&
+	identifier.parent.local.name === identifier.name

rules/utils/is-shorthand-property-identifier.js

@@ -0,0 +1,8 @@
+'use strict';
+
+const isSameNode = require('./is-same-node');
+
+module.exports = identifier =>
+	identifier.parent.type === 'Property' &&
+	identifier.parent.shorthand &&
+	isSameNode(identifier, identifier.parent.key);

rules/utils/rename-identifier.js

@@ -0,0 +1,37 @@
+'use strict';
+
+const isShorthandPropertyIdentifier = require('./is-shorthand-property-identifier');
+const isAssignmentPatternShorthandPropertyIdentifier = require('./is-assignment-pattern-shorthand-property-identifier');
+const isShorthandImportIdentifier = require('./is-shorthand-import-identifier');
+const isShorthandExportIdentifier = require('./is-shorthand-export-identifier');
+
+function renameIdentifier(identifier, name, fixer, sourceCode) {
+	if (
+		isShorthandPropertyIdentifier(identifier) ||
+		isAssignmentPatternShorthandPropertyIdentifier(identifier)
+	) {
+		return fixer.replaceText(identifier, `${identifier.name}: ${name}`);
+	}
+
+	if (isShorthandImportIdentifier(identifier)) {
+		return fixer.replaceText(identifier, `${identifier.name} as ${name}`);
+	}
+
+	if (isShorthandExportIdentifier(identifier)) {
+		return fixer.replaceText(identifier, `${name} as ${identifier.name}`);
+	}
+
+	// `TypeParameter` default value
+	if (identifier.default) {
+		return fixer.replaceText(identifier, `${name} = ${sourceCode.getText(identifier.default)}`);
+	}
+
+	// `typeAnnotation`
+	if (identifier.typeAnnotation) {
+		return fixer.replaceText(identifier, `${name}${sourceCode.getText(identifier.typeAnnotation)}`);
+	}
+
+	return fixer.replaceText(identifier, name);
+}
+
+module.exports = renameIdentifier