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

NoDefaultServletTest cannot find property quarkus.camel.servlet.url-patterns #853

Closed
jamesnetherton opened this issue Mar 6, 2020 · 6 comments · Fixed by #1046
Closed
Labels
bug Something isn't working

Comments

@jamesnetherton
Copy link
Contributor

[INFO] Running org.apache.camel.quarkus.component.servlet.test.NoDefaultServletTest
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.179 s <<< FAILURE! - in org.apache.camel.quarkus.component.servlet.test.NoDefaultServletTest
[ERROR] org.apache.camel.quarkus.component.servlet.test.NoDefaultServletTest  Time elapsed: 0.179 s  <<< ERROR!
java.lang.RuntimeException: java.lang.RuntimeException: java.util.NoSuchElementException: Property quarkus.camel.servlet.url-patterns not found
Caused by: java.lang.RuntimeException: java.util.NoSuchElementException: Property quarkus.camel.servlet.url-patterns not found
Caused by: java.util.NoSuchElementException: Property quarkus.camel.servlet.url-patterns not found
@jamesnetherton jamesnetherton added the bug Something isn't working label Mar 6, 2020
@lburgazzoli
Copy link
Contributor

@jamesnetherton I don't see this any-more, can you confirm ?

@jamesnetherton
Copy link
Contributor Author

The issue still seems to be present.

You need to remove the @Disabled annotation.

@lburgazzoli
Copy link
Contributor

ah right

@jamesnetherton
Copy link
Contributor Author

Do we even need those tests in the Servlet extension deployment module? They are more or less repeated over in the main integration test area.

@lburgazzoli
Copy link
Contributor

@ppalaga WDYT ?

@ppalaga
Copy link
Contributor

ppalaga commented Apr 6, 2020

Do we even need those tests in the Servlet extension deployment module? They are more or less repeated over in the main integration test area.

I'd say so, they were supposed to test various configs in isolation. Let me check what is the problem.

ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Apr 7, 2020
lburgazzoli pushed a commit that referenced this issue Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants