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,43 @@
+import importlib.util
+from pathlib import Path
+
+import pytest
+
+from itables.widget import ITable
+
+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))
+    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 app_name.startswith("shiny") or not app_name.endswith("app.py"):
+            raise
+    except ModuleNotFoundError as e:
+        if "shiny" in str(e) and app_name.startswith("shiny"):
+            pytest.skip("shiny is not installed")
+        if "dash" in str(e) and app_name.startswith("dash"):
+            pytest.skip("dash is not installed")
+        raise
+
+    # shinywidgets sets a call back that prevents other
+    # tests from running correctly
+    if ITable._widget_construction_callback is not None:
+        ITable._widget_construction_callback = None
+
+    assert module is not None