provide setting for choosing between minified and unminified files

Author: dojutsu-user

MANIFEST.in

@@ -1,4 +1,6 @@
 prune common
 include LICENSE
+include sphinx_search/_static/js/rtd_sphinx_search.js
 include sphinx_search/_static/js/rtd_sphinx_search.min.js
+include sphinx_search/_static/css/rtd_sphinx_search.css
 include sphinx_search/_static/css/rtd_sphinx_search.min.css

sphinx_search/extension.py

@@ -1,26 +1,43 @@
 import os
+
 from sphinx.util.fileutil import copy_asset
 
 
-CUSTOM_ASSETS_FILES = [
-    os.path.join('js', 'rtd_sphinx_search.min.js'),
-    os.path.join('css', 'rtd_sphinx_search.min.css'),
-]
+CUSTOM_ASSETS_FILES = {
+    'MINIFIED': [
+        os.path.join('js', 'rtd_sphinx_search.min.js'),
+        os.path.join('css', 'rtd_sphinx_search.min.css'),
+    ],
+    'UN_MINIFIED': [
+        os.path.join('js', 'rtd_sphinx_search.js'),
+        os.path.join('css', 'rtd_sphinx_search.css'),
+    ]
+}
 
 
 def copy_asset_files(app, exception):
     if exception is None:  # build succeeded
-        for f in CUSTOM_ASSETS_FILES:
-            path = os.path.join(os.path.dirname(__file__), '_static', f)
-            copy_asset(path, os.path.join(app.outdir, '_static', f.split('.')[-1]))
+        files = CUSTOM_ASSETS_FILES['MINIFIED'] + CUSTOM_ASSETS_FILES['UN_MINIFIED']
+        for file in files:
+            path = os.path.join(os.path.dirname(__file__), '_static', file)
+            copy_asset(path, os.path.join(app.outdir, '_static', file.split('.')[-1]))
 
 
-def setup(app):
+def inject_static_files(app):
+    """Inject correct CSS and JS files based on the value of ``RTD_SPHINX_SEARCH_FILE_TYPE``.""" 
+    file_type = app.config.RTD_SPHINX_SEARCH_FILE_TYPE
+    files = CUSTOM_ASSETS_FILES[file_type]
 
-    app.connect('build-finished', copy_asset_files)
-
-    for file in CUSTOM_ASSETS_FILES:
-        if file.endswith('.min.js'):
+    for file in files:
+        if file.endswith('.js'):
             app.add_js_file(file)
-        if file.endswith('.min.css'):
+        elif file.endswith('.css'):
             app.add_css_file(file)
+
+
+def setup(app):
+
+    app.add_config_value('RTD_SPHINX_SEARCH_FILE_TYPE', 'UN_MINIFIED', 'html')
+
+    app.connect('builder-inited', inject_static_files)
+    app.connect('build-finished', copy_asset_files)

tests/test_extension.py

@@ -6,6 +6,7 @@
 import pytest
 
 from tests import TEST_DOCS_SRC
+from sphinx_search.extension import CUSTOM_ASSETS_FILES
 
 
 @pytest.mark.sphinx(srcdir=TEST_DOCS_SRC)
@@ -14,31 +15,62 @@ def test_static_files_exists(app, status, warning):
     app.build()
     path = app.outdir
 
-    js_file = os.path.join(path, '_static', 'js', 'rtd_sphinx_search.min.js')
-    css_file = os.path.join(path, '_static', 'css', 'rtd_sphinx_search.min.css')
+    static_files = CUSTOM_ASSETS_FILES['MINIFIED'] + CUSTOM_ASSETS_FILES['UN_MINIFIED']
 
-    assert (
-        os.path.exists(js_file) is True
-    ), 'js file should be copied to build folder'
+    for file in static_files:
+        file_path = os.path.join(path, '_static', file)
+        assert (
+            os.path.exists(file_path)
+        ), f'{file_path} should be present in the _build folder'
 
-    assert (
-        os.path.exists(css_file) is True
-    ), 'css file should be copied to build folder'
 
+@pytest.mark.sphinx(
+    srcdir=TEST_DOCS_SRC,
+    confoverrides={
+        'RTD_SPHINX_SEARCH_FILE_TYPE': 'MINIFIED'
+    }
+)
+def test_minified_static_files_injected_in_html(selenium, app, status, warning):
+    """Test if the static files are correctly injected in the html."""
+    app.build()
+    path = app.outdir / 'index.html'
 
-@pytest.mark.sphinx(srcdir=TEST_DOCS_SRC)
-def test_static_files_injected_in_html(selenium, app, status, warning):
+    selenium.get(f'file://{path}')
+    page_source = selenium.page_source
+
+    assert app.config.RTD_SPHINX_SEARCH_FILE_TYPE == 'MINIFIED'
+
+    file_type = app.config.RTD_SPHINX_SEARCH_FILE_TYPE
+    files = CUSTOM_ASSETS_FILES[file_type]
+
+    for file in files:
+        file_name = file.split('/')[-1]
+        assert (
+            page_source.count(file_name) == 1
+        ), f'{file} should be present in the page source'
+
+
+@pytest.mark.sphinx(
+    srcdir=TEST_DOCS_SRC,
+    confoverrides={
+        'RTD_SPHINX_SEARCH_FILE_TYPE': 'UN_MINIFIED'
+    }
+)
+def test_un_minified_static_files_injected_in_html(selenium, app, status, warning):
     """Test if the static files are correctly injected in the html."""
     app.build()
     path = app.outdir / 'index.html'
 
     selenium.get(f'file://{path}')
     page_source = selenium.page_source
 
-    assert (
-        page_source.count('rtd_sphinx_search.min.js') == 1
-    ), 'js file should be injected only once'
+    assert app.config.RTD_SPHINX_SEARCH_FILE_TYPE == 'UN_MINIFIED'
+
+    file_type = app.config.RTD_SPHINX_SEARCH_FILE_TYPE
+    files = CUSTOM_ASSETS_FILES[file_type]
 
-    assert (
-        page_source.count('rtd_sphinx_search.min.css') == 1
-    ), 'css file should be injected only once'
+    for file in files:
+        file_name = file.split('/')[-1]
+        assert (
+            page_source.count(file_name) == 1
+        ), f'{file_name} should be present in the page source'