fix(rule): correct ignore implementation

textlint-rule · Oct 20, 2018 · 302aaaf · 302aaaf
1 parent f41eff3
commit 302aaaf
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/src/textlint-rule-no-unmatched-pair.js b/src/textlint-rule-no-unmatched-pair.js
@@ -22,15 +22,17 @@ const report = context => {
             sentences.children.filter(node => node.type === SentenceSyntax.Sentence).forEach(sentence => {
                 const source = new SourceCode(sentence.raw);
                 const pairMaker = new PairMaker();
+                const sentenceIndex = sentence.range[0];
                 while (source.canRead) {
-                    pairMaker.mark(source);
+                    // If the character is in ignored range, skip it
+                    const characterIndex = sentenceIndex + source.index;
+                    if (!ignoreNodeManager.isIgnoredIndex(characterIndex)) {
+                        pairMaker.mark(source);
+                    }
                     source.peek();
                 }
                 // Report Error for each existing context keys
                 source.contextLocations.forEach((contextLocation) => {
-                    if (ignoreNodeManager.isIgnoredIndex(node.range[0] + contextLocation.index)) {
-                        return;
-                    }
                     report(node, new RuleError(`Not found pair character for ${contextLocation.pairMark.start}.
                     
 You should close this sentence with ${contextLocation.pairMark.end}.

diff --git a/test/textlint-rule-no-unmatched-pair-test.js b/test/textlint-rule-no-unmatched-pair-test.js
@@ -9,7 +9,11 @@ tester.run("textlint-rule-no-unmatched-pair", rule, {
     valid: [
         "これは(秘密)です。",
         `John said "Hello World!".`,
-        "`(` is ok.", "文字列リテラルには3種類ありますが、まずは`\"`（ダブルクオート）と`'`（シングルクオート）について見ていきます。"
+        "`(` is ok.", "文字列リテラルには3種類ありますが、まずは`\"`（ダブルクオート）と`'`（シングルクオート）について見ていきます。",
+        `a is b.
+        
+\`"\`（ダブルクオート）と\`'\`（シングルクオート）に意味的な違いはありません。
+この書籍では、\`"\`（ダブルクオート）を主に文字列リテラルとして利用します。`
     ],
     invalid: [
         {