Skip to content

Commit 152d81e

Browse files
authored
ci: re-enable integration tests on CI and align Java version on CI (#383)
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
1 parent 40b25f7 commit 152d81e

File tree

4 files changed

+17
-6
lines changed

4 files changed

+17
-6
lines changed

.github/workflows/flow-deploy-release-artifact.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ on:
66
description: "Java JDK Version:"
77
type: string
88
required: false
9-
default: "21.0.3"
9+
default: "21.0.5"
1010
java-distribution:
1111
description: "Java JDK Distribution:"
1212
type: string
@@ -103,7 +103,7 @@ jobs:
103103
uses: actions/setup-java@7a6d8a8234af8eb26422e24e3006232cccaa061b # v4.6.0
104104
with:
105105
distribution: ${{ github.event.inputs.java-distribution || 'temurin' }}
106-
java-version: ${{ github.event.inputs.java-version || '21.0.3' }}
106+
java-version: ${{ github.event.inputs.java-version || '21.0.5' }}
107107

108108
- name: Setup Gradle
109109
uses: gradle/actions/setup-gradle@0bdd871935719febd78681f197cd39af5b6e16a6 # v4.2.2

.github/workflows/zxc-compile-pbj-code.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ on:
2626
description: "Java JDK Version:"
2727
type: string
2828
required: false
29-
default: "21.0.3"
29+
default: "21.0.5"
3030
gradle-version:
3131
description: "Gradle Version:"
3232
type: string
@@ -122,7 +122,7 @@ jobs:
122122

123123
- name: Gradle Assemble (PBJ Integration)
124124
id: gradle-integration-build
125-
if: ${{ (inputs.enable-integration-tests || inputs.enable-jmh-tests) && steps.gradle-publish-local.conclusion == 'success' && !cancelled() }}
125+
if: ${{ (inputs.enable-integration-tests || inputs.enable-jmh-tests) && !cancelled() }}
126126
working-directory: ${{ env.PBJ_INTEGRATION_TESTS }}
127127
run: ./gradlew assemble
128128

pbj-integration-tests/build.gradle.kts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ mainModuleInfo {
2727

2828
requires("com.google.common")
2929
requires("com.google.protobuf")
30-
requires("com.google.protobuf.util")
3130
requires("io.grpc")
3231
requires("io.grpc.protobuf")
3332
requires("io.grpc.stub")
@@ -38,10 +37,14 @@ mainModuleInfo {
3837
testModuleInfo {
3938
requires("org.junit.jupiter.api")
4039
requires("org.junit.jupiter.params")
40+
requires("com.google.protobuf.util")
4141
runtimeOnly("org.junit.jupiter.engine")
4242
}
4343

44-
jmhModuleInfo { requires("com.hedera.pbj.runtime") }
44+
jmhModuleInfo {
45+
requires("com.hedera.pbj.runtime")
46+
requires("com.google.protobuf.util")
47+
}
4548

4649
// IMPROVE: Disable module-info transform for 'testRuntimeClasspath' which leads to an error
4750
// possible caused by a cycle produced by depending on 'pbj-compiler' in multiple ways which

pbj-integration-tests/settings.gradle.kts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,14 @@ pluginManagement {
55

66
plugins { id("org.hiero.gradle.build") version "0.3.1" }
77

8+
// Downgrade 'dependency-analysis-gradle-plugin' due to regression in 2.7.0
9+
// https://github.com/autonomousapps/dependency-analysis-gradle-plugin/issues/1364
10+
buildscript {
11+
dependencies.constraints {
12+
classpath("com.autonomousapps:dependency-analysis-gradle-plugin:2.6.0!!")
13+
}
14+
}
15+
816
dependencyResolutionManagement {
917
// To use locally built 'pbj-runtime'
1018
includeBuild("../pbj-core")

0 commit comments

Comments
 (0)