fix: Catch issues in front of an ellipsis

sapegin · sapegin · commit de9953448325 · 2024-08-28T09:54:35.000+02:00
diff --git a/index.js b/index.js
@@ -123,7 +123,7 @@ function filterDict(rules, excludedWords) {
  */
 function getRegExp(word) {
 	const wordPattern = escapeRegExp(word).replace(/'/g, "['’‘]");
-	const punctuations = ['\\.', ',', ':', ';', '\\?', '!'];
+	const punctuations = ['\\.', ',', ':', ';', '\\?', '!', '…'];
 	const punctuationsRegExp = punctuations.map(
 		punctuation => `${punctuation} |${punctuation}$`
 	);
diff --git a/test.js b/test.js
@@ -258,6 +258,66 @@ tester.run('textlint-rule-stop-words', rule, {
 				},
 			],
 		},
+		{
+			// One word, after a comma
+			text: 'You should, hyperlocal Elm',
+			output: 'You should, hyperlocal Elm',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
+		{
+			// One word, after a colon
+			text: 'You should: hyperlocal Elm',
+			output: 'You should: hyperlocal Elm',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
+		{
+			// One word, after a semicolon
+			text: 'You should; hyperlocal Elm',
+			output: 'You should; hyperlocal Elm',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
+		{
+			// One word, before comma
+			text: 'You should hyperlocal,',
+			output: 'You should hyperlocal,',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
+		{
+			// One word, before point
+			text: 'You should hyperlocal.',
+			output: 'You should hyperlocal.',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
+		{
+			// One word, before ellipsis
+			text: 'You should hyperlocal…',
+			output: 'You should hyperlocal…',
+			errors: [
+				{
+					message: 'Avoid using “hyperlocal”',
+				},
+			],
+		},
 		{
 			// One word + fix
 			text: 'You should utilize Elm',
