[Fix] img-redundant-alt: fixed multibyte character support

Fixes #969

Author: makotot

__tests__/src/rules/img-redundant-alt-test.js

@@ -74,6 +74,7 @@ ruleTester.run('img-redundant-alt', rule, {
     { code: '<img alt="ImageMagick" />;' },
     { code: '<Image alt="Photo of a friend" />' },
     { code: '<Image alt="Foo" />', settings: componentsSettings },
+    { code: '<img alt="画像" />', options: [{ words: ['イメージ'] }] },
   )).map(parserOptionsMapper),
   invalid: parsers.all([].concat(
     { code: '<img alt="Photo of friend." />;', errors: [expectedError] },
@@ -129,5 +130,8 @@ ruleTester.run('img-redundant-alt', rule, {
     { code: '<img alt="Word2" />;', options: array, errors: [expectedError] },
     { code: '<Image alt="Word1" />;', options: array, errors: [expectedError] },
     { code: '<Image alt="Word2" />;', options: array, errors: [expectedError] },
+
+    { code: '<img alt="イメージ" />', options: [{ words: ['イメージ'] }], errors: [expectedError] },
+    { code: '<img alt="イメージです" />', options: [{ words: ['イメージ'] }], errors: [expectedError] },
   )).map(parserOptionsMapper),
 });

package.json

@@ -88,7 +88,8 @@
     "language-tags": "^1.0.9",
     "minimatch": "^3.1.2",
     "object.entries": "^1.1.7",
-    "object.fromentries": "^2.0.7"
+    "object.fromentries": "^2.0.7",
+    "string.prototype.includes": "^2.0.0"
   },
   "peerDependencies": {
     "eslint": "^3 || ^4 || ^5 || ^6 || ^7 || ^8"

src/rules/img-redundant-alt.js

@@ -8,6 +8,8 @@
 // ----------------------------------------------------------------------------
 
 import { getProp, getLiteralPropValue } from 'jsx-ast-utils';
+import includes from 'array-includes';
+import stringIncludes from 'string.prototype.includes';
 import { generateObjSchema, arraySchema } from '../util/schemas';
 import getElementType from '../util/getElementType';
 import isHiddenFromScreenReader from '../util/isHiddenFromScreenReader';
@@ -25,6 +27,16 @@ const schema = generateObjSchema({
   words: arraySchema,
 });
 
+function containsRedundantWord(value, redundantWords) {
+  const lowercaseRedundantWords = redundantWords.map((redundantWord) => redundantWord.toLowerCase());
+  const isASCII = /[\x20-\x7F]+/.test(value);
+
+  if (isASCII) {
+    return value.split(/\s+/).some((valueWord) => includes(lowercaseRedundantWords, valueWord.toLowerCase()));
+  }
+  return lowercaseRedundantWords.some((redundantWord) => stringIncludes(value.toLowerCase(), redundantWord));
+}
+
 export default {
   meta: {
     docs: {
@@ -63,7 +75,7 @@ export default {
         const redundantWords = REDUNDANT_WORDS.concat(words);
 
         if (typeof value === 'string' && isVisible) {
-          const hasRedundancy = new RegExp(`(?!{)\\b(${redundantWords.join('|')})\\b(?!})`, 'i').test(value);
+          const hasRedundancy = containsRedundantWord(value, redundantWords);
 
           if (hasRedundancy === true) {
             context.report({