Skip to content

Commit

Permalink
github: remove test-requirements.txt
Browse files Browse the repository at this point in the history
  • Loading branch information
aconchillo committed Jan 20, 2025
1 parent e3d53d3 commit db8e9f7
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 35 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,9 @@ jobs:
- name: Cache virtual environment
uses: actions/cache@v3
with:
# We are hashing dev-requirements.txt and test-requirements.txt which
# contain all dependencies needed to run the tests.
key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('dev-requirements.txt') }}-${{ hashFiles('test-requirements.txt') }}
# We are hashing dev-requirements.txt and pyproject.toml which contain
# all dependencies needed to run the tests.
key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('dev-requirements.txt') }}-${{ hashFiles('pyproject.toml') }}
path: .venv
- name: Install system packages
id: install_system_packages
Expand All @@ -45,7 +45,8 @@ jobs:
run: |
source .venv/bin/activate
python -m pip install --upgrade pip
pip install -r dev-requirements.txt -r test-requirements.txt
pip install -r dev-requirements.txt
pip install ".[all]"
- name: Test with pytest
run: |
source .venv/bin/activate
Expand Down
8 changes: 7 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,13 @@ pip install "path_to_this_repo[option,...]"

### Running tests

From the root directory, run:
Make sure you have all the dependencies installed:

```shell
pip install --editable ".[all]"
```

Then, from the root directory, run:

```shell
pytest --doctest-modules --ignore-glob="*to_be_updated*" --ignore-glob=*pipeline_source* src tests
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ dependencies = [
"loguru~=0.7.3",
"Markdown~=3.7",
"numpy~=1.26.4",
"Pillow~=11.1.0",
"Pillow~=10.4.0",
"protobuf~=5.29.3",
"pydantic~=2.10.5",
"pyloudnorm~=0.1.1",
Expand Down
29 changes: 0 additions & 29 deletions test-requirements.txt

This file was deleted.

0 comments on commit db8e9f7

Please sign in to comment.