-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Remove URI validation when @TestHTTPResource
is injected as String
#34968
Conversation
@TestHTTPResource
is injected as String
Thanks for this. We'll need some tests for this as well |
I tried this test inside
I got this exception:
Note that I had to add these test dependencies to
|
You can't use You can only use The alternative is to use |
I tried
|
It doesn't work inside the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
The CI needs an approval to start up :) |
👌🏼 |
CI fails because of imports order. Force-pushed a new commit. @geoand Can you please start CI again? Thanks. |
Failing Jobs - Building 11d6a01
Full information is available in the Build summary check run. Failures⚙️ JVM Tests - JDK 17 #- Failing: integration-tests/grpc-hibernate
📦 integration-tests/grpc-hibernate✖
⚙️ JVM Tests - JDK 17 Windows #- Failing: integration-tests/grpc-hibernate
📦 integration-tests/grpc-hibernate✖
⚙️ Maven Tests - JDK 11 Windows #📦 integration-tests/maven✖
✖
|
Fixes #34939