Merge branch 'main' into main

Author: mangiucugna

.pre-commit-config.yaml

@@ -8,7 +8,7 @@ repos:
     pass_filenames: false
 - repo: https://github.com/astral-sh/ruff-pre-commit
   # Ruff version.
-  rev: v0.12.12
+  rev: v0.13.0
   hooks:
     # Run the linter.
     - id: ruff-check
@@ -38,7 +38,7 @@ repos:
     pass_filenames: false
     types: [python]
 - repo: https://github.com/semgrep/pre-commit
-  rev: "v1.135.0"
+  rev: "v1.136.0"
   hooks:
     - id: semgrep
       args:

tests/test_performance.py

@@ -1,9 +1,12 @@
 import os.path
 import pathlib
 
+import pytest
+
 from src.json_repair import repair_json
 
 path = pathlib.Path(__file__).parent.resolve()
+CI = os.getenv("CI") is not None
 
 with open(os.path.join(path, "valid.json")) as fd:
     correct_json = fd.read()
@@ -12,6 +15,7 @@
     incorrect_json = fd.read()
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_true_true_correct(benchmark):
     benchmark(repair_json, correct_json, return_objects=True, skip_json_loads=True)
 
@@ -25,6 +29,7 @@ def test_true_true_correct(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_true_true_incorrect(benchmark):
     benchmark(repair_json, incorrect_json, return_objects=True, skip_json_loads=True)
 
@@ -38,6 +43,7 @@ def test_true_true_incorrect(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_true_false_correct(benchmark):
     benchmark(repair_json, correct_json, return_objects=True, skip_json_loads=False)
     # Retrieve the median execution time
@@ -50,6 +56,7 @@ def test_true_false_correct(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_true_false_incorrect(benchmark):
     benchmark(repair_json, incorrect_json, return_objects=True, skip_json_loads=False)
     # Retrieve the median execution time
@@ -62,6 +69,7 @@ def test_true_false_incorrect(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_false_true_correct(benchmark):
     benchmark(repair_json, correct_json, return_objects=False, skip_json_loads=True)
     # Retrieve the median execution time
@@ -74,6 +82,7 @@ def test_false_true_correct(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_false_true_incorrect(benchmark):
     benchmark(repair_json, incorrect_json, return_objects=False, skip_json_loads=True)
     # Retrieve the median execution time
@@ -86,6 +95,7 @@ def test_false_true_incorrect(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_false_false_correct(benchmark):
     benchmark(repair_json, correct_json, return_objects=False, skip_json_loads=False)
     # Retrieve the median execution time
@@ -98,6 +108,7 @@ def test_false_false_correct(benchmark):
     assert mean_time < max_time, f"Benchmark exceeded threshold: {mean_time:.3f}s > {max_time:.3f}s"
 
 
+@pytest.mark.skipif(CI, reason="Performance tests are skipped in CI")
 def test_false_false_incorrect(benchmark):
     benchmark(repair_json, incorrect_json, return_objects=False, skip_json_loads=False)
     # Retrieve the median execution time