Hide an explicitly unset logo name in the sidebar

.github/workflows/test.yml

@@ -35,6 +35,9 @@ jobs:
         python -m pip install "sphinx~=${{ matrix.sphinx-version }}.0"
         python -m pip install .
 
+    - name: Test theme templates
+      run: python -m unittest discover -s tests
+
     - name: Run Sphinx
       run: >
         sphinx-build

.readthedocs.yml

@@ -1,5 +1,8 @@
 version: 2
 
+sphinx:
+  configuration: docs/conf.py
+
 build:
   os: ubuntu-22.04
   tools:

alabaster/about.html

@@ -4,7 +4,7 @@
     <img class="logo" src="{{ pathto('_static/' ~ theme_logo, 1) }}" alt="Logo" />
     {% if theme_logo_name|lower == 'true' %}
     <h1 class="logo logo-name">{{ project }}</h1>
-    {% elif theme_logo_name|lower != 'false' %}
+    {% elif theme_logo_name is not none and theme_logo_name|lower != 'false' %}
     <h1 class="logo logo-name">{{ theme_logo_name }}</h1>
     {% endif %}
   </a>

tests/test_logo_name.py

@@ -0,0 +1,36 @@
+import unittest
+from pathlib import Path
+
+from jinja2 import Environment, FileSystemLoader
+
+
+class LogoNameTests(unittest.TestCase):
+    def render(self, name):
+        env = Environment(
+            loader=FileSystemLoader(Path(__file__).parents[1] / "alabaster")
+        )
+        return env.get_template("about.html").render(
+            theme_logo="logo.png",
+            theme_logo_name=name,
+            project="Project title",
+            master_doc="index",
+            pathto=lambda path, *args: path,
+        )
+
+    def test_none_does_not_render_as_a_name(self):
+        self.assertNotIn('class="logo logo-name"', self.render(None))
+
+    def test_boolean_options(self):
+        for value in (True, "true", "True"):
+            with self.subTest(value=value):
+                self.assertIn(
+                    'class="logo logo-name">Project title</h1>', self.render(value)
+                )
+        for value in (False, "false", "False"):
+            with self.subTest(value=value):
+                self.assertNotIn('class="logo logo-name"', self.render(value))
+
+    def test_custom_name(self):
+        self.assertIn(
+            'class="logo logo-name">Custom name</h1>', self.render("Custom name")
+        )