fix(rule): fix reported index again

azu · azu · commit 71b98b28a4ef · 2018-10-21T00:23:52.000+09:00
diff --git a/src/textlint-rule-no-unmatched-pair.js b/src/textlint-rule-no-unmatched-pair.js
@@ -38,7 +38,7 @@ const report = context => {
                     
 You should close this sentence with ${contextLocation.pairMark.end}.
 This pair mark is called ${contextLocation.pairMark.key}.`, {
-                        index: sentenceIndex + contextLocation.index
+                        index: (sentenceIndex - node.range[0]) + contextLocation.index
                     }));
                 });
             });
diff --git a/test/textlint-rule-no-unmatched-pair-test.js b/test/textlint-rule-no-unmatched-pair-test.js
@@ -19,7 +19,9 @@ tester.run("textlint-rule-no-unmatched-pair", rule, {
 
 また操作と表示が1対1で更新される場合、1つの操作に対して複数の箇所の表示が更新されることもあります。
 今回のTodoアプリでもTodoリスト（\`#js-todo-list\`）とTodoアイテム数（\`#js-todo-count\`）の2箇所を更新する必要があることからも分かります。
-`
+`, `\`Object.assign\`メソッドは、\`target\`オブジェクトに対して、1つ以上の\`sources\`オブジェクトを指定します。
+\`sources\`オブジェクト自身がもつ列挙可能なプロパティを第一引数の\`target\`オブジェクトに対してコピーします。
+\`Object.assign\`メソッドの返り値は、\`target\`オブジェクトになります。`
     ],
     invalid: [
         {
@@ -58,9 +60,22 @@ This pair mark is called double quote.`
             text: `このように\`count\`変数が自動解放されずに保持できているのは「（\`increment\`）関数が外側のスコープにある（\`count\`）変数への参照を保持できる」ためです。このような性質のことをクロージャー(関数閉包）と呼びます。クロージャーは静的スコープと変数は参照され続けていればデータは保持されるという2つの性質によって成り立っています。`,
             errors: [
                 {
+                    line: 1,
+                    column: 105,
                     index: 104
                 }
             ]
+        },{
+            text: `DUMMY DUUMY.
+            
+            
+このように\`count\`変数が自動解放されずに保持できているのは「（\`increment\`）関数が外側のスコープにある（\`count\`）変数への参照を保持できる」ためです。このような性質のことをクロージャー(関数閉包）と呼びます。クロージャーは静的スコープと変数は参照され続けていればデータは保持されるという2つの性質によって成り立っています。`,
+            errors: [
+                {
+                    line: 4,
+                    column: 105
+                }
+            ]
         }
     ]
 });

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,9 @@ tester.run("textlint-rule-no-unmatched-pair", rule, {`
`19`	`19`
`20`	`20`	`また操作と表示が1対1で更新される場合、1つの操作に対して複数の箇所の表示が更新されることもあります。`
`21`	`21`	今回のTodoアプリでもTodoリスト（\`#js-todo-list\`）とTodoアイテム数（\`#js-todo-count\`）の2箇所を更新する必要があることからも分かります。
`22`		-`
	`22`	+`, `\`Object.assign\`メソッドは、\`target\`オブジェクトに対して、1つ以上の\`sources\`オブジェクトを指定します。
	`23`	+\`sources\`オブジェクト自身がもつ列挙可能なプロパティを第一引数の\`target\`オブジェクトに対してコピーします。
	`24`	+\`Object.assign\`メソッドの返り値は、\`target\`オブジェクトになります。`
`23`	`25`	`],`
`24`	`26`	`invalid: [`
`25`	`27`	`{`
@@ -58,9 +60,22 @@ This pair mark is called double quote.`
`58`	`60`	text: `このように\`count\`変数が自動解放されずに保持できているのは「（\`increment\`）関数が外側のスコープにある（\`count\`）変数への参照を保持できる」ためです。このような性質のことをクロージャー(関数閉包）と呼びます。クロージャーは静的スコープと変数は参照され続けていればデータは保持されるという2つの性質によって成り立っています。`,
`59`	`61`	`errors: [`
`60`	`62`	`{`
	`63`	`+ line: 1,`
	`64`	`+ column: 105,`
`61`	`65`	`index: 104`
`62`	`66`	`}`
`63`	`67`	`]`
	`68`	`+ },{`
	`69`	+ text: `DUMMY DUUMY.
	`70`	`+`
	`71`	`+`
	`72`	+このように\`count\`変数が自動解放されずに保持できているのは「（\`increment\`）関数が外側のスコープにある（\`count\`）変数への参照を保持できる」ためです。このような性質のことをクロージャー(関数閉包）と呼びます。クロージャーは静的スコープと変数は参照され続けていればデータは保持されるという2つの性質によって成り立っています。`,
	`73`	`+ errors: [`
	`74`	`+ {`
	`75`	`+ line: 4,`
	`76`	`+ column: 105`
	`77`	`+ }`
	`78`	`+ ]`
`64`	`79`	`}`
`65`	`80`	`]`
`66`	`81`	`});`