Add option -F (or --error-on-fuzzy) to report fuzzy strings

Author: Ömer Faruk Korkmaz

.gitignore

@@ -10,3 +10,11 @@ build/
 dist/
 msgcheck.egg-info/
 nose-*.egg/
+.ruff_cache/
+.pytest_cache/
+.ty_cache/
+.coverage/
+.tox
+.venv/
+venv
+__pycache__/

src/msgcheck/msgcheck.py

@@ -82,6 +82,12 @@ def msgcheck_parser() -> argparse.ArgumentParser:
         action="store_true",
         help="check fuzzy strings",
     )
+    parser.add_argument(
+        "-F",
+        "--error-on-fuzzy",
+        action="store_true",
+        help="raise an error if fuzzy strings are found",
+    )
     parser.add_argument(
         "-n",
         "--check-noqa",
@@ -197,6 +203,7 @@ def msgcheck_check_files(args: argparse.Namespace) -> list[PoFileReport]:
         "no_punct",
         "no_whitespace",
         "no_whitespace_eol",
+        "error_on_fuzzy",
     ):
         if args.__dict__[option]:
             po_check.set_check(option.lstrip("no_"), not option.startswith("no_"))

src/msgcheck/po.py

@@ -515,6 +515,7 @@ def __init__(self) -> None:
             "whitespace": True,
             "whitespace_eol": True,
             "extract": False,
+            "error_on_fuzzy": False,
         }
         # spelling options
         self.spelling: str | None = None
@@ -618,9 +619,24 @@ def check_pofile(self, po_file: PoFile) -> list[PoReport]:
         checker = self._get_language_checker(po_file, reports)
 
         # check all messages
+        check_error_on_fuzzy = self.checks["error_on_fuzzy"]
         check_fuzzy = self.checks["fuzzy"]
         check_noqa = self.checks["check_noqa"]
         for msg in po_file.msgs:
+            if check_error_on_fuzzy and msg.fuzzy:
+                mid = msg.messages[0][0] if msg.messages else ""
+                mstr = msg.messages[0][1] if msg.messages else ""
+                reports.append(
+                    PoReport(
+                        "fuzzy string",
+                        "fuzzy",
+                        po_file.filename,
+                        msg.line,
+                        mid,
+                        mstr,
+                        fuzzy=True,
+                    ),
+                )
             if msg.noqa and not check_noqa:
                 continue
             if msg.fuzzy and not check_fuzzy:

tests/test_msgcheck.py

@@ -151,10 +151,50 @@ def test_checks_fuzzy() -> None:
     # be sure we have one file in result
     assert len(result) == 1
 
-    # the file has 11 errors (with the fuzzy string)
+    # the file has 10 errors (with the fuzzy string)
     assert len(result[0]) == 10
 
 
+def test_error_on_fuzzy() -> None:
+    """Test error_on_fuzzy option that raises an error when fuzzy strings are found."""
+    po_check = PoCheck()
+    # disable all tests and enable only "error_on_fuzzy"
+    po_check.checks = dict.fromkeys(po_check.checks, False)
+    po_check.set_check("error_on_fuzzy")
+    result = po_check.check_files([local_path("fr_errors.po")])
+
+    # be sure we have one file in result
+    assert len(result) == 1
+
+    # the file has 1 fuzzy string
+    assert len(result[0]) == 1
+
+    # check the error report
+    report = result[0][0]
+    assert report.idmsg == "fuzzy"
+    assert report.message == "fuzzy string"
+    assert report.fuzzy is True
+    assert "fr_errors.po" in report.filename
+    assert report.line == 58  # Line where the fuzzy string starts
+    assert report.mid == "Tested 3"
+    assert report.mstr == "Testé 3."
+
+
+def test_error_on_fuzzy_no_fuzzy_strings() -> None:
+    """Test error_on_fuzzy option when there are no fuzzy strings."""
+    po_check = PoCheck()
+    # disable all tests and enable only "error_on_fuzzy"
+    po_check.checks = dict.fromkeys(po_check.checks, False)
+    po_check.set_check("error_on_fuzzy")
+    result = po_check.check_files([local_path("fr.po")])
+
+    # be sure we have one file in result
+    assert len(result) == 1
+
+    # the file has 10 errors (no fuzzy strings)
+    assert len(result[0]) == 0
+
+
 def test_checks_noqa() -> None:
     """Test checks on a gettext file including `noqa`-commented lines."""
     po_check = PoCheck()