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

Test execution message is now copy/paste friendly #19890

Merged
merged 1 commit into from
Sep 3, 2021

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Sep 3, 2021

The "Executing" message shown in the ArtifactLauncher implementations now allows you to copy/paste the command easily

eg.
From:

Executing [/home/ggastald/Downloads/code-with-quarkus/target/code-with-quarkus-1.0.0-SNAPSHOT-runner, -Dquarkus.http.port=8081, -Dquarkus.http.ssl-port=8444, -Dtest.url=http://localhost:8081, -Dquarkus.log.file.path=/home/ggastald/Downloads/code-with-quarkus/target/quarkus.log, -Dquarkus.log.file.enable=true]

To

Executing "/home/ggastald/Downloads/code-with-quarkus/target/code-with-quarkus-1.0.0-SNAPSHOT-runner -Dquarkus.http.port=8081 -Dquarkus.http.ssl-port=8444 -Dtest.url=http://localhost:8081 -Dquarkus.log.file.path=/home/ggastald/Downloads/code-with-quarkus/target/quarkus.log -Dquarkus.log.file.enable=true"

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
The "Executing" message shown in the ArtifactLauncher implementations now allows you to copy/paste the command easily
@gastaldi gastaldi requested a review from geoand September 3, 2021 01:21
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 3, 2021

Failing Jobs - Building e9e4d48

Status Name Step Failures Logs Raw logs
Gradle Tests - JDK 11 Windows Build Failures Logs Raw logs
✔️ JVM Tests - JDK 11
JVM Tests - JDK 11 Windows Build Failures Logs Raw logs
✔️ JVM Tests - JDK 16

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.devmode.BasicKotlinApplicationModuleDevModeTest.main line 18 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:166)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/amazon-lambda/deployment 
! Skipped: docs extensions/amazon-lambda-http/deployment extensions/amazon-lambda-rest/deployment and 6 more

📦 extensions/amazon-lambda/deployment

io.quarkus.amazon.lambda.deployment.testing.LambdaDevServicesContinuousTestingTestCase.testLambda line 45 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: expected: <1> but was: <0>
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
	at org.junit.jupiter.api.AssertionUtils.failNotEqual(AssertionUtils.java:62)

Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

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

Thanks!

@geoand geoand merged commit e4073c9 into quarkusio:main Sep 3, 2021
@quarkus-bot quarkus-bot bot added this to the 2.3 - main milestone Sep 3, 2021
@gsmet
Copy link
Member

gsmet commented Sep 6, 2021

This is a very good idea!

@gastaldi gastaldi deleted the executing branch September 6, 2021 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants