Skip to content

Commit

Permalink
Different strategy
Browse files Browse the repository at this point in the history
  • Loading branch information
betodealmeida committed Oct 25, 2023
1 parent efad596 commit e4727e7
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 8 deletions.
14 changes: 8 additions & 6 deletions .github/workflows/python-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,15 @@ jobs:
- name: Start the Postgres service
run: |
docker-compose -f postgres/docker-compose.yml up --build -d
- name: Check if Postgres is ready
- name: Install PostgreSQL tools
run: |
docker run --network container:postgres_postgres_1 postgres_postgres pg_isready \
-h postgres \
-p 5432 \
-U shillelagh \
--timeout=30
sudo apt-get update
sudo apt-get install -y postgresql-client
- name: Wait for Postgres to become available
run: |
docker ps
sleep 300
#until pg_isready -h localhost -p 12345 -U shillelagh -d shillelagh; do sleep 10; done
- name: Test with pytest
env:
SHILLELAGH_ADAPTER_KWARGS: ${{ secrets.SHILLELAGH_ADAPTER_KWARGS }}
Expand Down
2 changes: 1 addition & 1 deletion postgres/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ services:
- ./entrypoint.sh:/usr/local/bin/custom_entrypoint.sh
- ..:/src
ports:
- "12345:5432"
- "5432:5432"
entrypoint: /usr/local/bin/custom_entrypoint.sh

volumes:
Expand Down
2 changes: 1 addition & 1 deletion tests/adapters/api/gsheets/integration_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -768,7 +768,7 @@ def test_public_sheet_multicorn() -> None:
user="shillelagh",
password="shillelagh123",
host="localhost",
port=12345,
port=5432,
database="shillelagh",
)
cursor = connection.cursor()
Expand Down

0 comments on commit e4727e7

Please sign in to comment.