Skip to content

Commit 132faa3

Browse files
committed
fixing tests once more
1 parent a103446 commit 132faa3

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed

.github/workflows/tests.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ jobs:
88
strategy:
99
matrix:
1010
python-version: ['3.9', '3.10', '3.11']
11-
install-type: [core]
11+
install-type: [core, all]
1212
env:
1313
CLIENT_ID: ${{ secrets.CLIENT_ID }}
1414
CLIENT_SECRET: ${{ secrets.CLIENT_SECRET }}
@@ -23,18 +23,26 @@ jobs:
2323
- name: Install dependencies
2424
run: |
2525
python -m pip install --upgrade pip
26+
# Install base package
2627
if [ "${{ matrix.install-type }}" = "core" ]; then
2728
pip install -e .
2829
else
2930
pip install -e .[all]
3031
fi
32+
# Install test dependencies
3133
pip install -r requirements-dev.txt
3234
pip install -r test-requirements.txt
35+
# Install optional dependencies for testing
36+
pip install rdkit pandas numpy h5py
3337
3438
- name: Run tests with coverage
3539
run: |
3640
if [ "${{ matrix.install-type }}" = "core" ]; then
37-
pytest --cov=foundry --cov-report=xml --cov-report=term-missing --ignore=tests/test_molecular.py --ignore=tests/test_ml_frameworks.py
41+
# Skip tests that require optional dependencies
42+
pytest --cov=foundry --cov-report=xml --cov-report=term-missing \
43+
--ignore=tests/test_molecular.py \
44+
--ignore=tests/test_ml_frameworks.py \
45+
--ignore=tests/test_loaders_specialized.py
3846
else
3947
pytest --cov=foundry --cov-report=xml --cov-report=term-missing
4048
fi

test-requirements.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,7 @@ pytest-mock
44
flake8
55
jsonschema
66
mdf_toolbox
7+
rdkit
8+
pandas
9+
numpy
10+
h5py

0 commit comments

Comments
 (0)