Skip to content

Commit

Permalink
Merge pull request #244 from RockefellerArchiveCenter/remove-coverage
Browse files Browse the repository at this point in the history
Update test workflow
  • Loading branch information
p-galligan authored Sep 5, 2024
2 parents 7cd7c5a + 32e66e2 commit ebe8b9f
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,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 @@ -61,8 +61,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 ebe8b9f

Please sign in to comment.