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

Continuous testing quarkus:test goal requires @QuarkusTest annotation #17717

Closed
kwakiutlCS opened this issue Jun 6, 2021 · 1 comment · Fixed by #17719
Closed

Continuous testing quarkus:test goal requires @QuarkusTest annotation #17717

kwakiutlCS opened this issue Jun 6, 2021 · 1 comment · Fixed by #17719
Labels
Milestone

Comments

@kwakiutlCS
Copy link

Describe the bug

Maven's quarkus:test goal seems to require at least one @QuarkusTest annotated test to run the tests. If no test is annotated, they don't run. If at least one is annotated, all tests run.
The quarkus:dev goal runs all the tests even without any annotation.

Expected behavior

mvn quarkus:test runs the same tests as mvn quarkus:dev and they don't necessarily require @QuarkusTest.

Actual behavior

mvn quarkus:test only runs tests if at least one is annotated with @QuarkusTest

To Reproduce

Reproducer https://github.com/kwakiutlCS/vigilant-funicular

Steps to reproduce the behavior:

  1. run mvn quarkus:test (hangs)
  2. run mvn quarkus:dev and run tests (succeeds)
  3. add a second test file with @QuarkusTest (go back 1 commit) and run mvn quarkus:test (succeeds)

Quarkus version or git rev

Quarkus 2.0.0.CR3

Additional context

Sometimes besides hanging a log message appears
INFO [org.jun.pla.lau.cor.EngineDiscoveryOrchestrator] (Test runner thread) 0 containers and XX tests were Unknown
where XX is the number of tests

@kwakiutlCS kwakiutlCS added the kind/bug Something isn't working label Jun 6, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Jun 6, 2021

/cc @stuartwdouglas

stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Jun 6, 2021
stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Jun 7, 2021
@quarkus-bot quarkus-bot bot added this to the 2.1 - main milestone Jun 7, 2021
@gsmet gsmet modified the milestones: 2.1 - main, 2.0.0.Final Jun 7, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants