Skip to content

Commit 345256e

Browse files
committed
Run CTS test as part of linux-amd64 CI job (1.3.2)
This is a backport of the PR duckdb#317 to `v1.3.2` stable branch. This change moves the compatibility test runs from a separate job into the `linux-amd64` job that is run first during the build. Updated CTS repo in `duckdb` org is used.
1 parent 8800165 commit 345256e

File tree

1 file changed

+33
-34
lines changed

1 file changed

+33
-34
lines changed

.github/workflows/Java.yml

Lines changed: 33 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,39 @@ jobs:
8585
-cp ./build/release/duckdb_jdbc_tests.jar:./build/release/duckdb_jdbc.jar \
8686
org.duckdb.TestDuckDBJDBC
8787
88+
- name: Checkout Platform TCK
89+
if: ${{ inputs.skip_tests != 'true' }}
90+
uses: actions/checkout@v4
91+
with:
92+
repository: jakartaee/platform-tck
93+
ref: 10.0.6
94+
path: platform-tck
95+
96+
- name: Checkout CTS Runner
97+
if: ${{ inputs.skip_tests != 'true' }}
98+
uses: actions/checkout@v4
99+
with:
100+
repository: duckdb/jdbc_compatibility_test_suite_runner
101+
path: jdbc_compatibility_test_suite_runner
102+
103+
- name: CTS tests
104+
if: ${{ inputs.skip_tests != 'true' }}
105+
shell: bash
106+
run: |
107+
docker run \
108+
-v.:/duckdb \
109+
-e JAVA_HOME=/usr/lib/jvm/java-1.8.0-openjdk \
110+
-e DUCKDB_JAR=/duckdb/build/release/duckdb_jdbc.jar \
111+
-e PLATFORM_TCK_DIR=/duckdb/platform-tck \
112+
${{ env.MANYLINUX_IMAGE }} \
113+
bash -c "
114+
set -e
115+
cat /etc/os-release
116+
dnf install -y \
117+
java-1.8.0-openjdk-devel
118+
make -C /duckdb/jdbc_compatibility_test_suite_runner test
119+
"
120+
88121
- name: Deploy
89122
shell: bash
90123
run: |
@@ -407,40 +440,6 @@ jobs:
407440
path: |
408441
jdbc-artifacts
409442
410-
jdbc-compliance:
411-
name: JDBC Compliance
412-
runs-on: ubuntu-latest
413-
if: ${{ inputs.skip_tests != 'true' }}
414-
needs: java-linux-amd64
415-
container: quay.io/pypa/manylinux_2_28_x86_64
416-
env:
417-
GEN: ninja
418-
JAVA_HOME: /usr/lib/jvm/java-1.8.0-openjdk
419-
420-
steps:
421-
- uses: actions/checkout@v4
422-
with:
423-
fetch-depth: 0
424-
ref: ${{ inputs.git_ref }}
425-
426-
- name: Install packages
427-
shell: bash
428-
run: |
429-
dnf install java-1.8.0-openjdk-devel ninja-build -y
430-
431-
- name: Install CTS
432-
shell: bash
433-
run: |
434-
git clone https://github.com/cwida/jdbccts.git
435-
436-
- name: Build
437-
shell: bash
438-
run: make release
439-
440-
- name: Test
441-
shell: bash
442-
run: (cd jdbccts && make DUCKDB_JAR=../build/release/duckdb_jdbc.jar test)
443-
444443
java-merge-vendoring-pr:
445444
name: Merge vendoring PR
446445
if: ${{ github.repository == 'duckdb/duckdb-java' && github.event_name == 'pull_request' && github.head_ref == format('vendoring-{0}', github.base_ref) }}

0 commit comments

Comments
 (0)