Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable integration tests in github actions #88

Merged
merged 32 commits into from
Mar 30, 2023
Merged

Conversation

nvoxland
Copy link
Contributor

@nvoxland nvoxland commented Oct 7, 2021

Removes the travis configuration and runs integration tests in github actions.

┆Issue is synchronized with this Jira Bug by Unito

@nvoxland nvoxland changed the title Integration test Enable integration tests in github actions Oct 7, 2021
@nvoxland
Copy link
Contributor Author

The main-branch build logic has shifted since this was originally done, and so some merge work will need to be done.

@filipelautert filipelautert self-assigned this Mar 29, 2023
# Conflicts:
#	.github/workflows/pull-request.yml
#	pom.xml
@github-actions
Copy link
Contributor

Testing These Changes

To test this PR, use the artifacts attached to the latest CI build

Artifacts Available:

  • liquibase-hanadb-artifacts: Zip containing the .jar file to test
  • test-reports-*: Detailed automated test results

Download with liquibase-sdk-maven-plugin

Alternately, you can use the Liquibase SDK Maven Plugin

Download the artifacts

mvn org.liquibase.ext:liquibase-sdk-maven-plugin:0.10.20:download-snapshot-artifacts -Dliquibase.sdk.repo=liquibase/liquibase-hanadb -Dliquibase.sdk.branchSearch=liquibase:integration-test -Dliquibase.sdk.downloadDirectory=download -Dliquibase.sdk.artifactPattern=*-artifacts -Dliquibase.sdk.unzipArtifacts=true

Install to your local maven cache

mvn org.liquibase.ext:liquibase-sdk-maven-plugin:0.10.20:install-snapshot -Dliquibase.sdk.repo=liquibase/liquibase-hanadb -Dliquibase.sdk.branchSearch=liquibase:integration-test

@github-actions
Copy link
Contributor

github-actions bot commented Mar 29, 2023

Test Results

20 files  ±0  20 suites  ±0   1s ⏱️ ±0s
12 tests ±0  12 ✔️ ±0  0 💤 ±0  0 ±0 
48 runs  ±0  48 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit daa1a3c. ± Comparison against base commit 75bf33d.

♻️ This comment has been updated with latest results.

@filipelautert filipelautert self-requested a review March 29, 2023 18:44
@filipelautert filipelautert requested a review from MalloD12 March 30, 2023 17:46
Copy link

@MalloD12 MalloD12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes looks good to me.

@filipelautert filipelautert merged commit 24ef209 into main Mar 30, 2023
@filipelautert filipelautert deleted the integration-test branch March 30, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants