Merge pull request #8 from orangain/fix-not-reported-issue

Report error correctly when pattern starts with expected value

Author: azu

src/prh-rule.js

@@ -77,8 +77,8 @@ function reporter(context, options = {}) {
                 // | ----[match------|
                 var matchedText = slicedText.slice(0, changeSet.matches[0].length);
                 var expected = matchedText.replace(changeSet.pattern, changeSet.expected);
-                // Avoid accidental match(ignore case, expected contain actual pattern)
-                if (slicedText.indexOf(expected) === 0) {
+                // Avoid accidental match(ignore case)
+                if (matchedText === expected) {
                     return;
                 }
                 /*

test/fixtures/rule.yaml

@@ -48,10 +48,12 @@ rules:
         to:   日経ソフトウエア
 
   - expected: ベンダー
-    pattern: ベンダ
+    pattern: /ベンダ(?!ー)/
     specs:
       - from: ベンダ
         to: ベンダー
+      - from: ベンダー
+        to: ベンダー
 # 単語境界の区別
   - expected: js
   # pattern: "/\b[JjＪｊ][SsＳｓ]\b/g" # と等価 \b が前後に付与される
@@ -65,3 +67,7 @@ rules:
       # 日本語+単語境界の仕様は自分で調べてね…！
       - from: 今日もJS祭り
         to:   今日もjs祭り
+
+  # 全角括弧前後の空白を除去
+  - expected: ）
+    pattern: /） | ）/

test/prh-rule-test.js

@@ -25,6 +25,15 @@ describe("prh-rule-test", function () {
             });
         });
     });
+    context("when match word and s/） /）/ pattern", function () {
+        it("should report error", function () {
+            return textlint.lintMarkdown("（図1） ").then(result => {
+                assert(result.messages.length > 0);
+                assert(result.messages[0].line === 1);
+                assert(result.messages[0].column === 4);
+            });
+        });
+    });
     context("when match word but s/Web/Web/i pattern", function () {
         // fix ignore (the) case
         it("should not report", function () {