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

Tests run in DEV mode are now logged with short names, previously signature was present #33744

Closed
michalvavrik opened this issue May 31, 2023 · 2 comments · Fixed by #33757
Closed
Labels
area/devmode kind/bug Something isn't working
Milestone

Comments

@michalvavrik
Copy link
Member

michalvavrik commented May 31, 2023

Describe the bug

Before #33685 our DEV mode test contained:

17:45:10,608 INFO  [app] Running 1/1. Running: io.quarkus.ts.http.minimum.HelloResourceTest#HelloResourceTest

now it contains

17:53:53,032 INFO  [app] Running 1/1. Running: #HelloResourceTest

I can so no information about this in PR description or note in migration guide or anything, therefore there is no way to tell whether this change is intentional or side effect.

Expected behavior

Previous behavior was clearer, but I just need a confirmation this was intentional change, that's all.

Actual behavior

io.quarkus.ts.http.minimum.HelloResourceTest#HelloResourceTest => HelloResourceTest

How to Reproduce?

Reproducer:

  1. git clone [email protected]:michalvavrik/quarkus-test-suite.git
  2. cd quarkus-test-suite/http/http-minimum-reactive
  3. git checkout -b reproducer/domain-socket-vertx-bump
  4. mvn clean verify -Dit.test=DevModeHttpMinimumReactiveIT

Output of uname -a or ver

Fedora 38

Output of java -version

17

GraalVM version (if different from Java)

22.3

Quarkus version or git rev

999-SNAPSHOT

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.9.1

Additional information

No response

@michalvavrik michalvavrik added the kind/bug Something isn't working label May 31, 2023
@michalvavrik
Copy link
Member Author

/cc @stuartwdouglas @mkouba

@rsvoboda
Copy link
Member

rsvoboda commented Jun 1, 2023

I would say FQCN should be present in the log

stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Jun 1, 2023
@quarkus-bot quarkus-bot bot added this to the 3.2 - main milestone Jun 2, 2023
sberyozkin pushed a commit to sberyozkin/quarkus that referenced this issue Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devmode kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants