Skip to content

Commit

Permalink
tests(datahub-client): new tests for the AvroSchemaConverter (#12087)
Browse files Browse the repository at this point in the history
  • Loading branch information
sgomezvillamor authored Dec 16, 2024
1 parent ecf6c8c commit d2ffeb0
Show file tree
Hide file tree
Showing 13 changed files with 1,598 additions and 750 deletions.
17 changes: 12 additions & 5 deletions .github/workflows/check-datahub-jars.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@ on:
branches:
- master
paths:
- "metadata-integration"
- "metadata-integration/**"
pull_request:
branches:
- "**"
paths:
- "metadata-integration"
- "metadata-integration/**"
release:
types: [published]

Expand All @@ -28,15 +28,22 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: acryldata/sane-checkout-action@v3
- uses: actions/setup-python@v5
with:
python-version: "3.10"
- uses: actions/cache@v4
with:
path: |
~/.cache/uv
key: ${{ runner.os }}-uv-${{ hashFiles('**/requirements.txt') }}
- name: Install dependencies
run: ./metadata-ingestion/scripts/install_deps.sh
- name: Set up JDK 17
uses: actions/setup-java@v4
with:
distribution: "zulu"
java-version: 17
- uses: gradle/actions/setup-gradle@v3
- uses: actions/setup-python@v5
with:
python-version: "3.10"
- name: check ${{ matrix.command }} jar
run: |
./gradlew :metadata-integration:java:${{ matrix.command }}:build --info
Expand Down
5 changes: 5 additions & 0 deletions metadata-integration/java/datahub-client/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,11 @@ test {
finalizedBy jacocoTestReport
}

// no submodule depends on datahub-schematron:cli
// and tests there are the ones checking python-java compatibility
test.dependsOn tasks.getByPath(":metadata-integration:java:datahub-schematron:cli:test")
test.dependsOn tasks.getByPath(":metadata-integration:java:datahub-schematron:lib:test")

task checkShadowJar(type: Exec) {
commandLine 'sh', '-c', 'scripts/check_jar.sh'
}
Expand Down
Loading

0 comments on commit d2ffeb0

Please sign in to comment.