Merge pull request #836 from tylerjereddy/treddy_lxml_skip

MAINT, TST: lxml optional skip

Author: tylerjereddy

.github/workflows/main_ci.yml

@@ -65,6 +65,12 @@ jobs:
         name: Install darshan_logs package
         run: |
           python -m pip install git+https://github.com/darshan-hpc/darshan-logs.git@main
+      - if: ${{matrix.python-version == 3.8}}
+        name: Use minimal deps
+        run: |
+          # uninstall deps that are not absolute requirements
+          # to make sure that i.e., tests skip appropriately
+          python -m pip uninstall -y lxml
       - name: Test with pytest
         run: |
           export LD_LIBRARY_PATH=$PWD/darshan_install/lib

darshan-util/pydarshan/darshan/tests/test_summary.py

@@ -14,6 +14,12 @@
 from darshan.log_utils import get_log_path, _provide_logs_repo_filepaths
 from darshan.experimental.plots.data_access_by_filesystem import plot_with_report
 
+try:
+    import lxml
+    has_lxml = True
+except ImportError:
+    has_lxml = False
+
 
 @pytest.mark.parametrize(
     "argv", [
@@ -337,6 +343,9 @@ def test_header_and_footer(self, log_name, expected_header):
     def test_metadata_table(self, log_path, expected_df):
         # regression test for `summary.ReportData.get_metadata_table()`
 
+        if not has_lxml:
+            pytest.skip("Test requires lxml")
+
         log_path = get_log_path(log_path)
         # generate the report data
         R = summary.ReportData(log_path=log_path)
@@ -446,6 +455,9 @@ def test_metadata_table(self, log_path, expected_df):
     def test_module_table(self, log_path, expected_df, expected_partial_flags):
         # regression test for `summary.ReportData.get_module_table()`
 
+        if not has_lxml:
+            pytest.skip("Test requires lxml")
+
         log_path = get_log_path(log_path)
         # collect the report data
         R = summary.ReportData(log_path=log_path)

darshan-util/pydarshan/mypy.ini

@@ -37,6 +37,9 @@ ignore_missing_imports = True
 [mypy-mako.*]
 ignore_missing_imports = True
 
+[mypy-lxml]
+ignore_missing_imports = True
+
 # pydarshan modules that lack types
 # or currently have errors