fix(4.3.1): revert #76 (#80)

textlint-ja · Sep 29, 2020 · 602e9a0 · 602e9a0
1 parent bc2630b
commit 602e9a0
Show file tree

Hide file tree

Showing 6 changed files with 46 additions and 45 deletions.
diff --git a/src/4.3.1.js b/src/4.3.1.js
@@ -31,9 +31,10 @@ function reporter(context) {
             // 半角のかっこ()は使用しないで全角のかっこを使用する
             const text = getSource(node);
             const matchRegExps = [
-                rx`([\(\)])(?:.*${japaneseRegExp}+.*)([\(\)])`,
-                rx`([\(\)])(?:.*${japaneseRegExp})`,
-                rx`(?:${japaneseRegExp}.*)([\(\)])`
+                // FIXME: https://github.com/textlint-ja/textlint-rule-preset-JTF-style/issues/79
+                // rx`([\(\)])(?:${japaneseRegExp}+)([\(\)])`,
+                // rx`([\(\)])(?:${japaneseRegExp})`,
+                rx`(?:${japaneseRegExp})([\(\)])`
             ];
             matchRegExps.forEach(matchRegExp => {
                 matchCaptureGroupAll(text, matchRegExp).forEach(match => {

diff --git a/test/4.3.1-test.js b/test/4.3.1-test.js
@@ -4,7 +4,14 @@ import TextLintTester from "textlint-tester";
 import rule from "../src/4.3.1.js";
 var tester = new TextLintTester();
 tester.run("4.3.1.丸かっこ()", rule, {
-    valid: ["クォーク（物質の素粒子）", "（物質の素粒子）", "（npm 2.x以上をインストールしている必要があります）"],
+    valid: [
+        "クォーク（物質の素粒子）",
+        "（物質の素粒子）",
+        "（npm 2.x以上をインストールしている必要があります）",
+        // 英語のみの半角括弧は許可
+        // https://github.com/textlint-ja/textlint-rule-preset-JTF-style/issues/79
+        "これは (English text in half-width parens) です。"
+    ],
     invalid: [
         {
             // 半角かっこ
@@ -21,36 +28,37 @@ tester.run("4.3.1.丸かっこ()", rule, {
                 }
             ]
         },
-        {
-            // 半角かっこ
-            text: "(物質の素粒子)",
-            output: "（物質の素粒子）",
-            errors: [
-                {
-                    message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
-                    column: 1
-                },
-                {
-                    message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
-                    column: 8
-                }
-            ]
-        },
-        {
-            // 半角かっこ
-            text: "(npm 2.x以上をインストールしている必要があります)",
-            output: "（npm 2.x以上をインストールしている必要があります）",
-            errors: [
-                {
-                    message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
-                    column: 1
-                },
-                {
-                    message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
-                    column: 29
-                }
-            ]
-        },
+        // // FIXME: https://github.com/textlint-ja/textlint-rule-preset-JTF-style/issues/79
+        // {
+        //     // 半角かっこ
+        //     text: "(物質の素粒子)",
+        //     output: "（物質の素粒子）",
+        //     errors: [
+        //         {
+        //             message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
+        //             column: 1
+        //         },
+        //         {
+        //             message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
+        //             column: 8
+        //         }
+        //     ]
+        // },
+        // {
+        //     // 半角かっこ
+        //     text: "(npm 2.x以上をインストールしている必要があります)",
+        //     output: "（npm 2.x以上をインストールしている必要があります）",
+        //     errors: [
+        //         {
+        //             message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
+        //             column: 1
+        //         },
+        //         {
+        //             message: "半角のかっこ()が使用されています。全角のかっこ（）を使用してください。",
+        //             column: 29
+        //         }
+        //     ]
+        // },
         {
             // 半角かっこ
             text: "例)test",

diff --git a/test/fixer-test.js b/test/fixer-test.js
@@ -18,7 +18,7 @@ describe("fixer-test", function() {
             for (var i = 0; i < inputs.length; i++) {
                 const input = inputs[i];
                 const output = outputs[i];
-                assert.equal(input, output);
+                assert.strictEqual(input, output);
             }
         });
     });

diff --git a/test/fixtures/input.md b/test/fixtures/input.md
@@ -85,10 +85,6 @@ A氏は「5月に新製品を発売します。」と述べました。
 
 クォーク(物質の素粒子)
 
-(物質の素粒子)
-
-(npm 2.x以上をインストールしている必要があります)
-
 例)test
 
 半角[かっこ

diff --git a/test/fixtures/output.md b/test/fixtures/output.md
@@ -85,10 +85,6 @@ A氏は「5月に新製品を発売します」と述べました。
 
 クォーク（物質の素粒子）
 
-（物質の素粒子）
-
-（npm 2.x以上をインストールしている必要があります）
-
 例）test
 
 半角［かっこ

diff --git a/tool/create-fixtures.js b/tool/create-fixtures.js
@@ -11,8 +11,8 @@ const RESULT = {
 function processFile(filePath) {
     const contents = fs.readFileSync(filePath, "utf-8");
     const lines = contents.split(/\n/);
-    const inputRegExp = /text:\s*?"(.*?)"/;
-    const outputRegExp = /output:\s*?"(.*?)"/;
+    const inputRegExp = /^\s+text:\s*?"(.*?)"/;
+    const outputRegExp = /^\s+output:\s*?"(.*?)"/;
     lines.forEach(function(line, index) {
         const nextLine = lines[index + 1];
         if (inputRegExp.test(line) && outputRegExp.test(nextLine)) {