FIX: use warning instead of errors for version json tests

src/pydata_sphinx_theme/__init__.py

@@ -21,6 +21,7 @@
 from pygments.formatters import HtmlFormatter
 from pygments.styles import get_all_styles
 import requests
+from requests.exceptions import ConnectionError, HTTPError, RetryError
 
 from .bootstrap_html_translator import BootstrapHTML5Translator
 
@@ -92,22 +93,34 @@ def update_config(app, env):
 
         # try to read the json file. If it's a url we use request,
         # else we simply read the local file from the source directory
-        # it will raise an error if the file does not exist
+        # display a log warning if the file cannot be reached
+        content = None
         if urlparse(json_url).scheme in ["http", "https"]:
-            content = requests.get(json_url).text
+            try:
+                request = requests.get(json_url)
+                request.raise_for_status()
+                content = request.text
+            except (ConnectionError, HTTPError, RetryError):
+                pass
         else:
-            content = Path(env.srcdir, json_url).read_text()
-
-        # check that the json file is not illformed
-        # it will throw an error if there is a an issue
-        switcher_content = json.loads(content)
-        missing_url = any(["url" not in e for e in switcher_content])
-        missing_version = any(["version" not in e for e in switcher_content])
-        if missing_url or missing_version:
-            raise AttributeError(
-                f'The version switcher "{json_url}" file is malformed'
-                ' at least one of the items is missing the "url" or "version" key'
-            )
+            try:
+                content = Path(env.srcdir, json_url).read_text()
+            except FileNotFoundError:
+                pass
+
+        if content is None:
+            logger.warning(f'The version switcher "{json_url}" file cannot be read.')
+        else:
+            # check that the json file is not illformed,
+            # throw a warning if the file is ill formed and an error if it's not json
+            switcher_content = json.loads(content)
+            missing_url = any(["url" not in e for e in switcher_content])
+            missing_version = any(["version" not in e for e in switcher_content])
+            if missing_url or missing_version:
+                logger.warning(
+                    f'The version switcher "{json_url}" file is malformed'
+                    ' at least one of the items is missing the "url" or "version" key'
+                )
 
     # Add an analytics ID to the site if provided
     analytics = theme_options.get("analytics", {})

tests/sites/base/missing_url.json

@@ -0,0 +1,14 @@
+[
+  {
+    "name": "v0.7.1 (stable)",
+    "version": "0.7.1",
+    "url": "https://pydata-sphinx-theme.readthedocs.io/en/v0.7.1/"
+  },
+  {
+    "version": "0.7.0",
+    "url": "https://pydata-sphinx-theme.readthedocs.io/en/v0.7.0/"
+  },
+  {
+    "version": "0.6.3"
+  }
+]

tests/test_build.py

@@ -1,4 +1,5 @@
 import os
+import re
 from pathlib import Path
 from shutil import copytree
 
@@ -11,6 +12,12 @@
 path_tests = Path(__file__).parent
 
 
+def escape_ansi(string):
+    """helper function to remove ansi coloring from sphinx warnings"""
+    ansi_escape = re.compile(r"(\x9B|\x1B\[)[0-?]*[ -\/]*[@-~]")
+    return ansi_escape.sub("", string)
+
+
 class SphinxBuild:
     def __init__(self, app: SphinxTestApp, src: Path):
         self.app = app
@@ -628,7 +635,12 @@ def test_show_nav_level(sphinx_build_factory):
         assert "checked" in checkbox.attrs
 
 
-def test_version_switcher(sphinx_build_factory, file_regression):
+switcher_files = ["switcher.json", "http://a.b/switcher.json", "missing_url.json"]
+"the switcher files tested in test_version_switcher, not all of them exist"
+
+
+@pytest.mark.parametrize("url", switcher_files)
+def test_version_switcher(sphinx_build_factory, file_regression, url):
     """Regression test the version switcher dropdown HTML.
 
     Note that a lot of the switcher HTML gets populated by JavaScript,
@@ -641,20 +653,28 @@ def test_version_switcher(sphinx_build_factory, file_regression):
         "html_theme_options": {
             "navbar_end": ["version-switcher"],
             "switcher": {
-                "json_url": "switcher.json",
+                "json_url": url,
                 "version_match": "0.7.1",
             },
         }
     }
-    sphinx_build = sphinx_build_factory("base", confoverrides=confoverrides).build()
-    switcher = sphinx_build.html_tree("index.html").select(
-        ".version-switcher__container"
-    )[
-        0
-    ]  # noqa
-    file_regression.check(
-        switcher.prettify(), basename="navbar_switcher", extension=".html"
-    )
+    factory = sphinx_build_factory("base", confoverrides=confoverrides)
+    sphinx_build = factory.build(no_warning=False)
+
+    if url == "switcher.json":  # this should work
+        index = sphinx_build.html_tree("index.html")
+        switcher = index.select(".version-switcher__container")[0]
+        file_regression.check(
+            switcher.prettify(), basename="navbar_switcher", extension=".html"
+        )
+
+    elif url == "http://a.b/switcher.json":  # this file doesn't exist"
+        not_read = 'WARNING: The version switcher "http://a.b/switcher.json" file cannot be read.'  # noqa
+        assert escape_ansi(sphinx_build.warnings).strip() == not_read
+
+    elif url == "missing_url.json":  # this file is missing the url key for one version
+        missing_url = 'WARNING: The version switcher "missing_url.json" file is malformed at least one of the items is missing the "url" or "version" key'  # noqa
+        assert escape_ansi(sphinx_build.warnings).strip() == missing_url
 
 
 def test_theme_switcher(sphinx_build_factory, file_regression):