CI: run SSL tests only when building with TLS support

launchbadge · Feb 15, 2023 · 6b8446f · 6b8446f
1 parent d32139b
commit 6b8446f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/.github/workflows/sqlx.yml b/.github/workflows/sqlx.yml
@@ -240,6 +240,7 @@ jobs:
           docker exec postgres_${{ matrix.postgres }}_client_ssl bash -c "until pg_isready; do sleep 1; done"
 
       - uses: actions-rs/cargo@v1
+        if: matrix.tls != 'none'
         with:
           command: test
           args: >
@@ -313,7 +314,7 @@ jobs:
 
       # MySQL 5.7 supports TLS but not TLSv1.3 as required by RusTLS.
       - uses: actions-rs/cargo@v1
-        if: ${{ !(matrix.mysql == '5_7' && matrix.tls == 'rustls') }}
+        if: ${{ matrix.mysql != '5_7' || matrix.tls == 'native-tls' }}
         with:
           command: test
           args: >
@@ -372,6 +373,7 @@ jobs:
           sleep 60
 
       - uses: actions-rs/cargo@v1
+        if: matrix.tls != 'none'
         with:
           command: test
           args: >

diff --git a/tests/docker.py b/tests/docker.py
@@ -36,9 +36,6 @@ def start_database(driver, database, cwd):
     elif driver.startswith("postgres"):
         port = 5432
 
-    elif driver.startswith("mssql"):
-        port = 1433
-
     else:
         raise NotImplementedError
 
@@ -80,8 +77,5 @@ def start_database(driver, database, cwd):
     elif driver.startswith("postgres"):
         return f"postgres://postgres{password}@localhost:{port}/{database}"
 
-    elif driver.startswith("mssql"):
-        return f"mssql://sa:[email protected]:{port}/{database}"
-
     else:
         raise NotImplementedError