Smallrye GraphQL 2.5.0

[jmartisk]

• test coverage for federation batch resolving

[jmartisk]

Also cc @robp94 , you authored some parts of this

[sberyozkin]

Hey @jmartisk Thanks very much.

I wonder if we should have 2 PRs, one - bumping the version, some tests, and then a dedicated one related to a new release noteworthy feature ?

[jmartisk]

Hey @jmartisk Thanks very much.

I wonder if we should have 2 PRs, one - bumping the version, some tests, and then a dedicated one related to a new release noteworthy feature ?

I squashed them into one because the OIDC client integration also relies on new functionality in SR-GQL 2.5.0. Possibly we can split it, but I don't see a strong reason why. One of the PRs would depend on merging the other first.

[sberyozkin]

@jmartisk Right, the first PR, dealing with the smallrye-graphql version bump only would be merged, and then a new PR introducing oidc-client-graphql would be opened, with the label highlighting its introduction - IMHO it is worth it

[jmartisk]

Ok then; I've removed the commit with the new OIDC client extension from this PR..

[sberyozkin]

Thanks @jmartisk, your enhancement is worthy a dedicated PR and highlight in the release notes :-)

[sberyozkin]

TCK test failures seem to be related

[jmartisk]

TCKs should hopefully be fixed now...

[quarkus-bot]

Failing Jobs - Building bba3785

Status	Name	Step	Failures	Logs	Raw logs	Build scan
✖	JVM Tests - JDK 11	Build	Failures	Logs	Raw logs
✔️	JVM Tests - JDK 17
✖	JVM Tests - JDK 20	Build	⚠️ Check →	Logs	Raw logs
✔️	Maven Tests - JDK 11
✖	Maven Tests - JDK 11 Windows	Build	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: integration-tests/rest-client-reactive 

📦 integration-tests/rest-client-reactive

✖ Failed to execute goal uk.co.automatictester:truststore-maven-plugin:3.0.0:generate-truststore (self-signed-truststore) on project quarkus-integration-test-rest-client-reactive: Execution self-signed-truststore of goal uk.co.automatictester:truststore-maven-plugin:3.0.0:generate-truststore failed: java.net.ConnectException: Connection timed out (Connection timed out)

---

⚙️ Maven Tests - JDK 11 Windows #

- Failing: integration-tests/maven 

📦 integration-tests/maven

✖ io.quarkus.maven.it.DevMojoIT.testExternalReloadableArtifacts line 1448 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.maven.it.DevMojoIT was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

✖ io.quarkus.maven.it.JarRunnerIT.testNonAsciiDir line 70 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

✖ io.quarkus.maven.it.DevMojoIT.testExternalReloadableArtifacts line 1448 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.maven.it.DevMojoIT was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

✖ io.quarkus.maven.it.JarRunnerIT.testNonAsciiDir line 70 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

[phillip-kruger]

Nice ! Thanks @jmartisk