fix: sphinx 7

sphinx_remove_toctrees/__init__.py

@@ -10,6 +10,12 @@
 logger = logging.getLogger(__name__)
 
 
+def findall(node):
+    # findall replaces traverse in docutils v0.18
+    # note a difference is that findall is an iterator
+    return getattr(node, "findall", node.traverse)
+
+
 def remove_toctrees(app, env):
     """Remove toctrees from pages a user provides.
 
@@ -37,7 +43,7 @@ def remove_toctrees(app, env):
 
     # Loop through all tocs and remove the ones that match our pattern
     for _, tocs in env.tocs.items():
-        for toctree in tocs.traverse(addnodes.toctree):
+        for toctree in findall(tocs)(addnodes.toctree):
             new_entries = []
             for entry in toctree.attributes.get("entries", []):
                 if entry[1] not in to_remove:

sphinx_remove_toctrees/tests/test_build.py

@@ -3,8 +3,7 @@
 from shutil import copytree
 
 from bs4 import BeautifulSoup
-from sphinx.testing.path import path as sphinx_path
-from sphinx.testing.util import SphinxTestApp
+from sphinx import version_info as sphinx_version_info
 
 pytest_plugins = "sphinx.testing.fixtures"
 
@@ -13,8 +12,19 @@
 
 def test_build_html(make_app, tmp_path):
     """Test building the base html template and config."""
-    copytree(path_test_doc, tmp_path / "test_doc")
-    app = make_app(srcdir=sphinx_path(tmp_path / "test_doc"))
+    src_dir = tmp_path / "test_doc"
+    copytree(path_test_doc, src_dir)    
+
+    # For compatibility with multiple versions of sphinx, convert pathlib.Path to
+    # sphinx.testing.path.path here.
+    if sphinx_version_info >= (7, 2):
+        app_src_dir = src_dir
+    else:
+        from sphinx.testing.path import path
+
+        app_src_dir = path(os.fspath(src_dir))
+
+    app = make_app(srcdir=app_src_dir)
     app.build()
     index = tmp_path / "test_doc" / "_build" / "html" / "index.html"
     assert index.exists()