add ci test for different databases

Author: WilliamBergamin

.github/workflows/ci-build.yml

@@ -114,13 +114,72 @@ jobs:
           token: ${{ secrets.CODECOV_TOKEN }}
           verbose: true
 
+  databases:
+    name: Database Unit Tests
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - database: postgres
+            sync_url: postgresql://test_user:password@localhost/test
+            async_url: postgresql+asyncpg://test_user:password@localhost/test
+          - database: mysql
+            sync_url: mysql+pymysql://test_user:password@localhost/test
+            async_url: mysql+aiomysql://test_user:password@localhost/test
+    services:
+      postgres:
+        image: postgres:16
+        env:
+          POSTGRES_USER: test_user
+          POSTGRES_PASSWORD: password
+          POSTGRES_DB: test
+        ports:
+          - 5432:5432
+      mysql:
+        image: mysql:8.0
+        env:
+          MYSQL_USER: test_user
+          MYSQL_PASSWORD: password
+          MYSQL_DATABASE: test
+          MYSQL_RANDOM_ROOT_PASSWORD: "yes"
+        ports:
+          - 3306:3306
+    steps:
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        with:
+          persist-credentials: false
+      - name: Set up Python ${{ env.LATEST_SUPPORTED_PY }}
+        uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
+        with:
+          python-version: ${{ env.LATEST_SUPPORTED_PY }}
+          cache: pip
+      - name: Install dependencies
+        run: |
+          pip install -U pip
+          pip install -r requirements/testing.txt
+          pip install -r requirements/optional.txt
+          pip install -r requirements/databases.txt
+      - name: Run sync tests (${{ matrix.database }})
+        env:
+          TEST_DATABASE_URL: ${{ matrix.sync_url }}
+        run: |
+          PYTHONPATH=$PWD:$PYTHONPATH pytest tests/slack_sdk/oauth/installation_store/test_sqlalchemy.py
+          PYTHONPATH=$PWD:$PYTHONPATH pytest tests/slack_sdk/oauth/state_store/test_sqlalchemy.py
+      - name: Run async tests (${{ matrix.database }})
+        env:
+          ASYNC_TEST_DATABASE_URL: ${{ matrix.async_url }}
+        run: |
+          PYTHONPATH=$PWD:$PYTHONPATH pytest tests/slack_sdk/oauth/installation_store/test_async_sqlalchemy.py
+          PYTHONPATH=$PWD:$PYTHONPATH pytest tests/slack_sdk/oauth/state_store/test_async_sqlalchemy.py
+
   notifications:
     name: Regression notifications
     runs-on: ubuntu-latest
     needs:
       - lint
       - typecheck
       - unittest
+      - databases
     if: ${{ !success() && github.ref == 'refs/heads/main' && github.event_name != 'workflow_dispatch' }}
     steps:
       - name: Send notifications of failing tests

requirements/databases.txt

@@ -0,0 +1,9 @@
+# Database drivers for CI testing
+
+# PostgreSQL drivers
+psycopg2-binary>=2.9,<3
+asyncpg>=0.27,<1
+
+# MySQL drivers
+PyMySQL>=1.0,<2
+aiomysql>=0.1,<1

tests/slack_sdk/oauth/installation_store/test_async_sqlalchemy.py

@@ -1,3 +1,4 @@
+import os
 import unittest
 from tests.helpers import async_test
 from sqlalchemy.ext.asyncio import AsyncEngine, create_async_engine
@@ -11,7 +12,7 @@ class TestAsyncSQLAlchemy(unittest.TestCase):
 
     @async_test
     async def setUp(self):
-        self.engine = create_async_engine("sqlite+aiosqlite:///:memory:")
+        self.engine = create_async_engine(os.environ.get("ASYNC_TEST_DATABASE_URL", "sqlite+aiosqlite:///:memory:"))
         self.store = AsyncSQLAlchemyInstallationStore(client_id="111.222", engine=self.engine)
         async with self.engine.begin() as conn:
             await conn.run_sync(self.store.metadata.create_all)
@@ -298,7 +299,6 @@ async def test_issue_1441_mixing_user_and_bot_installations(self):
         self.assertIsNone(installation)
 
     async def test_timezone_aware_datetime_compatibility(self):
-        """Test that timezone-aware datetimes work with database storage"""
         installation = Installation(
             app_id="A111",
             enterprise_id="E111",

tests/slack_sdk/oauth/installation_store/test_sqlalchemy.py

@@ -1,3 +1,4 @@
+import os
 import unittest
 
 import sqlalchemy
@@ -11,7 +12,7 @@ class TestSQLAlchemy(unittest.TestCase):
     engine: Engine
 
     def setUp(self):
-        self.engine = sqlalchemy.create_engine("sqlite:///:memory:")
+        self.engine = sqlalchemy.create_engine(os.environ.get("TEST_DATABASE_URL", "sqlite:///:memory:"))
         self.store = SQLAlchemyInstallationStore(client_id="111.222", engine=self.engine)
         self.store.metadata.create_all(self.engine)
 
@@ -291,7 +292,6 @@ def test_issue_1441_mixing_user_and_bot_installations(self):
         self.assertIsNone(installation)
 
     def test_timezone_aware_datetime_compatibility(self):
-        """Test that timezone-aware datetimes work with database storage"""
         installation = Installation(
             app_id="A111",
             enterprise_id="E111",

tests/slack_sdk/oauth/state_store/test_async_sqlalchemy.py

@@ -1,4 +1,5 @@
 import asyncio
+import os
 import unittest
 from tests.helpers import async_test
 from sqlalchemy.ext.asyncio import AsyncEngine, create_async_engine
@@ -11,7 +12,7 @@ class TestSQLAlchemy(unittest.TestCase):
 
     @async_test
     async def setUp(self):
-        self.engine = create_async_engine("sqlite+aiosqlite:///:memory:")
+        self.engine = create_async_engine(os.environ.get("ASYNC_TEST_DATABASE_URL", "sqlite+aiosqlite:///:memory:"))
         self.store = AsyncSQLAlchemyOAuthStateStore(engine=self.engine, expiration_seconds=2)
         async with self.engine.begin() as conn:
             await conn.run_sync(self.store.metadata.create_all)
@@ -39,7 +40,6 @@ async def test_expiration(self):
 
     @async_test
     async def test_timezone_aware_datetime_compatibility(self):
-        """Test that timezone-aware datetimes work with database storage"""
         # Issue a state (tests INSERT with timezone-aware datetime)
         state = await self.store.async_issue()
         self.assertIsNotNone(state)

tests/slack_sdk/oauth/state_store/test_sqlalchemy.py

@@ -1,3 +1,4 @@
+import os
 import time
 import unittest
 
@@ -11,7 +12,7 @@ class TestSQLAlchemy(unittest.TestCase):
     engine: Engine
 
     def setUp(self):
-        self.engine = sqlalchemy.create_engine("sqlite:///:memory:")
+        self.engine = sqlalchemy.create_engine(os.environ.get("TEST_DATABASE_URL", "sqlite:///:memory:"))
         self.store = SQLAlchemyOAuthStateStore(engine=self.engine, expiration_seconds=2)
         self.store.metadata.create_all(self.engine)
 
@@ -33,7 +34,6 @@ def test_expiration(self):
         self.assertFalse(result)
 
     def test_timezone_aware_datetime_compatibility(self):
-        """Test that timezone-aware datetimes work with database storage"""
         # Issue a state (tests INSERT with timezone-aware datetime)
         state = self.store.issue()
         self.assertIsNotNone(state)