Skip to content

Commit

Permalink
Merge pull request #146 from liquibase/build-test
Browse files Browse the repository at this point in the history
fix: remove integration tests executions as there are no integration tests
  • Loading branch information
filipelautert authored Dec 10, 2024
2 parents 0a465fa + 72ae976 commit d68577f
Showing 1 changed file with 1 addition and 42 deletions.
43 changes: 1 addition & 42 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ permissions:
pull-requests: write

jobs:

authorize:
environment: ${{ github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name != github.repository && 'external' || 'internal' }}
runs-on: ubuntu-latest
Expand All @@ -25,47 +24,7 @@ jobs:
uses: liquibase/build-logic/.github/workflows/os-extension-test.yml@main
secrets: inherit

integration-test:
needs: authorize
name: Test Harness for Maxdb ${{ matrix.maxdb }}
runs-on: ubuntu-latest

strategy:
fail-fast: false
matrix:
maxdb: [ "" ]

steps:
- uses: actions/checkout@v4

- name: Set up JDK
uses: actions/setup-java@v4
with:
java-version: 17
distribution: 'temurin'
cache: 'maven'

- name: Build Cache
uses: actions/cache@v3.3.3
with:
key: build-${{ github.run_number }}-${{ github.run_attempt }}
path: |
**/target/**
~/.m2/repository/org/liquibase/
- name: Run Tests
run: mvn -B jacoco:prepare-agent surefire:test -Dmaxdb.version=${{ matrix.maxdb }}

- name: Archive Test Results
if: ${{ always() }}
uses: actions/upload-artifact@v4
with:
name: test-reports-maxdb-${{ matrix.maxdb }}
path: |
**/target/surefire-reports
**/target/jacoco.exec
dependabot-automerge:
needs: integration-test
needs: build-test
uses: liquibase/build-logic/.github/workflows/dependabot-automerge.yml@main
secrets: inherit

0 comments on commit d68577f

Please sign in to comment.