Skip to content

Commit

Permalink
Merge branch 'main' into patch-1
Browse files Browse the repository at this point in the history
  • Loading branch information
yshyn-iohk committed Apr 29, 2024
2 parents 500ed5e + 88806d9 commit b027970
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ jobs:
- name: Extract test results
id: analyze_test_results
if: github.ref_name == 'main'
if: github.ref_name == 'main' && always()
run: |
JSON_RESULTS="target/site/serenity/serenity-summary.json"
CONCLUSION=failure
Expand Down Expand Up @@ -127,14 +127,14 @@ jobs:
check_name: "Integration Test Results"

- name: Upload artifacts
if: github.ref_name == 'main' || steps.analyze_test_results.outputs.conclusion == 'failure'
if: github.ref_name == 'main' || failure()
uses: actions/upload-artifact@v4
with:
name: integration-tests-result
path: ${{ env.REPORTS_DIR }}

- name: Slack Notification
if: github.ref_name == 'main' && steps.analyze_test_results.outputs.conclusion == 'failure'
if: github.ref_name == 'main' && failure()
uses: rtCamp/action-slack-notify@v2
env:
SLACK_COLOR: ${{ steps.analyze_test_results.outputs.conclusion }}
Expand Down
2 changes: 1 addition & 1 deletion tests/integration-tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ Here is an example of the agent configuration for sandbox environment:
1. Docker and Docker Compose are installed.
2. JDK >=17.
3. IntelliJ IDEA with Kotlin and Cucumber plugins.
4. `ATALA_GITHUB_ACTOR` and `ATALA_GITHUB_TOKEN` environment variables are set to download dependencies.
4. `GITHUB_ACTOR` and `GITHUB_TOKEN` environment variables are set to download dependencies.

> If you have troubles with JDK version incompatibilities in IntelliJ IDEA,
> make sure JDK is set not only for the project itself, but also for the Gradle daemon.
Expand Down

0 comments on commit b027970

Please sign in to comment.