Increase coverage

Author: mwouts

.github/workflows/continuous-integration.yml

@@ -71,7 +71,6 @@ jobs:
             numpy-version: '<2.0'
           - python-version: "3.13"
             pandas-version: pre
-            polars: true
           - python-version: "3.13"
             uninstall_non_essential_dependencies: true
           - python-version: "3.10"
@@ -110,16 +109,12 @@ jobs:
         if: matrix.typeguard-version != 'latest'
         run: pip install 'typeguard${{ matrix.typeguard-version }}'
 
-      - name: Install polars
-        if: matrix.polars
-        run: pip install -e .[polars]
-
       - name: Install shiny
         run: pip install "shiny>=1.0" shinywidgets
 
       - name: Uninstall non-essential dependencies
         if: matrix.uninstall_non_essential_dependencies
-        run: pip uninstall jinja2 dash anywidget streamlit shiny shinywidgets -y
+        run: pip uninstall polars jinja2 dash anywidget streamlit shiny shinywidgets -y
 
       - name: Install a Jupyter Kernel
         run: python -m ipykernel install --name itables --user

packages/dt_for_itables/add_host_to_root.py

@@ -61,31 +61,21 @@ def process_rule(match):
     return re.sub(pattern, process_rule, css_content)
 
 
-if __name__ == "__main__":
-
-    def main():
-        parser = argparse.ArgumentParser(
-            description="Add :host to each :root selector in a CSS file."
-        )
-        parser.add_argument("css_file", help="Path to the CSS file to modify")
-        args = parser.parse_args()
+def main(argv):
+    parser = argparse.ArgumentParser(
+        description="Add :host to each :root selector in a CSS file."
+    )
+    parser.add_argument("css_file", help="Path to the CSS file to modify")
+    args = parser.parse_args(argv[1:])
 
-        try:
-            with open(args.css_file, "r") as file:
-                css_content = file.read()
+    with open(args.css_file, "r") as file:
+        css_content = file.read()
 
-            modified_content = add_host_to_root(css_content)
+    modified_content = add_host_to_root(css_content)
 
-            with open(args.css_file, "w") as file:
-                file.write(modified_content)
+    with open(args.css_file, "w") as file:
+        file.write(modified_content)
 
-            print(f"Successfully updated {args.css_file}")
-            return 0
-        except FileNotFoundError:
-            print(f"Error: File {args.css_file} not found.")
-            return 1
-        except Exception as e:
-            print(f"Error processing file: {e}")
-            return 1
 
-    sys.exit(main())
+if __name__ == "__main__":
+    sys.exit(main(sys.argv))

packages/dt_for_itables/test_add_host_to_root.py

@@ -1,5 +1,5 @@
 import pytest
-from add_host_to_root import add_host_to_root
+from add_host_to_root import add_host_to_root, main
 
 
 def test_add_host_to_root_example():
@@ -66,3 +66,15 @@ def test_add_host_to_root_complex_selectors():
 :root[data-theme="light"] .element, :host[data-theme="light"] .element { background-color: white; }
 """
     assert add_host_to_root(original_css) == expected_css
+
+
+def test_main_function(tmp_path):
+    """Test the main function modifies the file as expected."""
+    css_content = ":root { color: red; }"
+    expected_content = ":root, :host { color: red; }"
+    css_file = tmp_path / "test.css"
+    css_file.write_text(css_content)
+
+    main(["add_host_to_root.py", str(css_file)])
+    result = css_file.read_text()
+    assert result == expected_content

tests/test_apps.py

@@ -0,0 +1,49 @@
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+APP_FOLDER = Path(__file__).parent.parent / "apps"
+APP_FILES = list(APP_FOLDER.glob("**/*.py"))
+assert APP_FILES, APP_FOLDER
+
+
+@pytest.fixture(params=APP_FILES, ids=lambda p: str(p.relative_to(APP_FOLDER)))
+def app_file(request):
+    return request.param
+
+
+def test_app_can_be_imported(app_file: Path):
+    app_name = str(app_file.relative_to(APP_FOLDER))
+    framework = app_file.relative_to(APP_FOLDER).parts[0]
+    if framework == "shiny":
+        pytest.importorskip("shiny")
+    if framework == "streamlit":
+        pytest.importorskip("streamlit")
+    if framework == "dash":
+        pytest.importorskip("dash")
+
+    is_shiny_app = framework == "shiny" and app_name.endswith("app.py")
+
+    code = f"""
+import importlib.util
+
+spec = importlib.util.spec_from_file_location("{app_file.stem}", "{app_file}")
+assert spec is not None
+module = importlib.util.module_from_spec(spec)
+assert spec.loader is not None
+
+try:
+    spec.loader.exec_module(module)
+except RuntimeError:
+    # In Shiny apps we expect this error:
+    # RuntimeError: express.ui.page_opts() can only be used inside of a standalone Shiny Express app.
+    if not {is_shiny_app}:
+        raise
+"""
+
+    result = subprocess.run(
+        [sys.executable, "-c", code], capture_output=True, text=True
+    )
+    assert result.returncode == 0, result.stderr

tests/test_documentation_notebooks_run.py

@@ -1,10 +1,10 @@
+import subprocess
+import sys
 from pathlib import Path
 
 import jupytext
 import pytest
 
-import itables.options as opt
-from itables import init_notebook_mode
 from itables.javascript import pd_style
 
 try:
@@ -34,20 +34,16 @@ def test_run_documentation_notebooks(notebook):
         pytest.skip("Polars is not available")
     if "pandas_style" in notebook.stem and pd_style is None:
         pytest.skip("Pandas Style is not available")
-    if "shiny" in notebook.stem:
-        pytest.skip("shinywidgets makes the widget.md notebook fail")
-    if "marimo" in notebook.stem or "widget" in notebook.stem:
+    if (
+        "shiny" in notebook.stem
+        or "marimo" in notebook.stem
+        or "widget" in notebook.stem
+    ):
         pytest.importorskip("anywidget")
 
-    org_options = dir(opt)
-
     nb = jupytext.read(notebook)
     py_notebook = jupytext.writes(nb, "py:percent")
-    exec(py_notebook, {})
-
-    new_options = set(dir(opt)).difference(org_options)
-    for name in new_options:
-        delattr(opt, name)
-
-    # Revert back to the non initialized mode
-    init_notebook_mode(all_interactive=False, connected=True)
+    result = subprocess.run(
+        [sys.executable, "-c", py_notebook], capture_output=True, text=True
+    )
+    assert result.returncode == 0, result.stderr