Skip to content

Commit

Permalink
build(deps): bump the api reqs and test Procfile in CI
Browse files Browse the repository at this point in the history
Minimally run the command in Procfile in CI, with two trivial queries.
Unit testing covers the API calls in a more efficient way, but we also
want to make sure the command in Procfile remains functional.
  • Loading branch information
joanise committed Jun 12, 2024
1 parent c4cac88 commit 69458c9
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
9 changes: 9 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,15 @@ jobs:
coverage combine
coverage xml
- name: Exercise the Heroku Procfile
run: |
cmd=$(grep web: Procfile | cut -d' ' -f2-)
echo "Running $cmd"
$cmd &
curl --retry 20 --retry-delay 1 --retry-all-errors http://127.0.0.1:8000/api/v1/docs | grep SwaggerUIBundle
curl http://127.0.0.1:8000/api/v1/langs | grep Cree
kill %1
- name: Nitpicking
run: |
# coding style: we want black compliance
Expand Down
4 changes: 2 additions & 2 deletions requirements.api.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# These are dependencies required by the production Web API
# gunivorn is only available on *nix, but it's only required for production deployments
gunicorn
gunicorn>=22.0.0
# uvicorn works on all platforms and is required for both dev and prod deployments
uvicorn
uvicorn>=0.30.1
# For deployment on Heroku, we want the latest g2p off GitHub
g2p @ git+https://github.com/roedoejet/g2p.git@main

0 comments on commit 69458c9

Please sign in to comment.