gzhao9 commented on code in PR #16092:
URL: https://github.com/apache/druid/pull/16092#discussion_r1518802441
##########
services/src/test/java/org/apache/druid/server/AsyncQueryForwardingServletTest.java:
##########
@@ -484,43 +401,26 @@
public void
testHandleQueryParseExceptionWithFilterEnabledButMessageMatchAllowedRegex()
throws Exception
{
String errorMessage = "test exception message";
- ObjectMapper mockMapper = Mockito.mock(ObjectMapper.class);
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- HttpServletResponse response = Mockito.mock(HttpServletResponse.class);
- ServletOutputStream outputStream = Mockito.mock(ServletOutputStream.class);
- Mockito.when(response.getOutputStream()).thenReturn(outputStream);
- final AsyncQueryForwardingServlet servlet = new
AsyncQueryForwardingServlet(
- new MapQueryToolChestWarehouse(ImmutableMap.of()),
- mockMapper,
- TestHelper.makeSmileMapper(),
- null,
- null,
- null,
- new NoopServiceEmitter(),
- new NoopRequestLogger(),
- new DefaultGenericQueryMetricsFactory(),
- new AuthenticatorMapper(ImmutableMap.of()),
- new Properties(),
- new ServerConfig()
- {
- @Override
- public boolean isShowDetailedJettyErrors()
- {
- return true;
- }
+ boolean haveRequest = true;
Review Comment:
These three have been removed. @kfaraz
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
