From caf604e43da63da98e25e642ba55558706baefe1 Mon Sep 17 00:00:00 2001 From: Benjamin Mako Hill Date: Sat, 8 Aug 2026 14:52:59 -0700 Subject: [PATCH] document the requirements of the two skipped diff matcher tests test_diff_consistency and test_benchmark_diff both read an uncompressed test/dumps/ikwiki.xml that is not in the repository, so enabling them requires decompressing the .bz2 first; test_diff_consistency also writes debug files to the current directory. Say so where the skip markers are. Co-Authored-By: Claude Fable 5 (cherry picked from commit fde1452666f43997dacefc07dac997407c823fc0) --- test/test_wiki_diff_matcher.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/test/test_wiki_diff_matcher.py b/test/test_wiki_diff_matcher.py index bf06212..f5381d0 100644 --- a/test/test_wiki_diff_matcher.py +++ b/test/test_wiki_diff_matcher.py @@ -347,6 +347,10 @@ def test_actually_equal(): assert_equal_enough(a, rev1) # slow test. comment out the following line to enable it. +# Requires an uncompressed test/dumps/ikwiki.xml, which is not in the +# repository; decompress the .bz2 in test/dumps first. Also writes debug +# output to test_unicode_highlight_from/_to in the current directory on +# every iteration. @pytest.mark.skip def test_diff_consistency(): from mwxml import Dump @@ -364,6 +368,9 @@ def test_diff_consistency(): assert_equal_enough(b, rev) last_rev = rev +# benchmark, not a pass/fail test; run it deliberately when tuning diff +# performance. Requires an uncompressed test/dumps/ikwiki.xml, which is +# not in the repository; decompress the .bz2 in test/dumps first. @pytest.mark.skip def test_benchmark_diff(benchmark): from mwxml import Dump