Skip to content

Commit

Permalink
Merge pull request #328 from RockefellerArchiveCenter/remove-coverage
Browse files Browse the repository at this point in the history
Update tests workflow
  • Loading branch information
p-galligan authored Sep 5, 2024
2 parents 969c308 + 1392c70 commit 088d4d1
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
run: docker compose up -d

- name: Wait for services to be ready
run: ./wait-for-it.sh $CONTAINER:$APPLICATION_PORT -- docker compose exec -T $CONTAINER pip install coverage
run: ./wait-for-it.sh $CONTAINER:$APPLICATION_PORT -- echo "$CONTAINER is ready"

- name: Install pre-commit
run: |
Expand All @@ -52,8 +52,5 @@ jobs:
- name: Run pre-commit checks
run: pre-commit run --all-files --show-diff-on-failure

- name: Run tests with coverage
run: docker compose exec -T $CONTAINER coverage run manage.py test

- name: Generate coverage report
run: docker compose exec -T $CONTAINER coverage report --omit=*/migrations/* -m
- name: Run tests
run: docker compose exec -T $CONTAINER python manage.py test

0 comments on commit 088d4d1

Please sign in to comment.