From fde1452666f43997dacefc07dac997407c823fc0 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 --- 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 364d3a1..33404e6 100644 --- a/test/test_wiki_diff_matcher.py +++ b/test/test_wiki_diff_matcher.py @@ -352,6 +352,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 @@ -369,6 +373,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