Skip to content

Commit 3bc629b

Browse files
Update ci-build.yml
1 parent ea1f3cb commit 3bc629b

1 file changed

Lines changed: 5 additions & 28 deletions

File tree

.github/workflows/ci-build.yml

Lines changed: 5 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -117,16 +117,7 @@ jobs:
117117
databases:
118118
name: Database Unit Tests
119119
runs-on: ubuntu-latest
120-
strategy:
121-
fail-fast: false
122-
matrix:
123-
include:
124-
- database: postgres
125-
sync_url: postgresql://test_user:password@localhost/test
126-
async_url: postgresql+asyncpg://test_user:password@localhost/test
127-
- database: mysql
128-
sync_url: mysql+pymysql://test_user:password@localhost/test
129-
async_url: mysql+aiomysql://test_user:password@localhost/test
120+
# TODO: Add MySQL and other database testing
130121
services:
131122
postgres:
132123
image: postgres:16
@@ -141,20 +132,6 @@ jobs:
141132
--health-interval 10s
142133
--health-timeout 5s
143134
--health-retries 5
144-
mysql:
145-
image: mysql:8.0
146-
env:
147-
MYSQL_USER: test_user
148-
MYSQL_PASSWORD: password
149-
MYSQL_DATABASE: test
150-
MYSQL_ROOT_PASSWORD: root_password
151-
ports:
152-
- 3306:3306
153-
options: >-
154-
--health-cmd "mysqladmin ping -h localhost"
155-
--health-interval 10s
156-
--health-timeout 5s
157-
--health-retries 5
158135
steps:
159136
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
160137
with:
@@ -170,15 +147,15 @@ jobs:
170147
pip install -r requirements/testing.txt
171148
pip install -r requirements/optional.txt
172149
pip install -r requirements/databases.txt
173-
- name: Run sync tests (${{ matrix.database }})
150+
- name: Run sync tests (PostgreSQL)
174151
env:
175-
TEST_DATABASE_URL: ${{ matrix.sync_url }}
152+
TEST_DATABASE_URL: postgresql://test_user:password@localhost/test
176153
run: |
177154
PYTHONPATH=$PWD:$PYTHONPATH pytest -s tests/slack_sdk/oauth/installation_store/test_sqlalchemy.py
178155
PYTHONPATH=$PWD:$PYTHONPATH pytest -s tests/slack_sdk/oauth/state_store/test_sqlalchemy.py
179-
- name: Run async tests (${{ matrix.database }})
156+
- name: Run async tests (PostgreSQL)
180157
env:
181-
ASYNC_TEST_DATABASE_URL: ${{ matrix.async_url }}
158+
ASYNC_TEST_DATABASE_URL: postgresql+asyncpg://test_user:password@localhost/test
182159
run: |
183160
PYTHONPATH=$PWD:$PYTHONPATH pytest -s tests/slack_sdk/oauth/installation_store/test_async_sqlalchemy.py
184161
PYTHONPATH=$PWD:$PYTHONPATH pytest -s tests/slack_sdk/oauth/state_store/test_async_sqlalchemy.py

0 commit comments

Comments
 (0)