Skip to content

Commit

Permalink
fix: reenable test harness (#266)
Browse files Browse the repository at this point in the history
* fix: reenable test harness

* chore: allows testing TH from branch

* Fix jar name + pin th version

* chore: docs

* fix scope for test jar
  • Loading branch information
filipelautert authored Dec 13, 2024
1 parent 49ab6d2 commit e854861
Show file tree
Hide file tree
Showing 2 changed files with 85 additions and 2 deletions.
81 changes: 79 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Build and Test

on:
workflow_dispatch:
pull_request_target:
pull_request:
types:
- opened
- reopened
Expand All @@ -11,7 +11,7 @@ on:
jobs:

authorize:
environment: ${{ github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name != github.repository && 'external' || 'internal' }}
environment: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository && 'external' || 'internal' }}
runs-on: ubuntu-latest
steps:
- run: true
Expand All @@ -23,6 +23,83 @@ jobs:
with:
extraMavenArgs: -Dtest="RedshiftDatabaseTest"

prepare-database:
name: Clean and initialize database
needs: build
runs-on: ubuntu-latest
container:
image: liquibase/liquibase:latest

strategy:
fail-fast: false

steps:
- uses: actions/checkout@v2

- name: Download Artifacts
uses: actions/download-artifact@v3
with:
name: liquibase-redshift-artifacts

# FIXME the redshift jar version should come from the pom.xml file
- name: Download AWS Redshift driver
run: wget https://s3.amazonaws.com/redshift-downloads/drivers/jdbc/2.1.0.14/redshift-jdbc42-2.1.0.14.jar

- name: Add Redshift extension and driver to liquibase classpath
run: |
cp redshift-jdbc42-2.1.0.14.jar /liquibase/lib/
cp liquibase-redshift-*-SNAPSHOT.jar /liquibase/lib/
- name: Clean AWS Redshift Database
run: liquibase --username="${{ secrets.TH_DB_ADMIN }}" --password="${{ secrets.TH_DB_PASSWD }}" --url="${{ secrets.TH_REDSHIFTURL }}" dropAll

- name: Init Database
run: liquibase --classpath="src/test/resources" --changeLogFile="harness.initScript.sql" --username="${{ secrets.TH_DB_ADMIN }}" --password="${{ secrets.TH_DB_PASSWD }}" --url="${{ secrets.TH_REDSHIFTURL }}" update

integration-test:
name: Test Harness for Redshift ${{ matrix.redshift }}
needs: prepare-database
runs-on: ubuntu-latest

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

steps:
- uses: actions/checkout@v2

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

- name: Build Cache
uses: actions/[email protected]
with:
key: build-${{ github.run_number }}-${{ github.run_attempt }}
path: |
**/target/**
~/.m2/repository/org/liquibase/
- name: Harness Test Run
run: mvn -Dtest="LiquibaseHarnessSuiteIT" -DdbName=redshift -DdbUsername=${{secrets.TH_DB_ADMIN}} -DdbPassword=${{secrets.TH_DB_PASSWD}} -DdbUrl=${{secrets.TH_REDSHIFTURL}} test

- name: Foundational Harness Test Run
run: mvn -Dtest="LiquibaseHarnessFoundationalSuiteTest" -DdbName=redshift -DdbUsername=${{secrets.TH_DB_ADMIN}} -DdbPassword=${{secrets.TH_DB_PASSWD}} -DdbUrl=${{secrets.TH_REDSHIFTURL}} test

- name: Advanced Harness Test Run
run: mvn -Dtest="LiquibaseHarnessAdvancedSuiteTest" -DdbName=redshift -DdbUsername=${{secrets.TH_DB_ADMIN}} -DdbPassword=${{secrets.TH_DB_PASSWD}} -DdbUrl=${{secrets.TH_REDSHIFTURL}} test

- name: Archive Redshift Test Results
if: ${{ always() }}
uses: actions/upload-artifact@v3
with:
name: redshift-test-results
path: build/spock-reports

dependabot-automerge:
needs: build-test
uses: liquibase/build-logic/.github/workflows/dependabot-automerge.yml@main
Expand Down
6 changes: 6 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,12 @@
<artifactId>redshift-jdbc42</artifactId>
<version>2.1.0.31</version>
</dependency>
<dependency>
<groupId>org.liquibase</groupId>
<artifactId>liquibase-test-harness</artifactId>
<version>1.0.9</version>
<scope>test</scope>
</dependency>
</dependencies>

<build>
Expand Down

0 comments on commit e854861

Please sign in to comment.