Skip to content

Conversation

@rustyrazorblade
Copy link
Contributor

No description provided.

name: unit-test-results-java-${{ matrix.java }}
path: |
**/build/test-results/test/*.xml
**/build/reports/tests/
Copy link
Contributor

Choose a reason for hiding this comment

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

test reports, index.html files are also placed in **/build/test-results/test/

In build.gradle

    reports {
        junitXml.setRequired(true)
        def destDir = Paths.get(rootProject.rootDir.absolutePath, "build", "test-results", "test").toFile()
        println("Destination directory for unit tests: ${destDir}")
        junitXml.getOutputLocation().set(destDir)
        html.setRequired(true)
        html.getOutputLocation().set(destDir)
    }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Switched it to **/build/test-results/test/** to grab everything.

Implements complete GitHub Actions pipeline replacing CircleCI:

Main CI (.github/workflows/ci.yml):
- Test matrix: Java 11/17/21 × Cassandra 4.0/4.1/5.0/5.1
- Cached dtest jar builds (saves ~25 min on subsequent runs)
- Unit tests with static analysis (SpotBugs, Apache RAT, JaCoCo)
- Lightweight + heavyweight integration tests (48 combinations total)
- Checkstyle/Javadoc disabled pending fixes

Artifact Publishing (.github/workflows/publish-test-artifacts.yml):
- Auto-publishes after successful trunk CI runs
- Docker: ghcr.io/apache/cassandra-sidecar:latest
- Tarball: GitHub release at 'test-artifacts' tag

Dependabot (.github/dependabot.yml):
- Weekly Gradle dependency updates
- Weekly GitHub Actions version updates
Copy link
Contributor

@yifan-c yifan-c left a comment

Choose a reason for hiding this comment

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

A great start

@rustyrazorblade rustyrazorblade merged commit 37a881f into trunk Nov 11, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants