Skip to content

Commit

Permalink
ci: enable Java integration tests
Browse files Browse the repository at this point in the history
Fixes #728.
  • Loading branch information
lidavidm committed Sep 11, 2023
1 parent a9c381d commit 864116b
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 15 deletions.
20 changes: 5 additions & 15 deletions .github/workflows/java.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,19 +43,6 @@ jobs:
strategy:
matrix:
java: ['8', '11', '17', '20']
services:
postgres:
image: postgres
env:
POSTGRES_DB: postgres
POSTGRES_PASSWORD: password
options: >-
--health-cmd pg_isready
--health-interval 10s
--health-timeout 5s
--health-retries 5
ports:
- 5432:5432
steps:
- uses: actions/checkout@v3
with:
Expand All @@ -66,13 +53,16 @@ jobs:
cache: "maven"
distribution: "temurin"
java-version: ${{ matrix.java }}
- name: Start SQLite server
- name: Start test services
shell: bash -l {0}
run: |
docker-compose up -d flightsql-sqlite-test
docker compose up -d --wait flightsql-sqlite-test postgresql-test
- name: Build/Test
env:
ADBC_SQLITE_FLIGHTSQL_URI: "grpc+tcp://localhost:8080"
ADBC_JDBC_MSSQL_URL: "localhost:1433"
ADBC_JDBC_MSSQL_USER: "SA"
ADBC_JDBC_MSSQL_PASSWORD: "Password1!"
ADBC_JDBC_POSTGRESQL_URL: "localhost:5432/postgres"
ADBC_JDBC_POSTGRESQL_USER: "postgres"
ADBC_JDBC_POSTGRESQL_PASSWORD: "password"
Expand Down
9 changes: 9 additions & 0 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,15 @@ services:
ports:
- 8080:8080

mssql-test:
container_name: adbc_mssql_test
image: mcr.microsoft.com/mssql/server:2022-latest
environment:
ACCEPT_EULA: "Y"
MSSQL_SA_PASSWORD: "Password1!"
ports:
- "1433:1433"

postgres-test:
container_name: adbc_postgres_test
image: postgres:latest
Expand Down

0 comments on commit 864116b

Please sign in to comment.