Skip to content

Commit

Permalink
Update build_and_test file to include pytest for test_speedup.py
Browse files Browse the repository at this point in the history
  • Loading branch information
Michael O'Keefe authored and Michael O'Keefe committed Nov 18, 2024
1 parent 9ca2e11 commit abd68a4
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/build_and_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,10 @@ jobs:
python-version: '3.${{ matrix.python-version }}'
- name: install python dependencies
run: python -m pip install --upgrade pip setuptools wheel pytest twine cibuildwheel
- name: install FASTSim python
run: python -m pip install -e .[dev]
- name: run cargo tests
run: cargo test
- name: install FASTSim python
run: pip install -e .[dev]
- name: run pytest
run: pytest -v
- name: build source distribution
Expand All @@ -52,8 +52,8 @@ jobs:
CIBW_SKIP: "*-win32 *-musllinux* *i686 *ppc64le *s390x *aarch64"
CIBW_PLATFORM: ${{ matrix.platform || matrix.os }}
# TODO: why doesn't pytest work with cibuildwheel?
# CIBW_TEST_COMMAND: "pytest -v {project}/python/fastsim/tests"
CIBW_TEST_COMMAND: "python -m unittest discover {project}/python/fastsim/tests"
CIBW_TEST_COMMAND: "pytest -v"
# CIBW_TEST_COMMAND: "python -m unittest discover {project}/python/fastsim/tests"
CIBW_ARCHS_MACOS: "universal2"
# see https://cibuildwheel.readthedocs.io/en/stable/faq/#universal2
CIBW_TEST_SKIP: "*_universal2:arm64"
Expand All @@ -62,12 +62,12 @@ jobs:
CIBW_MANYLINUX_X86_64_IMAGE: "manylinux2014"
CIBW_MANYLINUX_I686_IMAGE: "manylinux2014"
CIBW_BEFORE_BUILD: >
pip install -U setuptools-rust &&
python -m pip install -U setuptools-rust pytest &&
rustup default stable &&
rustup show
CIBW_BEFORE_BUILD_LINUX: >
yum -y install openssl openssl-devel &&
pip install -U setuptools-rust &&
pip install -U setuptools-rust pytest &&
curl https://sh.rustup.rs -sSf | sh -s -- --default-toolchain=nightly --profile=minimal -y &&
rustup show
- name: list dist files
Expand Down

0 comments on commit abd68a4

Please sign in to comment.