Convert restlet 1.1 tests from groovy to java #12337
Conversation
| } | ||
| } | ||
|
|
||
| public static class ExceptionResource extends BaseResource { |
There was a problem hiding this comment.
| public static class ExceptionResource extends BaseResource { | |
| static class ExceptionResource extends BaseResource { |
| } | ||
| } | ||
|
|
||
| public static class QueryParamResource extends BaseResource { |
There was a problem hiding this comment.
| public static class QueryParamResource extends BaseResource { | |
| static class QueryParamResource extends BaseResource { |
There was a problem hiding this comment.
these need to be public so jetty can access the constructors. When testing them as package private i see the following:
18:26:18.461 [qtp1342212468-54] WARN org.eclipse.jetty.util.log - Exception while instantiating the target handler.
java.lang.NoSuchMethodException: io.opentelemetry.instrumentation.restlet.v1_1.RestletAppTestBase$QueryParamResource.()
and for PathParamResource
18:27:52.806 [qtp111210112-54] WARN org.eclipse.jetty.util.log - Exception while instantiating the target handler.
java.lang.NoSuchMethodException: io.opentelemetry.instrumentation.restlet.v1_1.RestletAppTestBase$PathParamResource.()
etc
| } | ||
| } | ||
|
|
||
| public static class PathParamResource extends BaseResource { |
There was a problem hiding this comment.
| public static class PathParamResource extends BaseResource { | |
| static class PathParamResource extends BaseResource { |
| } | ||
| } | ||
|
|
||
| public static class RedirectResource extends BaseResource { |
There was a problem hiding this comment.
| public static class RedirectResource extends BaseResource { | |
| static class RedirectResource extends BaseResource { |
| } | ||
| } | ||
|
|
||
| public static class CaptureHeadersResource extends BaseResource { |
There was a problem hiding this comment.
| public static class CaptureHeadersResource extends BaseResource { | |
| static class CaptureHeadersResource extends BaseResource { |
| } | ||
| } | ||
|
|
||
| public static class IndexedChildResource extends BaseResource { |
There was a problem hiding this comment.
| public static class IndexedChildResource extends BaseResource { | |
| static class IndexedChildResource extends BaseResource { |
part of #7195